RFR: 8365880: Shenandoah: Unify memory usage accounting in ShenandoahFreeSet [v3]
Kelvin Nilsen
kdnilsen at openjdk.org
Fri Oct 3 22:07:55 UTC 2025
On Thu, 2 Oct 2025 23:31:35 GMT, William Kemper <wkemper at openjdk.org> wrote:
>> 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.
You're right. We can simplify this. Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26867#discussion_r2403216907
More information about the hotspot-gc-dev
mailing list