RFR: 8290525: Move HeapRegion::_compaction_top to G1FullCollector [v2]
    Sangheon Kim 
    sangheki at openjdk.org
       
    Thu Jul 28 21:11:34 UTC 2022
    
    
  
On Thu, 21 Jul 2022 08:02:05 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Hi all,
>> 
>>   can I have reviews for this change that moves `HeapRegion::_compaction_top` into the `G1FullCollector` class; my reasoning is that these values are transient temporary values only ever useful for the full collection algorithm, basically results of the application of the algorithm, that are otherwise unused. This makes `HeapRegion` a bit easier to understand/read.
>> 
>> Testing: tier1-3
>> 
>> Thanks,
>>   Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   kbarrett review
Looks good.
-------------
Marked as reviewed by sangheki (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9560
    
    
More information about the serviceability-dev
mailing list