Skip to content

Commit 8dfb6d7

Browse files
committed
8298651: Serial: Remove MarkSweep::follow_klass
Reviewed-by: kbarrett, tschatzl
1 parent de8153c commit 8dfb6d7

File tree

2 files changed

+1
-8
lines changed

2 files changed

+1
-8
lines changed

src/hotspot/share/gc/serial/markSweep.cpp

+1-6
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ void MarkSweep::push_objarray(oop obj, size_t index) {
7676
}
7777

7878
void MarkSweep::follow_array(objArrayOop array) {
79-
MarkSweep::follow_klass(array->klass());
79+
mark_and_push_closure.do_klass(array->klass());
8080
// Don't push empty arrays to avoid unnecessary work.
8181
if (array->length() > 0) {
8282
MarkSweep::push_objarray(array, 0);
@@ -203,11 +203,6 @@ template <class T> void MarkSweep::mark_and_push(T* p) {
203203
}
204204
}
205205

206-
void MarkSweep::follow_klass(Klass* klass) {
207-
oop op = klass->class_loader_data()->holder_no_keepalive();
208-
MarkSweep::mark_and_push(&op);
209-
}
210-
211206
template <typename T>
212207
void MarkAndPushClosure::do_oop_work(T* p) { MarkSweep::mark_and_push(p); }
213208
void MarkAndPushClosure::do_oop( oop* p) { do_oop_work(p); }

src/hotspot/share/gc/serial/markSweep.hpp

-2
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,6 @@ class MarkSweep : AllStatic {
148148

149149
static void follow_stack(); // Empty marking stack.
150150

151-
static void follow_klass(Klass* klass);
152-
153151
template <class T> static inline void adjust_pointer(T* p);
154152

155153
// Check mark and maybe push on marking stack

0 commit comments

Comments
 (0)