RFR: 8177544: Restructure G1 Full GC code

Erik Helin erik.helin at oracle.com
Tue Aug 15 08:50:20 UTC 2017


On 07/11/2017 04:37 PM, Stefan Johansson wrote:
> 
> On 2017-07-10 15:01, Thomas Schatzl wrote:
>> ...
>>> I see your point and I think it would be good. But as we discussed
>>> over chat, might be something to look at once everything else in this
>>> area is done. Will create a RFE for this.
>> Yes, that's fine.
>>
>>>>     - g1CollectedHeap.hpp: please try to sort the definitions of the
>>>> new methods in order of calling them.
>>> Done.
>>>
>>> Here are updated webrevs:
>>> Full: http://cr.openjdk.java.net/~sjohanss/8177544/hotspot.01/

Looks good, Reviewed.

Thanks,
Erik

>>> Inc: http://cr.openjdk.java.net/~sjohanss/8177544/hotspot.00-01/
>>>
>> Looks good to me. Sorry for the late reply.
> Thanks for reviewing Thomas!
> 
> No problem, I might not push this before getting back from vacation 
> anyways.
> 
> Thanks,
> Stefan
>>
>> Thanks,
>>    Thomas
>>
> 



More information about the hotspot-gc-dev mailing list