RFR: 8355913: RISC-V: improve hotspot/jtreg/compiler/vectorization/runner/BasicFloatOpTest.java

Fei Yang fyang at openjdk.org
Wed Apr 30 05:48:50 UTC 2025


On Tue, 29 Apr 2025 13:42:22 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this simple patch?
> Previously, BasicFloatOpTest.java is accidently not really enabled on riscv.
> And FmaVF/FmaVD depends on both UseFMA and UseRVV, the code should make it clear in this sense.
> And IR verification of FmaVF in BasicFloatOpTest.java should only be enabled when UseFMA && rvv.
> 
> Thanks!

test/hotspot/jtreg/compiler/vectorization/runner/BasicFloatOpTest.java line 35:

> 33:  * @run main/othervm -Xbootclasspath/a:.
> 34:  *                   -XX:+UnlockDiagnosticVMOptions
> 35:  *                   -XX:+WhiteBoxAPI

This test also covers x86 avx and aarch64 asimd targets. I think UseFMA is also needed for IR checks for these platforms, right? Seems to me more reasonable to add a `-XX:+UseFMA` VM option when running this test. Then we won't need to add all these `applyIf = {"UseFMA", "true"}`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24950#discussion_r2067889769


More information about the hotspot-compiler-dev mailing list