RFR(XS) 8183935: G1GCPhaseTimes and G1RootProcessor do not include macros.hpp

Thomas Schatzl thomas.schatzl at oracle.com
Mon Jul 17 14:21:37 UTC 2017


Hi Mikael,

On Mon, 2017-07-17 at 16:12 +0200, Mikael Gerdin wrote:
> Hi,
> 
> Please review this trivial change to add includes of macros.hpp to 
> G1GCPhaseTimes and G1RootProcessor. They both the value of
> INCLUDE_AOT 
> and as such should explicitly include the proper header to ensure
> that 
> it is set to the correct value.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8183935
> Webrev: http://cr.openjdk.java.net/~mgerdin/8183935/webrev.0/
> Testing: JPRT build-only

  ship it :) Although I do not think it is necessary to include
macros.hpp both in the hpp and cpp file, but not sure. It won't hurt.

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list