RFR: 8357554: Enable vectorization of Bool -> CMove with different type size (on riscv)

Hamlin Li mli at openjdk.org
Tue Nov 11 11:32:42 UTC 2025


Hi,

Can you help to review this patch?

This patch enables the vectorization of statement like `op_1 bop op_2 ? res_f_d_1 : res_f_d_2` in a loop, where op_x's size is different from res_f_d_x's.

To assist with code review, this pr contains only the shared code change, is splitted from https://github.com/openjdk/jdk/pull/28230, which enable & implement the riscv part. The similar optimization could be extended to other platforms. 

## Some background

Previously, it's https://github.com/openjdk/jdk/pull/25336, which was blocked by unsigned comparison issue. The issue was recently resolved by https://github.com/openjdk/jdk/pull/27942, so I'm re-start working on this optimization.

This pr only relaxes one of the constraints in https://github.com/openjdk/jdk/pull/25336, i.e. transform CMoveF/D to vector operations no matter what's the size of comparison's operator, but remove the optimization of transform CMoveI/L to vector operations which I think need more investigation.

# Test
## Jtreg

in progress...

## Performance

check the performance data in https://github.com/openjdk/jdk/pull/25341 on riscv.

Thanks

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

Commit messages:
 - comments
 - simplify
 - fix code path change in VectorNode::implemented
 - fix JDK-8371297: assert in BoolTest
 - revert supports_transform_cmove_to_vectorblend for all cpus
 - disable Op_CMoveI/Op_CMoveL in VectorNode::opcode
 - disable riscv
 - Merge branch 'master' into vectorize-CMove-Bool
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - ... and 17 more: https://git.openjdk.org/jdk/compare/667744c3...56b6e029

Changes: https://git.openjdk.org/jdk/pull/28231/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=28231&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8357554
  Stats: 44 lines in 10 files changed: 40 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/28231.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28231/head:pull/28231

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


More information about the hotspot-compiler-dev mailing list