RFR (S): 8199326: Remove G1 gc time stamp logic

[Stefan Johansson]

On 2018-03-13 14:42, Thomas Schatzl wrote:
> Hi,
>
> On Mon, 2018-03-12 at 16:48 +0100, Stefan Johansson wrote:
>> Hi Thomas,
>> [...]
>>> Sorry. The correct link is:
>>> http://cr.openjdk.java.net/~tschatzl/8199326/webrev
>> Thanks for doing this cleanup, I found one more reference to the GC
>> timestamp in g1CollectedHeap.hpp:
>>    296   // Clear RSets after a compaction. It also resets the GC
>> time stamps.
>>    297   void clear_rsets_post_compaction();
>>
>> You can actually remove those two lines completely if you like. I
>> forgot the declaration when removing this method in the fix for:
>> 8177544: Restructure G1 Full GC code
>>
>> Otherwise the change looks good and if you don't want to remove the
>> declaration in this patch we can create a separate clean up for this.
>>
>    thanks for your review. I posted a webrev for the removal of this
> code as imho it does not completely fit here.
I agree, then consider this fix reviewed.

Thanks,
Stefan
> Thanks,
>    Thomas
>