8290366: Remove unused during_conc_mark parameter in HeapRegion::note…

…_self_forwarding_removal_start Reviewed-by: ayang
openjdk · Jul 15, 2022 · b4e2ce0 · b4e2ce0
1 parent f3abb82
commit b4e2ce0
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/src/hotspot/share/gc/g1/g1EvacFailure.cpp b/src/hotspot/share/gc/g1/g1EvacFailure.cpp
@@ -162,10 +162,8 @@ class RemoveSelfForwardPtrHRClosure: public HeapRegionClosure {
     hr->clear_index_in_opt_cset();
 
     bool during_concurrent_start = _g1h->collector_state()->in_concurrent_start_gc();
-    bool during_concurrent_mark = _g1h->collector_state()->mark_or_rebuild_in_progress();
 
-    hr->note_self_forwarding_removal_start(during_concurrent_start,
-                                           during_concurrent_mark);
+    hr->note_self_forwarding_removal_start(during_concurrent_start);
 
     _phase_times->record_or_add_thread_work_item(G1GCPhaseTimes::RestoreRetainedRegions,
                                                    _worker_id,

diff --git a/src/hotspot/share/gc/g1/heapRegion.cpp b/src/hotspot/share/gc/g1/heapRegion.cpp
@@ -271,8 +271,7 @@ void HeapRegion::report_region_type_change(G1HeapRegionTraceType::Type to) {
                                             used());
 }
 
-void HeapRegion::note_self_forwarding_removal_start(bool during_concurrent_start,
-                                                    bool during_conc_mark) {
+void HeapRegion::note_self_forwarding_removal_start(bool during_concurrent_start) {
   // We always scrub the region to make sure the entire region is
   // parsable after the self-forwarding point removal, and update _marked_bytes
   // at the end.

diff --git a/src/hotspot/share/gc/g1/heapRegion.hpp b/src/hotspot/share/gc/g1/heapRegion.hpp
@@ -521,8 +521,7 @@ class HeapRegion : public CHeapObj<mtGC> {
 
   // Notify the region that we are about to start processing
   // self-forwarded objects during evac failure handling.
-  void note_self_forwarding_removal_start(bool during_concurrent_start,
-                                          bool during_conc_mark);
+  void note_self_forwarding_removal_start(bool during_concurrent_start);
 
   // Notify the region that we have finished processing self-forwarded
   // objects during evac failure handling.