RFR: 8329672: Only call SetupNativeCompilation from SetupJdkNativeCompilation [v2]

Magnus Ihse Bursie ihse at openjdk.org
Thu Apr 4 21:22:00 UTC 2024


On Thu, 4 Apr 2024 20:50:25 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> This patch will fix the few remaining places where a "raw" call to SetupNativeCompilation was made, instead of going via SetupJdkNativeCompilation. This include repairing the poor old X11Wrapper, which has been broken for some time.
>> 
>> After this we can finally make SetupNativeCompilation an "inner macro" of SetupJdkNativeCompilation, and stop exposing two different ways of compiling native code.
>> 
>> I have also added standard header comments to some places where I have missed it, and corrected an incorrect indentation (all things I found while double-checking all SetupJdkNativeCompilation calls).
>
> Magnus Ihse Bursie has updated the pull request incrementally with three additional commits since the last revision:
> 
>  - Update documentation
>  - Do not add LIBCXX for tests
>  - Add ability to disable DEFAULT_VERSIONINFO_RESOURCE

Just to confirm; this PR now passes with byte-by-byte identical builds compared to master.

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

PR Comment: https://git.openjdk.org/jdk/pull/18631#issuecomment-2038245442


More information about the build-dev mailing list