> 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.
Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Respond to review feedback. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/12821/files - new: https://git.openjdk.org/jdk/pull/12821/files/481af9ad..417cb739 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=12821&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12821&range=01-02 Stats: 5 lines in 1 file changed: 0 ins; 1 del; 4 mod Patch: https://git.openjdk.org/jdk/pull/12821.diff Fetch: git fetch https://git.openjdk.org/jdk pull/12821/head:pull/12821 PR: https://git.openjdk.org/jdk/pull/12821
