8332255: Shenandoah: Remove duplicate definition of init mark closure
Reviewed-by: shade, kdnilsen
This commit is contained in:
parent
42ccb74399
commit
9c02c8dd71
@ -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") {}
|
||||
|
Loading…
x
Reference in New Issue
Block a user