[jdk8u-dev] RFR: 8281096: Flags introduced by configure script are not passed to ADLC build

[Thomas Stuefe]

On Wed, 23 Aug 2023 03:58:10 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

> 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)

Looks good and makes sense. Why not passing LFLAGS to solaris?

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

PR Review: https://git.openjdk.org/jdk8u-dev/pull/357#pullrequestreview-1594912757