RFR: 8365206: RISC-V: compiler/c2/irTests/TestFloat16ScalarOperations.java is failing on riscv64
Hamlin Li
mli at openjdk.org
Tue Aug 19 08:29:13 UTC 2025
Hi,
Can you help to review this patch?
Currently implementation of conversion from float to float16 only preserve some significant bits of a NaN, which is not right in some cases.
As this (NaN conversion from float to float16) is already in the slow path, so I'll just fix it by preserving all significant bits in the same way as j.l.Float.floatToFloat16.
As current tests does not catch the issue, except of TestFloat16ScalarOperations.java, so add another test.
There is also the similar issue in vector version of conversion from float to float16, I'll address it in another pr [JDK-8365772](https://bugs.openjdk.org/browse/JDK-8365772)
Thanks!
-------------
Commit messages:
- initial commit
Changes: https://git.openjdk.org/jdk/pull/26838/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26838&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8365206
Stats: 230 lines in 6 files changed: 203 ins; 21 del; 6 mod
Patch: https://git.openjdk.org/jdk/pull/26838.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26838/head:pull/26838
PR: https://git.openjdk.org/jdk/pull/26838
More information about the hotspot-dev
mailing list