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