RFR: 8333457: GenShen: Move remembered set into new generational code [v2]
William Kemper
wkemper at openjdk.org
Tue Jun 4 17:29:15 UTC 2024
On Mon, 3 Jun 2024 22:54:30 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:
>> William Kemper has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix typo in comment
>
> src/hotspot/share/gc/shenandoah/shenandoahOldGeneration.cpp line 786:
>
>> 784: }
>> 785:
>> 786: void ShenandoahOldGeneration::mark_card_as_dirty(void* location) {
>
> Do we need a paranoid assert here?
>
>
> assert(is_in(location), ...);
This ultimately bottoms out in `CardTable::index_for`, which does assert that the location is in the heap: https://github.com/openjdk/shenandoah/blob/master/src/hotspot/share/gc/shared/cardTable.hpp#L160
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/443#discussion_r1626374415
More information about the shenandoah-dev
mailing list