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

[REDO] PhaseIdealLoop::conditional_move is too conservative

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Unresolved
    • Icon: P4 P4
    • tbd
    • 22, 23, 24
    • hotspot
    • b03

      When transforming a Phi into a CMove, the threshold is set to be approximately BlockLayoutMinDiamondPercentage, the reason is given:

      // BlockLayoutByFrequency optimization moves infrequent branch
      // from hot path. No point in CMOV'ing in such case

      This sets the default value of the threshold to be around 18%, which is too conservative. The reason also does not make a lot of sense since the important property which makes jumping expensive is not code layout. We should remove this.

            Unassigned Unassigned
            qamai Quan Anh Mai
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: