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

C2: Unneeded branch in reduce_phi

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Unresolved
    • Icon: P5 P5
    • tbd
    • 26
    • hotspot

      In escape.cpp, the function reduce_phi has the following fragment:

      ```
      ...
          } else if (use->is_SafePoint()) {
            // processed later
          } else {
            assert(use->is_SafePoint(), "Unexpected user of reducible Phi %d -> %d:%s:%d", ophi->_idx, use->_idx, use->Name(), use->outcnt());
          }
      ```

      This is unnecessary as the assert will always trigger. We can either remove the else if branch or change the assert for a ShouldNotReachHere

            aseoane Anton Seoane Ampudia
            aseoane Anton Seoane Ampudia
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: