[jdk11u-dev] Integrated: 8335801: [11u] Backport of 8210988 to 11u removes gcc warnings

[Andrew John Hughes]

On Sat, 6 Jul 2024 15:00:54 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

> The backport of 8210988 contained a number of issues, the most serious of which is it completely removed warnings from `gcc` builds by changing `WARNING_CFLAGS_JDK` and `WARNING_CFLAGS_JVM`, but not defining `WARNINGS_ENABLE_ALL`, `WARNINGS_ENABLE_ADDITIONAL_JVM` and `DISABLED_WARNINGS`.
> 
> The other more minor issues are the silent inclusion of `-Wreorder` to clang builds (should be a backport of [JDK-8208672](https://bugs.openjdk.org/browse/JDK-8208672)) and `-xc99=%none` (reverting [JDK-8215296](https://bugs.openjdk.org/browse/JDK-8215296)), because new lines were used as is without comparing differences in what was being replaced.
> 
> All three of these changes show up clearly when comparing the 11u & 12u changesets, and are corrected by this patch.
> 
> I can also confirm this has the desired effect with warnings as my local build on GCC 13 now fails again, whereas it was passing before (we need some backports there to make that work)

This pull request has now been integrated.

Changeset: c99638d3
Author:    Andrew John Hughes <andrew at openjdk.org>
URL:       https://git.openjdk.org/jdk11u-dev/commit/c99638d35c8aea16eae244d26bc4ab8dcb080788
Stats:     7 lines in 1 file changed: 6 ins; 0 del; 1 mod

8335801: [11u] Backport of 8210988 to 11u removes gcc warnings

Reviewed-by: phh

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

PR: https://git.openjdk.org/jdk11u-dev/pull/2838