C2: Unneeded branch in reduce_phi

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Fixed
    • Priority: P5
    • 26
    • Affects Version/s: 26
    • Component/s: hotspot
    • b21

      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

            Assignee:
            Anton Seoane Ampudia
            Reporter:
            Anton Seoane Ampudia
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: