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

Check if CMoveX is supported

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Fixed
    • Icon: P4 P4
    • 25
    • 25
    • hotspot
    • b16

      Currently, it seems CMoveX are fully supported on most platforms, except of riscv64.
      On riscv64, there is no efficient way to implement CMoveF/D as other CMoveX (e.g. CMoveI), but it will still bring benefit by just supporting CMoveX without CMoveF/D. This patch is to supply such option.

      As other platforms already supported CMoveX, this patch should not impact them, as !CMoveNode::supported(_igvn.type(phi)) should always be false.

      BTW, in a subsequent pr for riscv, I'll implement CMoveX except of CMoveF/D, and also return false for CMoveF/D in Matcher::match_rule_supported.

            mli Hamlin Li
            mli Hamlin Li
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: