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 <da...@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 <da...@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
