RFR: 8295159: DSO created with -ffast-math breaks Java floating-point arithmetic [v16]

Andrew Haley aph at openjdk.org
Fri Oct 27 11:51:47 UTC 2023


On Thu, 26 Oct 2023 17:42:39 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Andrew Haley has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove accidental include
>
> make/test/JtregNativeHotspot.gmk line 854:
> 
>> 852:   BUILD_HOTSPOT_JTREG_EXECUTABLES_LIBS_exeFPRegs := -ldl
>> 853:   BUILD_HOTSPOT_JTREG_LIBRARIES_LIBS_libAsyncGetCallTraceTest := -ldl
>> 854:   BUILD_HOTSPOT_JTREG_LIBRARIES_LDFLAGS_libfast-math := -ffast-math
> 
> Is the flag redundant by now? The test explicitly works with corresponding platform-specific registers.

it's belt-and braces (or belt-and-suspenders in American). This way, we can still do something useful for platforms not fully supported by mainline. I don't want to add ifdefs for every platform to the test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/10661#discussion_r1374458024


More information about the build-dev mailing list