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

Tue Nov 11 15:08:04 UTC 2025

On Tue, 11 Nov 2025 14:04:26 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Sounds like this PR should include some IR tests?
>
> @galderz @eme64 BTW, there is an assert fix in this pr, which is also in another specific pr: https://github.com/openjdk/jdk/pull/28141. Please let me know if I should do it in this pr or not. Thanks!

@Hamlin-Li At a quick glance, https://github.com/openjdk/jdk/pull/28230 also has some scalar backend implementations of CMove. I think you could just integrate those separately first, and only then do the vectorization.

Additionally: it may be easier to first ensure that the Vector API tests work for riscv backend vector instructions. And then we can work on Auto Vectorization once the all the backend instructions are already in place and tested via the Vector API.

That would be a way I usually see aarch64 and x64 engineers split up the work. Also makes it easier to get specialists for the area to review the code.

What do you think?

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

PR Comment: https://git.openjdk.org/jdk/pull/28231#issuecomment-3517342327