RFR: 8320715: Improve the tests of test/hotspot/jtreg/compiler/intrinsics/float16 [v2]
Hamlin Li
mli at openjdk.org
Tue Nov 28 17:34:27 UTC 2023
> Hi,
> Can you review the patch to improve the tests of test/hotspot/jtreg/compiler/intrinsics/float16?
> Thanks.
>
> Currently, there are several areas where improvements can be made:
> 1. test should not exit early before all test cases have run.
> 2. some output hide the data details, which should be printed out to help debug any potential issues, e.g. currently it could output some log like below in case of wrong implementation of a ConvF2HF intrinsic:
> `Inconsistent result for Float.floatToFloat16(NaN/ff802000): fc01 != fc01`, which is rather confusing, not helpful. Suggested output would be: `Inconsistent result for Float.floatToFloat16(NaN/ff802000): fc01(64513) != fc01(-1023)`
Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
fix typo
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16815/files
- new: https://git.openjdk.org/jdk/pull/16815/files/d93a6e6c..95ee65c4
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16815&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16815&range=00-01
Stats: 4 lines in 1 file changed: 2 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/16815.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16815/head:pull/16815
PR: https://git.openjdk.org/jdk/pull/16815
More information about the hotspot-compiler-dev
mailing list