Integrated: 8256267: Relax compiler/floatingpoint/NaNTest.java for x86_32 and lower -XX:+UseSSE
Aleksey Shipilev
shade at openjdk.java.net
Wed Nov 25 20:03:57 UTC 2020
On Thu, 12 Nov 2020 13:06:30 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Reproduces like this:
>
> $ CONF=linux-x86-server-fastdebug make images run-test TEST=compiler/floatingpoint/NaNTest.java TEST_VM_OPTS="-XX:UseSSE=1"
>
> STDOUT:
> ### NanTest started
> Written and read back float values match
> 0x7F800001 0x7F800001
> STDERR:
> java.lang.RuntimeException: Original and read back double values mismatch
> 0xFFF0000000000001 0xFFF8000000000001
>
> at compiler.floatingpoint.NaNTest.testDouble(NaNTest.java:56)
> at compiler.floatingpoint.NaNTest.main(NaNTest.java:69)
>
> After reading through [JDK-8076373](https://bugs.openjdk.java.net/browse/JDK-8076373), I think the test cannot be expected to pass without SSE >= 2 for doubles, and SSE >= 1 for floats on x86_32. This change adds the platform and UseSSE sensing to test.
>
> Additional testing:
> - [x] Affected test on Linux x86_32 with `-XX:UseSSE={0,1,2}`
> - [x] Affected test on Linux x86_64 with `-XX:UseSSE={0,1,2}`
> - [x] Affected test on Linux AArch64
This pull request has now been integrated.
Changeset: a14f02d8
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/a14f02d8
Stats: 38 lines in 1 file changed: 33 ins; 0 del; 5 mod
8256267: Relax compiler/floatingpoint/NaNTest.java for x86_32 and lower -XX:+UseSSE
Reviewed-by: kvn, iignatyev
-------------
PR: https://git.openjdk.java.net/jdk/pull/1187
More information about the hotspot-compiler-dev
mailing list