improve JitStatsDiagnoser based on feedback from @kouvel (#2246)

adamsitnik · web-flow · commit e715d5bb6398 · 2023-01-17T22:27:49.000+01:00
diff --git a/src/BenchmarkDotNet.Diagnostics.Windows/JitStatsDiagnoser.cs b/src/BenchmarkDotNet.Diagnostics.Windows/JitStatsDiagnoser.cs
@@ -48,11 +48,11 @@ protected override void AttachToEvents(TraceEventSession session, BenchmarkCase
                 }
             };
 
-            session.Source.Clr.TieredCompilationBackgroundJitStart += tieredData =>
+            session.Source.Clr.TieredCompilationBackgroundJitStop += tieredData =>
             {
                 if (StatsPerProcess.TryGetValue(tieredData.ProcessID, out JitStats jitStats))
                 {
-                    Interlocked.Increment(ref jitStats.MethodsTiered);
+                    Interlocked.Add(ref jitStats.MethodsTiered, tieredData.JittedMethodCount);
                 }
             };
         }

Original file line number	Diff line number	Diff line change
`@@ -48,11 +48,11 @@ protected override void AttachToEvents(TraceEventSession session, BenchmarkCase`
`48`	`48`	`}`
`49`	`49`	`};`
`50`	`50`
`51`		`- session.Source.Clr.TieredCompilationBackgroundJitStart += tieredData =>`
	`51`	`+ session.Source.Clr.TieredCompilationBackgroundJitStop += tieredData =>`
`52`	`52`	`{`
`53`	`53`	`if (StatsPerProcess.TryGetValue(tieredData.ProcessID, out JitStats jitStats))`
`54`	`54`	`{`
`55`		`- Interlocked.Increment(ref jitStats.MethodsTiered);`
	`55`	`+ Interlocked.Add(ref jitStats.MethodsTiered, tieredData.JittedMethodCount);`
`56`	`56`	`}`
`57`	`57`	`};`
`58`	`58`	`}`