Integrated: 8290525: Move HeapRegion::_compaction_top to G1FullCollector

Thomas Schatzl tschatzl at openjdk.org
Fri Jul 29 15:48:45 UTC 2022


On Tue, 19 Jul 2022 15:47:01 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

This pull request has now been integrated.

Changeset: 95fc16bd
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/95fc16bdfa7325ce9f6f8220964f78e6ab63078e
Stats:     79 lines in 14 files changed: 28 ins; 32 del; 19 mod

8290525: Move HeapRegion::_compaction_top to G1FullCollector

Reviewed-by: kbarrett, sangheki

-------------

PR: https://git.openjdk.org/jdk/pull/9560


More information about the serviceability-dev mailing list