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