RFR: Add generations to freeset [v5]

Kelvin Nilsen kdnilsen at openjdk.org
Tue Apr 18 21:12:52 UTC 2023


On Thu, 13 Apr 2023 11:05:48 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove debug instrumentation
>
> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 138:
> 
>> 136:         ShenandoahHeapRegion* r = _heap->get_region(c);
>> 137:         assert(r->is_trash() || (r->affiliation() == ShenandoahAffiliation::FREE)
>> 138:                || (r->affiliation() == ShenandoahAffiliation::OLD_GENERATION),
> 
> Now replaceable with `!r->is_affiliated()`, `r->old()`, etc.

replaced throughout.

> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 279:
> 
>> 277:         if (allow_new_region) {
>> 278:           // Then try a free region that is dedicated to GC allocations.
>> 279:           if (req.affiliation() == ShenandoahAffiliation::OLD_GENERATION) {
> 
> Here and later, now replaceable with `req.is_old()`.

done.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1170573577
PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1170574017


More information about the shenandoah-dev mailing list