RFR: 8328146: Set LIBCXX automatically [v2]

Magnus Ihse Bursie ihse at openjdk.org
Thu Mar 14 13:42:41 UTC 2024


On Thu, 14 Mar 2024 13:29:05 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> We are adding LIBCXX to LIBS in calls to SetupJdkLibrary whenever LINK_TYPE is C++. We should do this automatically in SetupJdkLibrary for C++ linking.
>> 
>> I also removed the superfluous `-lc` from some places where it had been added.
>
> Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Fix extra space
>  - Restore $(LIBCXX) in tests since they don't use SetupJdkLibrary.

I have verified with COMPARE_BUILD that there is no significant difference in the build result with or without this patch on Oracle default CI platforms (linux x64/aarch64, macos x64/aarch64, windows x64).

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

PR Comment: https://git.openjdk.org/jdk/pull/18298#issuecomment-1997486448


More information about the build-dev mailing list