RFR: 8324649: Shenandoah: replace implementation of free set [v45]

Roman Kennke rkennke at openjdk.org
Tue Apr 23 17:33:45 UTC 2024


On Wed, 17 Apr 2024 16:08:42 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahFullGC.cpp line 915:
>> 
>>> 913: public:
>>> 914:   ShenandoahPostCompactClosure() : _heap(ShenandoahHeap::heap()), _live(0) {
>>> 915:     _heap->free_set()->clear();
>> 
>> Why is that ok?
>
> There is no need to clear() because prepare_to_rebuild() invokes clear_internal() before it establishes new partition assignments.

Ok.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17561#discussion_r1576629210


More information about the shenandoah-dev mailing list