RFR: 8318507: G1: Improve remset clearing for humongous candidates [v2]
Albert Mingkun Yang
ayang at openjdk.org
Mon Oct 23 07:43:58 UTC 2023
On Mon, 23 Oct 2023 07:41:05 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Hi all,
>>
>> please review this small cleanup that merges two calls to `HeapRegionRemSet::clear()/HeapRegionRemSet::set_state_complete()` to a single call to `clear()` with appropriate parameters.
>>
>> Also remove now obsolete asserts that are done in `clear()` already; actually the second one is wrong, there may be code roots attached to the region.
>>
>> Testing: gha, tier1-5 almost done
>>
>> Thanks,
>> Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>
> add assert
Marked as reviewed by ayang (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/16270#pullrequestreview-1691887521
More information about the hotspot-gc-dev
mailing list