Skip to content

Commit 9c02c8d

Browse files
author
William Kemper
committed
8332255: Shenandoah: Remove duplicate definition of init mark closure
Reviewed-by: shade, kdnilsen
1 parent 42ccb74 commit 9c02c8d

File tree

1 file changed

+0
-23
lines changed

1 file changed

+0
-23
lines changed

src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1637,29 +1637,6 @@ void ShenandoahHeap::parallel_heap_region_iterate(ShenandoahHeapRegionClosure* b
16371637
}
16381638
}
16391639

1640-
class ShenandoahInitMarkUpdateRegionStateClosure : public ShenandoahHeapRegionClosure {
1641-
private:
1642-
ShenandoahMarkingContext* const _ctx;
1643-
public:
1644-
ShenandoahInitMarkUpdateRegionStateClosure() : _ctx(ShenandoahHeap::heap()->marking_context()) {}
1645-
1646-
void heap_region_do(ShenandoahHeapRegion* r) {
1647-
assert(!r->has_live(), "Region " SIZE_FORMAT " should have no live data", r->index());
1648-
if (r->is_active()) {
1649-
// Check if region needs updating its TAMS. We have updated it already during concurrent
1650-
// reset, so it is very likely we don't need to do another write here.
1651-
if (_ctx->top_at_mark_start(r) != r->top()) {
1652-
_ctx->capture_top_at_mark_start(r);
1653-
}
1654-
} else {
1655-
assert(_ctx->top_at_mark_start(r) == r->top(),
1656-
"Region " SIZE_FORMAT " should already have correct TAMS", r->index());
1657-
}
1658-
}
1659-
1660-
bool is_thread_safe() { return true; }
1661-
};
1662-
16631640
class ShenandoahRendezvousClosure : public HandshakeClosure {
16641641
public:
16651642
inline ShenandoahRendezvousClosure() : HandshakeClosure("ShenandoahRendezvous") {}

0 commit comments

Comments
 (0)