RFR: 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 13:54:13 GMT, Ivan Walulya <iwalulya 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 iwalulya (Reviewer).

Thanks @walulyai @albertnetymk for your reviews

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

PR Comment: https://git.openjdk.org/jdk/pull/18024#issuecomment-1968696935


More information about the hotspot-gc-dev mailing list