RFR: 8365880: Shenandoah: Unify memory usage accounting in ShenandoahFreeSet [v3]

William Kemper wkemper at openjdk.org
Fri Oct 3 00:04:00 UTC 2025


On Fri, 12 Sep 2025 17:48:23 GMT, William Kemper <wkemper at openjdk.org> wrote:

>> Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   another tweak to make GHA gtest happy
>
> src/hotspot/share/gc/shenandoah/shenandoahGenerationalEvacuationTask.cpp line 253:
> 
>> 251:     if (available_in_region < plab_min_size_in_bytes) {
>> 252:       // The available memory in young had been retired.  Retire it in old also.
>> 253:       region_to_be_used_in_old += available_in_region;
> 
> This looks like a dead store.

This chunk of code from 253-258 looks like it could be removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26867#discussion_r2400392332


More information about the hotspot-gc-dev mailing list