RFR: 8348663: [AIX] clang pollutes the burned-in library search paths of the generated executables [v3]

[Magnus Ihse Bursie]

On Wed, 29 Jan 2025 12:21:29 GMT, Joachim Kern <jkern at openjdk.org> wrote:

>> The clang compiler which we use since jdk21 pollutes the burned-in library search paths of the generated executables.
>> e.g. a dump -X64 -H ... /jdk-23.0.1+11-jre/lib/libnio.so
>> 
>> amongst others produces the following lines
>> 
>>                         ***Import File Strings***
>> INDEX PATH BASE MEMBER
>> 0 /opt/IBM/openxlC/17.1.1/../../../../usr/lpp/xlC/lib:/opt/IBM/openxlC/17.1.1/lib:/opt/IBM/openxlC/17.1.1/bin/../../../../../usr/lpp/xlC/lib:/usr/opt/zlibNX/lib:/opt/freeware/lib/pthread:/opt/freeware/lib:/usr/lib
>> 
>> The long string is the burned in library search path which will be used after searching with LIBPATH envvar.
>> You see there are some compiler paths which customers will not have; also /opt/freeware is unwanted.
>> A proper executable should only have the burned in default search path
>> /usr/lib:lib
>> 
>> To ensure this we want to call the linker with the flag
>> -fuse-ld=our-wrapper-script-cleaning-LIBPATH- and-calling-original-ld-afterwards
>> This script we want to deliver in make/scripts/aix/ld.sh
>> just containing
>> #!/bin/bash
>> unset LIBPATH
>> exec /usr/bin/ld "$@"
>> return $?
>
> Joachim Kern has updated the pull request incrementally with one additional commit since the last revision:
> 
>   cosmetic changes

Just one comment fix and then this is okay. Also please verify that the latest version still works on AIX.

make/autoconf/basic.m4 line 628:

> 626:   $CHMOD +x $OUTPUTDIR/compare.sh
> 627: 
> 628:   # Copy the linker wrapper script for AIX' clang and make it executable

Suggestion:

  # Copy the linker wrapper script for clang on AIX and make it executable

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

PR Comment: https://git.openjdk.org/jdk/pull/23313#issuecomment-2621499967
PR Review Comment: https://git.openjdk.org/jdk/pull/23313#discussion_r1933786266