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

Fei Yang fyang at openjdk.org
Mon Mar 10 03:40:02 UTC 2025


On Fri, 7 Mar 2025 11:42:53 GMT, Hamlin Li <mli 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?

That makes sense to me. Thanks.

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

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


More information about the hotspot-dev mailing list