Hi! On 01/18/2018 06:27 PM, Erik Joelsson wrote:
When adding a disabled warning like this, we need to also add a clear comment describing why it's necessary. In this case it's caused by a bug in GCC and only affects certain versions. Otherwise, we will likely try to remove them later and without information on why it was added, we will just conclude that the warning is not triggering with the official GCC version and remove it.
I don't quite understand why we would want to patch OpenJDK-11 in the first place when the reason this bug is triggered is a) a known toolchain bug in an old and unsupported version of gcc and b) can be easily disabled by building with "--disable-warnings-as-errors". The only thing here to patch is obviously gcc and not any other project that is built with gcc. I am pretty confident that OpenJDK isn't the only project that is affected by this particular gcc bug. And if a Linux distribution is using gcc-4.8.5 as their default compiler, they will most likely not be using OpenJDK-11 but rather be stuck with OpenJDK-7 or 8. Thus, I don't see a point in working around bugs in an ancient toolchain in the current OpenJDK code. Thanks, Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer - glaub...@debian.org `. `' Freie Universitaet Berlin - glaub...@physik.fu-berlin.de `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913