Integrated: 8326781: G1ConcurrentMark::top_at_rebuild_start() should take a HeapRegion* not an uint

Thomas Schatzl tschatzl at openjdk.org
Wed Feb 28 10:40:57 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

This pull request has now been integrated.

Changeset: e7e80831
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/e7e8083139f2cf7aa3e49dd1d72335ccb4ff00af
Stats:     8 lines in 3 files changed: 0 ins; 1 del; 7 mod

8326781: G1ConcurrentMark::top_at_rebuild_start() should take a HeapRegion* not an uint

Reviewed-by: iwalulya, ayang

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

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


More information about the hotspot-gc-dev mailing list