8305086: G1 Redirty Cards phase printed twice

Thomas Schatzl · Thomas Schatzl · commit 014c658708f4 · 2023-03-29T15:21:02.000Z
Reviewed-by: ayang, kbarrett
diff --git a/src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp b/src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp
@@ -501,7 +501,6 @@ double G1GCPhaseTimes::print_post_evacuate_collection_set(bool evacuation_failed
     debug_phase(_gc_par_phases[RemoveSelfForwards], 2);
   }
 
-  trace_phase(_gc_par_phases[RedirtyCards]);
   debug_time("Post Evacuate Cleanup 2", _cur_post_evacuate_cleanup_2_time_ms);
   if (evacuation_failed) {
     debug_phase(_gc_par_phases[RecalculateUsed], 1);

Original file line number	Diff line number	Diff line change
`@@ -501,7 +501,6 @@ double G1GCPhaseTimes::print_post_evacuate_collection_set(bool evacuation_failed`
`501`	`501`	`debug_phase(_gc_par_phases[RemoveSelfForwards], 2);`
`502`	`502`	`}`
`503`	`503`
`504`		`- trace_phase(_gc_par_phases[RedirtyCards]);`
`505`	`504`	`debug_time("Post Evacuate Cleanup 2", _cur_post_evacuate_cleanup_2_time_ms);`
`506`	`505`	`if (evacuation_failed) {`
`507`	`506`	`debug_phase(_gc_par_phases[RecalculateUsed], 1);`