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