RFR: 8345298: RISC-V: Add riscv backend for Float16 operations - scalar [v4]

Hamlin Li mli at openjdk.org
Fri Mar 7 11:45:57 UTC 2025


On Fri, 7 Mar 2025 01:59:06 GMT, Fei Yang <fyang at openjdk.org> wrote:

> Maybe we should also update the `@requires` of the test at the same time? Currently, it says `| (os.arch == "riscv64" & vm.cpu.features ~= ".*zvfh.*")`. Maybe we change `zvfh` into `zfh`?

No, as this test is for "vector conversion chain", only support of `zfh` should not trigger the test. BTW, scalar tests are in other test files.
In the future, when we support vectorization the IR verification test should be enabled again, but it still depends on `zvfh` rather than `zfh`.
Hope this answer your question?

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

PR Comment: https://git.openjdk.org/jdk/pull/23844#issuecomment-2706242048


More information about the hotspot-dev mailing list