Skip to content

Commit 517d543

Browse files
committed
8370234: Remove CardTableBarrierSet::write_region
Reviewed-by: tschatzl, fandreuzzi
1 parent d4c0239 commit 517d543

File tree

5 files changed

+2
-13
lines changed

5 files changed

+2
-13
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ void G1BarrierSet::write_ref_array_pre(narrowOop* dst, size_t count, bool dest_u
111111
}
112112
}
113113

114-
void G1BarrierSet::write_region(JavaThread* thread, MemRegion mr) {
114+
void G1BarrierSet::write_region(MemRegion mr) {
115115
if (mr.is_empty()) {
116116
return;
117117
}

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,7 @@ class G1BarrierSet: public CardTableBarrierSet {
9999
template <DecoratorSet decorators, typename T>
100100
void write_ref_field_pre(T* field);
101101

102-
inline void write_region(MemRegion mr);
103-
void write_region(JavaThread* thread, MemRegion mr);
102+
virtual void write_region(MemRegion mr);
104103

105104
template <DecoratorSet decorators = DECORATORS_NONE, typename T>
106105
void write_ref_field_post(T* field);

src/hotspot/share/gc/g1/g1BarrierSet.inline.hpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,10 +68,6 @@ inline void G1BarrierSet::write_ref_field_pre(T* field) {
6868
enqueue(field);
6969
}
7070

71-
inline void G1BarrierSet::write_region(MemRegion mr) {
72-
write_region(JavaThread::current(), mr);
73-
}
74-
7571
template <DecoratorSet decorators, typename T>
7672
inline void G1BarrierSet::write_ref_field_post(T* field) {
7773
volatile CardValue* byte = _card_table->byte_for(field);

src/hotspot/share/gc/shared/cardTableBarrierSet.hpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,6 @@ class CardTableBarrierSet: public ModRefBarrierSet {
6161

6262
CardTable* card_table() const { return _card_table; }
6363

64-
void write_region(JavaThread* thread, MemRegion mr) {
65-
write_region(mr);
66-
}
67-
6864
// Record a reference update. Note that these versions are precise!
6965
// The scanning code has to handle the fact that the write barrier may be
7066
// either precise or imprecise. We make non-virtual inline variants of

src/hotspot/share/gc/shared/modRefBarrierSet.hpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,6 @@ class ModRefBarrierSet: public BarrierSet {
5353

5454
// Causes all refs in "mr" to be assumed to be modified (by this JavaThread).
5555
virtual void write_region(MemRegion mr) = 0;
56-
// Causes all refs in "mr" to be assumed to be modified by the given JavaThread.
57-
virtual void write_region(JavaThread* thread, MemRegion mr) = 0;
5856

5957
// Operations on arrays, or general regions (e.g., for "clone") may be
6058
// optimized by some barriers.

0 commit comments

Comments
 (0)