Integrated: JDK-8302801: Remove fdlibm C sources

David Holmes david.holmes at oracle.com
Wed Mar 8 00:56:31 UTC 2023


Please note this fix has a problem with a missing definition for 
IEEEremainder that is causing UnsatisfiedLinkError. You can expect some 
significant noise in testing above tier 1 until this is fixed - which 
will hopefully be in the next 30 minutes or so. Otherwise a Backout will 
be performed.

David

On 8/03/2023 8:31 am, Joe Darcy wrote:
> On Thu, 2 Mar 2023 05:54:52 GMT, Joe Darcy <darcy at openjdk.org> wrote:
> 
>> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I thought I'd get out for the review the next phase of the FDLIBM port: removing the FDLIBM C sources from the repo.
>>
>> A repo with the changes for JDK-8302027 and this PR successful build on the default set of platform and successfully run tier 1 tests, which includes tests of the math library.
>>
>> There are a few remaining references to the case-independent string "fdlibm" in the make directory and HotSpot sources. HotSpot contains a partial fork for FDLIBM (a tine of FDLIBM?) to use for intrinsics. The remaining make machinery contains logic to determine what set of gcc options can be used for the compile.
>>
>> The intention of this change is to remove use of FDLIBM for the core libraries.
> 
> This pull request has now been integrated.
> 
> Changeset: b5b5cba7
> Author:    Joe Darcy <darcy at openjdk.org>
> URL:       https://git.openjdk.org/jdk/commit/b5b5cba7feb0e7ef957fd6bef1e591fdb6fdaa9f
> Stats:     6643 lines in 65 files changed: 20 ins; 6613 del; 10 mod
> 
> 8302801: Remove fdlibm C sources
> 
> Reviewed-by: bpb, dholmes, alanb, kvn
> 
> -------------
> 
> PR: https://git.openjdk.org/jdk/pull/12821



More information about the build-dev mailing list