8316428: G1: Nmethod count statistics only count last code root set iterated

Thomas Schatzl · Thomas Schatzl · commit fab372d3a23b · 2023-09-19T08:30:17.000Z
Reviewed-by: iwalulya, ayang
diff --git a/src/hotspot/share/gc/g1/g1RemSet.cpp b/src/hotspot/share/gc/g1/g1RemSet.cpp
@@ -849,7 +849,7 @@ class G1ScanCollectionSetRegionClosure : public HeapRegionClosure {
       // Scan the code root list attached to the current region
       r->code_roots_do(&cl);
 
-      _code_roots_scanned = cl.count();
+      _code_roots_scanned += cl.count();
 
       event.commit(GCId::current(), _worker_id, G1GCPhaseTimes::phase_name(_code_roots_phase));
     }

Original file line number	Diff line number	Diff line change
`@@ -849,7 +849,7 @@ class G1ScanCollectionSetRegionClosure : public HeapRegionClosure {`
`849`	`849`	`// Scan the code root list attached to the current region`
`850`	`850`	`r->code_roots_do(&cl);`
`851`	`851`
`852`		`- _code_roots_scanned = cl.count();`
	`852`	`+ _code_roots_scanned += cl.count();`
`853`	`853`
`854`	`854`	`event.commit(GCId::current(), _worker_id, G1GCPhaseTimes::phase_name(_code_roots_phase));`
`855`	`855`	`}`