Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8186666

Bug in the C2 matcher code

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P4 P4
    • 10
    • 9, 10
    • hotspot
    • b22

        /* CompareAndSet, Long: */
        case vmIntrinsics::_compareAndSetLong:
          if (!Matcher::match_rule_supported(Op_CompareAndSwapL)) return false;
          break;
        case vmIntrinsics::_weakCompareAndSetLongPlain:
        case vmIntrinsics::_weakCompareAndSetLongAcquire:
        case vmIntrinsics::_weakCompareAndSetLongRelease:
        case vmIntrinsics::_weakCompareAndSetLong:
          if (!Matcher::match_rule_supported(Op_WeakCompareAndSwapL)) return false;
          break;

        /* CompareAndSet, Int: */
        case vmIntrinsics::_compareAndSetInt:
          if (!Matcher::match_rule_supported(Op_CompareAndSwapI)) return false;
          break;
        case vmIntrinsics::_weakCompareAndSetIntPlain:
        case vmIntrinsics::_weakCompareAndSetIntAcquire:
        case vmIntrinsics::_weakCompareAndSetIntRelease:
        case vmIntrinsics::_weakCompareAndSetInt:
          if (!Matcher::match_rule_supported(Op_WeakCompareAndSwapL)) return false;
          break;

      Note that last line:

          if (!Matcher::match_rule_supported(Op_WeakCompareAndSwapL)...

      Shouldn't that be Op_WeakCompareAndSwapI ?

            rraghavan Rahul Raghavan
            aph Andrew Haley
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: