Skip to content

Commit 720c3d0

Browse files
committed
8296932: G1: Remove unused G1RootClosures::weak_oops
Reviewed-by: tschatzl
1 parent 196d021 commit 720c3d0

File tree

3 files changed

+0
-5
lines changed

3 files changed

+0
-5
lines changed

src/hotspot/share/gc/g1/g1RootClosures.cpp

-2
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ class G1EvacuationClosures : public G1EvacuationRootClosures {
3737
bool in_young_gc) :
3838
_closures(g1h, pss, in_young_gc) {}
3939

40-
OopClosure* weak_oops() { return &_closures._oops; }
4140
OopClosure* strong_oops() { return &_closures._oops; }
4241

4342
CLDClosure* weak_clds() { return &_closures._clds; }
@@ -61,7 +60,6 @@ class G1ConcurrentStartMarkClosures : public G1EvacuationRootClosures {
6160
_strong(g1h, pss, /* process_only_dirty_klasses */ false),
6261
_weak(g1h, pss, /* process_only_dirty_klasses */ false) {}
6362

64-
OopClosure* weak_oops() { return &_weak._oops; }
6563
OopClosure* strong_oops() { return &_strong._oops; }
6664

6765
CLDClosure* weak_clds() { return &_weak._clds; }

src/hotspot/share/gc/g1/g1RootClosures.hpp

-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ class G1ParScanThreadState;
3434
class G1RootClosures : public CHeapObj<mtGC> {
3535
public:
3636
// Closures to process raw oops in the root set.
37-
virtual OopClosure* weak_oops() = 0;
3837
virtual OopClosure* strong_oops() = 0;
3938

4039
// Closures to process CLDs in the root set.

src/hotspot/share/gc/g1/g1RootProcessor.cpp

-2
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ class StrongRootsClosures : public G1RootClosures {
8585
StrongRootsClosures(OopClosure* roots, CLDClosure* clds, CodeBlobClosure* blobs) :
8686
_roots(roots), _clds(clds), _blobs(blobs) {}
8787

88-
OopClosure* weak_oops() { return NULL; }
8988
OopClosure* strong_oops() { return _roots; }
9089

9190
CLDClosure* weak_clds() { return NULL; }
@@ -116,7 +115,6 @@ class AllRootsClosures : public G1RootClosures {
116115
AllRootsClosures(OopClosure* roots, CLDClosure* clds) :
117116
_roots(roots), _clds(clds) {}
118117

119-
OopClosure* weak_oops() { return _roots; }
120118
OopClosure* strong_oops() { return _roots; }
121119

122120
// By returning the same CLDClosure for both weak and strong CLDs we ensure

0 commit comments

Comments
 (0)