[jdk8u-dev] RFR: 8281096: Flags introduced by configure script are not passed to ADLC build [v2]
Andrew John Hughes
andrew at openjdk.org
Wed Jan 24 01:39:47 UTC 2024
> The 8u configure script defines compiler flags in `EXTRA_CFLAGS`, `EXTRA_LDFLAGS` and `EXTRA_ASFLAGS`. Some are added by configure tests, while others are taken directly from corresponding options passed by the user.
>
> 8u still use the legacy HotSpot build system which is not fully integrated with the autoconf system. Variables defined by configure thus have to be explicitly passed down to the separate HotSpot build.
>
> ADLC is a tool used at build-time and so the flags it uses don't impact on the end product. So, for a long time, it has been ignoring these flags defined by configure and using just its own minimal set.
>
> However, with newer compilers, this means that the code is compiled to a newer version of the C++ standard, as the default has changed in GCC 6 and later (see [JDK-8151841](https://bugs.openjdk.org/browse/JDK-8151841)). With the latest versions of GCC (11 and 12), this actually leads to build failures due to the use of 'register' (GCC 11) and the way comments are used (GCC 12) in the code.
>
> We should fix the ADLC build to use the same flags as the rest of the build. The impact should be negligible, given the same flags are already used in the code that is actually shipped.
>
> This does not affect 9+ where HotSpot's build system has been replaced with full integration in the autoconf system.
>
> With this change, 8u can be built with GCC 11 on GNU/Linux. I'd appreciate testing on other platforms, particularly those not covered by GHA (Solaris, AIX - @adamfarley, @sxa & @deepa181 who have provided previous fixes for these platforms)
Andrew John Hughes has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
- Temporarily enable debug logging for cross-compile build
- 8281096: Flags introduced by configure script are not passed to ADLC build
-------------
Changes:
- all: https://git.openjdk.org/jdk8u-dev/pull/357/files
- new: https://git.openjdk.org/jdk8u-dev/pull/357/files/c6dbc0b9..d1c15a55
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=357&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=357&range=00-01
Stats: 19444 lines in 831 files changed: 7585 ins; 10453 del; 1406 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/357.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/357/head:pull/357
PR: https://git.openjdk.org/jdk8u-dev/pull/357
More information about the jdk8u-dev
mailing list