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

[Mikael Gerdin]

Hi Thomas,

On 2017-07-17 16:21, Thomas Schatzl wrote:
> 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.

I sort of agree but I think it's a nice convention to always #include 
macros in files which look at the INCLUDE_* macros.

Thanks for the review!
/Mikael

> 
> Thanks,
>    Thomas
>