8256039: Shenandoah: runtime/stringtable/StringTableCleaningTest.java fails

zhengyu123 · zhengyu123 · commit 1332ba3c3c16 · 2020-11-10T00:05:20.000Z
Reviewed-by: shade, rkennke
diff --git a/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.inline.hpp b/src/hotspot/share/gc/shenandoah/shenandoahParallelCleaning.inline.hpp
@@ -53,7 +53,9 @@ ShenandoahParallelWeakRootsCleaningTask<IsAlive, KeepAlive>::~ShenandoahParallel
   if (StringDedup::is_enabled()) {
     StringDedup::gc_epilogue();
   }
-  _weak_processing_task.report_num_dead();
+  if (_include_concurrent_roots) {
+    _weak_processing_task.report_num_dead();
+  }
 }
 
 template<typename IsAlive, typename KeepAlive>

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,9 @@ ShenandoahParallelWeakRootsCleaningTask<IsAlive, KeepAlive>::~ShenandoahParallel`
`53`	`53`	`if (StringDedup::is_enabled()) {`
`54`	`54`	`StringDedup::gc_epilogue();`
`55`	`55`	`}`
`56`		`- _weak_processing_task.report_num_dead();`
	`56`	`+ if (_include_concurrent_roots) {`
	`57`	`+ _weak_processing_task.report_num_dead();`
	`58`	`+ }`
`57`	`59`	`}`
`58`	`60`
`59`	`61`	`template<typename IsAlive, typename KeepAlive>`