8241838: Shenandoah: no need to trash cset during final mark

Reviewed-by: rkennke
openjdk · Mar 30, 2020 · af15605babb765a9d1015ea3ca33773c81b3ce58 · af15605
1 parent 8cb7635
commit af15605babb765a9d1015ea3ca33773c81b3ce58
Unified Split

Showing with 0 additions and 7 deletions.

+0 −6 src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp

+0 −1 src/hotspot/share/gc/shenandoah/shenandoahPhaseTimings.hpp
diff --git a/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp b/src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp
@@ -1489,12 +1489,6 @@ void ShenandoahHeap::op_final_mark() {
       sync_pinned_region_status();
     }
 
-    // Trash the collection set left over from previous cycle, if any.
-    {
-      ShenandoahGCPhase phase(ShenandoahPhaseTimings::trash_cset);
-      trash_cset_regions();
-    }
-
     {
       ShenandoahGCPhase phase(ShenandoahPhaseTimings::prepare_evac);
 

diff --git a/src/hotspot/share/gc/shenandoah/shenandoahPhaseTimings.hpp b/src/hotspot/share/gc/shenandoah/shenandoahPhaseTimings.hpp
@@ -82,7 +82,6 @@ class outputStream;
   f(complete_liveness,                              "  Complete Liveness")             \
   f(retire_tlabs,                                   "  Retire TLABs")                  \
   f(sync_pinned,                                    "  Sync Pinned")                   \
-  f(trash_cset,                                     "  Trash CSet")                    \
   f(prepare_evac,                                   "  Prepare Evacuation")            \
   f(init_evac,                                      "  Initial Evacuation")            \
   SHENANDOAH_GC_PAR_PHASE_DO(evac_,                 "    E: ", f)                      \