RFR: 8352248: Check if CMoveX is supported [v2]

Hamlin Li mli at openjdk.org
Wed Mar 19 09:43:24 UTC 2025


> Hi,
> Can you help to review this patch?
> 
> Currenlty, 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.
> 
> Thanks!

Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:

  minor

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/24095/files
  - new: https://git.openjdk.org/jdk/pull/24095/files/6cfbe221..5a92aa07

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24095&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24095&range=00-01

  Stats: 6 lines in 1 file changed: 0 ins; 0 del; 6 mod
  Patch: https://git.openjdk.org/jdk/pull/24095.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24095/head:pull/24095

PR: https://git.openjdk.org/jdk/pull/24095


More information about the hotspot-compiler-dev mailing list