RFR (XXS): 8208672: Enable -Wreorder in make files
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Aug 6 12:38:09 UTC 2018
Hi David,
On Fri, 2018-08-03 at 10:20 +1000, David Holmes wrote:
> Hi Thomas,
>
> On 2/08/2018 7:14 PM, Thomas Schatzl wrote:
> > Hi all,
> >
> > there have been several suggestions to try to fix the Hotspot
> > code to allow us to enable -Wreorder in the Hotspot sources.
> > This should make problems due to use-before-initialization much
> > more obvious.
> >
> > This change enables -Wreorder for gcc and clang.
>
> For Windows (VS2017+) see:
>
> https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings
> /c5038
>
> Otherwise change seems okay.
>
Thanks!
http://cr.openjdk.java.net/~tschatzl/8208672/webrev.0_to_1 (diff)
http://cr.openjdk.java.net/~tschatzl/8208672/webrev.1 (full)
I verified that all our platforms (including Windows) still build.
Thanks,
Thomas
More information about the hotspot-dev
mailing list