Skip to content

Commit

Permalink
8274546: Shenandoah: Remove unused ShenandoahUpdateRootsTask copy
Browse files Browse the repository at this point in the history
Reviewed-by: zgu, tschatzl
  • Loading branch information
linade authored and Thomas Schatzl committed Oct 7, 2021
1 parent d5ccfa2 commit 8319836
Showing 1 changed file with 0 additions and 27 deletions.
27 changes: 0 additions & 27 deletions src/hotspot/share/gc/shenandoah/shenandoahConcurrentMark.cpp
Expand Up @@ -42,33 +42,6 @@
#include "memory/iterator.inline.hpp"
#include "memory/resourceArea.hpp"

class ShenandoahUpdateRootsTask : public AbstractGangTask {
private:
ShenandoahRootUpdater* _root_updater;
bool _check_alive;
public:
ShenandoahUpdateRootsTask(ShenandoahRootUpdater* root_updater, bool check_alive) :
AbstractGangTask("Shenandoah Update Roots"),
_root_updater(root_updater),
_check_alive(check_alive){
}

void work(uint worker_id) {
assert(ShenandoahSafepoint::is_at_shenandoah_safepoint(), "Must be at a safepoint");
ShenandoahParallelWorkerSession worker_session(worker_id);

ShenandoahHeap* heap = ShenandoahHeap::heap();
ShenandoahUpdateRefsClosure cl;
if (_check_alive) {
ShenandoahForwardedIsAliveClosure is_alive;
_root_updater->roots_do<ShenandoahForwardedIsAliveClosure, ShenandoahUpdateRefsClosure>(worker_id, &is_alive, &cl);
} else {
AlwaysTrueClosure always_true;;
_root_updater->roots_do<AlwaysTrueClosure, ShenandoahUpdateRefsClosure>(worker_id, &always_true, &cl);
}
}
};

class ShenandoahConcurrentMarkingTask : public AbstractGangTask {
private:
ShenandoahConcurrentMark* const _cm;
Expand Down

1 comment on commit 8319836

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.