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