Skip to content

Commit

Permalink
Get rid of AIX linker option: -bnorwexec
Browse files Browse the repository at this point in the history
High-level: this linker option is oxymoronic with JIT environment (where executable code are generated on the fly anyway).

At least for the time being before we figured out why default codeCache is allocated through malloc (instead of mmap or shmget, for which that linker option appeared to have no effect by my tests).

Even after we figured it out eventually, it is a legitimate question what value this option provides for JIT environment.

this also has effect on FFI thunks (which are not generated in JIT codecache).

Issue ibmruntimes/openj9-openjdk-jdk21#162
  • Loading branch information
zl-wang committed May 29, 2024
1 parent 6d32fe3 commit df01409
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions make/autoconf/flags-ldflags.m4
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
#

# ===========================================================================
# (c) Copyright IBM Corp. 2019, 2019 All Rights Reserved
# (c) Copyright IBM Corp. 2019, 2024 All Rights Reserved
# ===========================================================================

AC_DEFUN([FLAGS_SETUP_LDFLAGS],
Expand Down Expand Up @@ -86,7 +86,7 @@ AC_DEFUN([FLAGS_SETUP_LDFLAGS_HELPER],
LDFLAGS_CXX_PARTIAL_LINKING="$MACHINE_FLAG -r"
if test "x$OPENJDK_TARGET_OS" = xaix; then
BASIC_LDFLAGS="-Wl,-b64 -Wl,-brtl -Wl,-bnorwexec -Wl,-bnolibpath -Wl,-bnoexpall \
BASIC_LDFLAGS="-Wl,-b64 -Wl,-brtl -Wl,-bnolibpath -Wl,-bnoexpall \
-Wl,-bernotok -Wl,-bdatapsize:64k -Wl,-btextpsize:64k -Wl,-bstackpsize:64k"
BASIC_LDFLAGS_JVM_ONLY="$BASIC_LDFLAGS_JVM_ONLY -Wl,-lC_r -Wl,-bbigtoc"
fi
Expand Down

0 comments on commit df01409

Please sign in to comment.