On Fri, 4 Aug 2023 09:22:00 GMT, Andreas Steiner <d...@openjdk.org> wrote:

> On AIX we need the -X64 option for NM in the build. The handling is 
> equivalent to the other used tools flags like AR.
> This change will replace the quick fix done in 
> https://bugs.openjdk.org/browse/JDK-8312466.

Looks good overall. I suggest to remove the BUILD_NM variable in this PR as 
well. We should then wait with a final review until some folks from the Oracle 
build team are back.

make/autoconf/flags-other.m4 line 96:

> 94:   if test "x$OPENJDK_TARGET_OS" = xaix; then
> 95:     NMFLAGS="-X64"
> 96:   else

The else case is not needed, as an empty NMFLAGS would be the default. See the 
handling of RCFLAGS above.

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

PR Review: https://git.openjdk.org/jdk/pull/15150#pullrequestreview-1562552521
PR Review Comment: https://git.openjdk.org/jdk/pull/15150#discussion_r1284223066

Reply via email to