From 9c02c8dd71023df6338cb94997bca6b00768af6f Mon Sep 17 00:00:00 2001 From: William Kemper Date: Wed, 15 May 2024 16:42:19 +0000 Subject: [PATCH] 8332255: Shenandoah: Remove duplicate definition of init mark closure Reviewed-by: shade, kdnilsen --- .../share/gc/shenandoah/shenandoahHeap.cpp | 23 ------------------- 1 file changed, 23 deletions(-) diff --git a/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp b/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp index c9d3d1d9364..870885b385c 100644 --- a/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp +++ b/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp @@ -1637,29 +1637,6 @@ void ShenandoahHeap::parallel_heap_region_iterate(ShenandoahHeapRegionClosure* b } } -class ShenandoahInitMarkUpdateRegionStateClosure : public ShenandoahHeapRegionClosure { -private: - ShenandoahMarkingContext* const _ctx; -public: - ShenandoahInitMarkUpdateRegionStateClosure() : _ctx(ShenandoahHeap::heap()->marking_context()) {} - - void heap_region_do(ShenandoahHeapRegion* r) { - assert(!r->has_live(), "Region " SIZE_FORMAT " should have no live data", r->index()); - if (r->is_active()) { - // Check if region needs updating its TAMS. We have updated it already during concurrent - // reset, so it is very likely we don't need to do another write here. - if (_ctx->top_at_mark_start(r) != r->top()) { - _ctx->capture_top_at_mark_start(r); - } - } else { - assert(_ctx->top_at_mark_start(r) == r->top(), - "Region " SIZE_FORMAT " should already have correct TAMS", r->index()); - } - } - - bool is_thread_safe() { return true; } -}; - class ShenandoahRendezvousClosure : public HandshakeClosure { public: inline ShenandoahRendezvousClosure() : HandshakeClosure("ShenandoahRendezvous") {}