RFR: 8371920: [TEST] Enable CMove tests on other platforms [v2]

Emanuel Peter epeter at openjdk.org
Tue Dec 9 14:16:08 UTC 2025


On Tue, 9 Dec 2025 09:25:35 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Hi,
>> Can you help to review this patch?
>> 
>> [JDK-8357551](https://bugs.openjdk.org/browse/JDK-8357551) add support of CMoveF/D vectorization, at the same time it also adds some tests for scalar CMove on riscv.
>> It's good to enable these tests on other platforms, like x86/aarch64 or maybe others.
>> 
>> At the same time, this pr also move these tests under `compiler/c2/cmove`, as suggested here https://github.com/openjdk/jdk/pull/28309#discussion_r2598664764.
>> 
>> Thanks!
>> 
>> ## Test
>> In progress... (I'm using github CI to run the tests.)
>
> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
> 
>   riscv + aarch64

@Hamlin-Li Thanks for publishing this so quickly, and for considering other platforms, much appreciated 😊 

The patch looks good to me.
I'll run some internal testing now.

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

PR Comment: https://git.openjdk.org/jdk/pull/28702#issuecomment-3632474530


More information about the hotspot-compiler-dev mailing list