RFR: Add generations to freeset [v5]

Kelvin Nilsen kdnilsen at openjdk.org
Tue Apr 18 23:21:35 UTC 2023


On Thu, 13 Apr 2023 16:26:25 GMT, Y. Srinivas Ramakrishna <ysr 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 991:
> 
>> 989:       if ((_old_capacity < to_reserve_old) && (r->is_trash() || (r->affiliation() == ShenandoahAffiliation::FREE))) {
>> 990:         clear_mutator_free(idx);
>> 991:         set_old_collector_free(idx);
> 
> this would read:
> 
> 
>   remove_from_mutator(idx);
>   add_to_old(idx);
> 
> 
> or
> 
> 
>     move_from_mutator_to_old(idx)
> 
> 
> in the suggested set membership nomenclature.
> 
> Also, since we are always dealing with things in `ShenandoahFreeSet`, I'd dispense with the `free` adjective altogether and make the names more compact.

I'll make this change now.  I'd prefer to make more radical refactoring changes later.

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

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


More information about the shenandoah-dev mailing list