RFR: 8326781: G1ConcurrentMark::top_at_rebuild_start() should take a HeapRegion* not an uint
Albert Mingkun Yang
ayang at openjdk.org
Tue Feb 27 14:16:53 UTC 2024
On Tue, 27 Feb 2024 10:31:20 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
> Hi all,
>
> please review this refactoring to have `G1ConcurrentMark::top_at_rebuild_start` take a `HeapRegion*` instead of `uint` since all callers calculate the `uint` by calling `HeapRegion::hrm_index()` themselves in the argument.
>
> Testing: gha, compilation
>
> Thanks,
> Thomas
Marked as reviewed by ayang (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/18024#pullrequestreview-1903630866
More information about the hotspot-gc-dev
mailing list