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 build-dev mailing list