Skip to content

[Dynamic Instrumentation] DEBUG-5317 fix snapshot stackoverflow in collection#8310

Merged
dudikeleti merged 2 commits into
masterfrom
fix/di-snapshot-stackoverflow
Mar 17, 2026
Merged

[Dynamic Instrumentation] DEBUG-5317 fix snapshot stackoverflow in collection#8310
dudikeleti merged 2 commits into
masterfrom
fix/di-snapshot-stackoverflow

Conversation

@dudikeleti

@dudikeleti dudikeleti commented Mar 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary of changes

Prevent stack overflows in Dynamic Instrumentation snapshot serialization for cyclic collections.
Preserve existing collection depth semantics so collection elements do not consume an extra reference-depth level.
Add regression tests for collection depth behavior and cyclic collections.

Reason for change

Snapshot serialization could recurse indefinitely when serializing supported collections whose elements reference the collection (directly or indirectly), eventually crashing the process with a stack overflow.

Implementation details

Update DebuggerSnapshotSerializer.SerializeEnumerable to track collections currently being serialized using per-serialization reference tracking.
Stop serialization when the same collection instance is encountered again, emitting notCapturedReason: depth.
Keep MaxReferenceDepth enforcement on collection nodes.
Preserve existing snapshot behavior by serializing collection elements at the same logical depth as the collection itself.

Test coverage

Limits_SelfReferencingEnumerable_DoesNotStackOverflow
Limits_IndirectCycleEnumerable_DoesNotStackOverflow
Limits_Depth_AppliesToCollectionsAtMaxDepth
Limits_CollectionElements_DoNotIncreaseDepth

@dudikeleti
dudikeleti requested a review from a team as a code owner March 16, 2026 11:27
andrewlock
andrewlock approved these changes Mar 16, 2026
View reviewed changes

@andrewlock andrewlock left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@pr-commenter

pr-commenter Bot commented Mar 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2026-03-17 01:29:12

Comparing candidate commit cc2c348 in PR branch fix/di-snapshot-stackoverflow with baseline commit 7a84734 in branch master.

Found 12 performance improvements and 5 performance regressions! Performance is the same for 159 metrics, 16 unstable metrics.

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleSimpleBody net6.0

  • 🟩 execution_time [-21.287ms; -16.931ms] or [-9.699%; -7.714%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleSimpleBody netcoreapp3.1

  • 🟥 execution_time [+14.143ms; +19.899ms] or [+7.222%; +10.162%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.ObjectExtractorMoreComplexBody net6.0

  • 🟥 execution_time [+9.437ms; +13.032ms] or [+5.001%; +6.907%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.ObjectExtractorSimpleBody netcoreapp3.1

  • 🟩 execution_time [-22.317ms; -16.684ms] or [-10.267%; -7.676%]

scenario:Benchmarks.Trace.Asm.AppSecEncoderBenchmark.EncodeLegacyArgs netcoreapp3.1

  • 🟩 execution_time [-20.949ms; -20.024ms] or [-10.249%; -9.797%]

scenario:Benchmarks.Trace.CharSliceBenchmark.OptimizedCharSlice net6.0

  • 🟩 execution_time [-250.711µs; -242.796µs] or [-15.308%; -14.825%]
  • 🟩 throughput [+106.441op/s; +110.209op/s] or [+17.432%; +18.049%]

scenario:Benchmarks.Trace.CharSliceBenchmark.OptimizedCharSlice netcoreapp3.1

  • 🟩 execution_time [-182.375µs; -178.265µs] or [-6.252%; -6.111%]
  • 🟩 throughput [+22.322op/s; +22.846op/s] or [+6.512%; +6.665%]

scenario:Benchmarks.Trace.CharSliceBenchmark.OriginalCharSlice net6.0

  • 🟩 execution_time [-234.041µs; -226.546µs] or [-10.614%; -10.274%]
  • 🟩 throughput [+52.038op/s; +53.735op/s] or [+11.475%; +11.849%]

scenario:Benchmarks.Trace.Iast.StringAspectsBenchmark.StringConcatAspectBenchmark net6.0

  • 🟩 execution_time [-1008.956µs; -923.277µs] or [-67.816%; -62.057%]

scenario:Benchmarks.Trace.Iast.StringAspectsBenchmark.StringConcatBenchmark net6.0

  • 🟩 throughput [+1230.819op/s; +3082.616op/s] or [+5.721%; +14.328%]

scenario:Benchmarks.Trace.Log4netBenchmark.EnrichedLog netcoreapp3.1

  • 🟥 execution_time [+39.034ms; +42.289ms] or [+24.282%; +26.307%]

scenario:Benchmarks.Trace.SingleSpanAspNetCoreBenchmark.SingleSpanAspNetCore netcoreapp3.1

  • 🟥 throughput [-16006515.937op/s; -14797595.908op/s] or [-6.635%; -6.133%]

scenario:Benchmarks.Trace.SpanBenchmark.StartFinishSpan net6.0

  • 🟥 execution_time [+14.544ms; +20.270ms] or [+7.350%; +10.244%]

scenario:Benchmarks.Trace.SpanBenchmark.StartFinishTwoScopes net6.0

  • 🟩 execution_time [-22.602ms; -18.628ms] or [-10.385%; -8.559%]

@dd-trace-dotnet-ci-bot

dd-trace-dotnet-ci-bot Bot commented Mar 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Execution-Time Benchmarks Report ⏱️

Execution-time results for samples comparing This PR (8310) and master.

✅ No regressions detected - check the details below

Full Metrics Comparison

FakeDbCommand

Metric Master (Mean ± 95% CI) Current (Mean ± 95% CI) Change Status
.NET Framework 4.8 - Baseline
duration75.20 ± (75.23 - 75.58) ms74.94 ± (74.88 - 75.26) ms-0.3%✅
.NET Framework 4.8 - Bailout
duration79.80 ± (79.73 - 80.11) ms79.76 ± (79.64 - 80.03) ms-0.1%✅
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration1102.56 ± (1102.61 - 1108.70) ms1105.20 ± (1105.66 - 1113.01) ms+0.2%✅⬆️
.NET Core 3.1 - Baseline
process.internal_duration_ms23.41 ± (23.36 - 23.47) ms23.06 ± (23.02 - 23.11) ms-1.5%✅
process.time_to_main_ms88.61 ± (88.42 - 88.81) ms87.22 ± (87.01 - 87.43) ms-1.6%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.91 ± (10.90 - 10.91) MB10.91 ± (10.91 - 10.92) MB+0.0%✅⬆️
runtime.dotnet.threads.count12 ± (12 - 12)12 ± (12 - 12)+0.0%✅
.NET Core 3.1 - Bailout
process.internal_duration_ms23.18 ± (23.13 - 23.23) ms22.83 ± (22.78 - 22.88) ms-1.5%✅
process.time_to_main_ms89.71 ± (89.49 - 89.94) ms88.32 ± (88.11 - 88.54) ms-1.5%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.94 ± (10.93 - 10.94) MB10.94 ± (10.93 - 10.94) MB-0.0%✅
runtime.dotnet.threads.count13 ± (13 - 13)13 ± (13 - 13)+0.0%✅
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms227.75 ± (226.58 - 228.92) ms228.96 ± (227.70 - 230.21) ms+0.5%✅⬆️
process.time_to_main_ms504.40 ± (503.40 - 505.40) ms501.16 ± (500.38 - 501.95) ms-0.6%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed47.18 ± (47.16 - 47.20) MB47.26 ± (47.24 - 47.28) MB+0.2%✅⬆️
runtime.dotnet.threads.count28 ± (28 - 28)28 ± (28 - 28)+0.0%✅⬆️
.NET 6 - Baseline
process.internal_duration_ms21.92 ± (21.87 - 21.97) ms21.80 ± (21.75 - 21.85) ms-0.6%✅
process.time_to_main_ms76.31 ± (76.14 - 76.48) ms75.72 ± (75.55 - 75.90) ms-0.8%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.63 ± (10.62 - 10.63) MB10.63 ± (10.63 - 10.64) MB+0.1%✅⬆️
runtime.dotnet.threads.count10 ± (10 - 10)10 ± (10 - 10)+0.0%✅
.NET 6 - Bailout
process.internal_duration_ms21.85 ± (21.80 - 21.90) ms21.75 ± (21.69 - 21.80) ms-0.5%✅
process.time_to_main_ms77.49 ± (77.27 - 77.71) ms76.83 ± (76.61 - 77.04) ms-0.9%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed10.72 ± (10.71 - 10.72) MB10.74 ± (10.74 - 10.75) MB+0.2%✅⬆️
runtime.dotnet.threads.count11 ± (11 - 11)11 ± (11 - 11)+0.0%✅
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms211.24 ± (209.73 - 212.74) ms212.98 ± (211.24 - 214.71) ms+0.8%✅⬆️
process.time_to_main_ms501.60 ± (500.84 - 502.36) ms502.18 ± (501.36 - 503.00) ms+0.1%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed49.21 ± (49.18 - 49.24) MB49.24 ± (49.21 - 49.28) MB+0.1%✅⬆️
runtime.dotnet.threads.count29 ± (29 - 29)29 ± (29 - 29)-0.0%✅
.NET 8 - Baseline
process.internal_duration_ms20.07 ± (20.02 - 20.11) ms20.09 ± (20.05 - 20.14) ms+0.1%✅⬆️
process.time_to_main_ms76.05 ± (75.86 - 76.25) ms75.53 ± (75.37 - 75.69) ms-0.7%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed7.64 ± (7.64 - 7.65) MB7.68 ± (7.67 - 7.68) MB+0.4%✅⬆️
runtime.dotnet.threads.count10 ± (10 - 10)10 ± (10 - 10)+0.0%✅
.NET 8 - Bailout
process.internal_duration_ms20.22 ± (20.18 - 20.27) ms20.09 ± (20.04 - 20.14) ms-0.7%✅
process.time_to_main_ms77.46 ± (77.25 - 77.66) ms77.00 ± (76.82 - 77.19) ms-0.6%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed7.72 ± (7.71 - 7.73) MB7.76 ± (7.74 - 7.77) MB+0.5%✅⬆️
runtime.dotnet.threads.count11 ± (11 - 11)11 ± (11 - 11)+0.0%✅
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms169.29 ± (168.26 - 170.31) ms168.98 ± (167.98 - 169.97) ms-0.2%✅
process.time_to_main_ms485.18 ± (484.38 - 485.98) ms482.06 ± (481.23 - 482.90) ms-0.6%✅
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%✅
runtime.dotnet.mem.committed36.51 ± (36.49 - 36.52) MB36.58 ± (36.56 - 36.60) MB+0.2%✅⬆️
runtime.dotnet.threads.count28 ± (28 - 28)28 ± (28 - 28)+0.0%✅

HttpMessageHandler

Metric Master (Mean ± 95% CI) Current (Mean ± 95% CI) Change Status
.NET Framework 4.8 - Baseline
duration194.01 ± (193.97 - 194.65) ms193.71 ± (193.75 - 194.60) ms-0.2%✅
.NET Framework 4.8 - Bailout
duration197.80 ± (197.68 - 198.42) ms196.41 ± (196.19 - 196.73) ms-0.7%✅
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration1148.41 ± (1149.16 - 1155.66) ms1144.90 ± (1146.18 - 1152.73) ms-0.3%✅
.NET Core 3.1 - Baseline
process.internal_duration_ms189.20 ± (188.87 - 189.52) ms187.62 ± (187.32 - 187.92) ms-0.8%✅
process.time_to_main_ms81.15 ± (80.89 - 81.41) ms80.81 ± (80.62 - 81.00) ms-0.4%✅
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%✅
runtime.dotnet.mem.committed16.12 ± (16.10 - 16.14) MB16.13 ± (16.10 - 16.16) MB+0.0%✅⬆️
runtime.dotnet.threads.count20 ± (19 - 20)20 ± (20 - 20)+0.3%✅⬆️
.NET Core 3.1 - Bailout
process.internal_duration_ms187.80 ± (187.46 - 188.15) ms187.07 ± (186.76 - 187.37) ms-0.4%✅
process.time_to_main_ms82.21 ± (82.08 - 82.34) ms81.90 ± (81.78 - 82.03) ms-0.4%✅
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%✅
runtime.dotnet.mem.committed16.14 ± (16.11 - 16.18) MB16.27 ± (16.24 - 16.29) MB+0.8%✅⬆️
runtime.dotnet.threads.count21 ± (21 - 21)21 ± (21 - 21)-0.2%✅
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms396.67 ± (394.88 - 398.46) ms396.02 ± (394.33 - 397.70) ms-0.2%✅
process.time_to_main_ms476.27 ± (475.67 - 476.88) ms472.45 ± (471.85 - 473.05) ms-0.8%✅
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%✅
runtime.dotnet.mem.committed57.91 ± (57.77 - 58.05) MB57.95 ± (57.79 - 58.11) MB+0.1%✅⬆️
runtime.dotnet.threads.count30 ± (30 - 30)30 ± (30 - 30)-0.0%✅
.NET 6 - Baseline
process.internal_duration_ms192.79 ± (192.40 - 193.18) ms192.09 ± (191.76 - 192.43) ms-0.4%✅
process.time_to_main_ms70.51 ± (70.34 - 70.68) ms70.22 ± (70.09 - 70.35) ms-0.4%✅
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed16.22 ± (16.10 - 16.34) MB16.18 ± (16.03 - 16.32) MB-0.3%✅
runtime.dotnet.threads.count19 ± (19 - 19)18 ± (18 - 19)-2.7%✅
.NET 6 - Bailout
process.internal_duration_ms191.54 ± (191.21 - 191.87) ms191.02 ± (190.74 - 191.31) ms-0.3%✅
process.time_to_main_ms71.34 ± (71.22 - 71.45) ms71.13 ± (71.03 - 71.23) ms-0.3%✅
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed16.13 ± (15.98 - 16.27) MB16.14 ± (15.99 - 16.29) MB+0.1%✅⬆️
runtime.dotnet.threads.count20 ± (19 - 20)19 ± (19 - 19)-2.2%✅
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms431.80 ± (430.17 - 433.42) ms428.66 ± (426.93 - 430.38) ms-0.7%✅
process.time_to_main_ms479.94 ± (479.08 - 480.81) ms476.22 ± (475.41 - 477.02) ms-0.8%✅
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed60.32 ± (60.25 - 60.39) MB60.20 ± (60.12 - 60.28) MB-0.2%✅
runtime.dotnet.threads.count30 ± (30 - 30)30 ± (30 - 31)+0.2%✅⬆️
.NET 8 - Baseline
process.internal_duration_ms189.85 ± (189.51 - 190.19) ms190.14 ± (189.81 - 190.46) ms+0.2%✅⬆️
process.time_to_main_ms69.77 ± (69.59 - 69.95) ms69.63 ± (69.46 - 69.81) ms-0.2%✅
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed11.82 ± (11.79 - 11.85) MB11.81 ± (11.78 - 11.83) MB-0.1%✅
runtime.dotnet.threads.count18 ± (18 - 18)18 ± (18 - 18)-0.5%✅
.NET 8 - Bailout
process.internal_duration_ms189.32 ± (188.95 - 189.69) ms190.20 ± (189.88 - 190.53) ms+0.5%✅⬆️
process.time_to_main_ms70.61 ± (70.50 - 70.73) ms70.78 ± (70.66 - 70.90) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed11.82 ± (11.78 - 11.85) MB11.87 ± (11.83 - 11.92) MB+0.5%✅⬆️
runtime.dotnet.threads.count19 ± (19 - 19)19 ± (19 - 19)+0.1%✅⬆️
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms354.37 ± (352.79 - 355.94) ms353.65 ± (352.19 - 355.11) ms-0.2%✅
process.time_to_main_ms451.88 ± (451.24 - 452.52) ms453.13 ± (452.31 - 453.96) ms+0.3%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%✅
runtime.dotnet.mem.committed48.40 ± (48.35 - 48.44) MB48.44 ± (48.39 - 48.48) MB+0.1%✅⬆️
runtime.dotnet.threads.count30 ± (30 - 30)30 ± (30 - 30)+0.1%✅⬆️
Comparison explanation

Execution-time benchmarks measure the whole time it takes to execute a program, and are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are highlighted in **red**. The following thresholds were used for comparing the execution times:

  • Welch test with statistical test for significance of 5%
  • Only results indicating a difference greater than 5% and 5 ms are considered.

Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard.

Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph).

Duration charts
FakeDbCommand (.NET Framework 4.8) 
gantt
    title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (75ms)  : 72, 78
    master - mean (75ms)  : 73, 78

    section Bailout
    This PR (8310) - mean (80ms)  : 77, 82
    master - mean (80ms)  : 77, 82

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (1,109ms)  : 1055, 1164
    master - mean (1,106ms)  : 1061, 1150
Loading
FakeDbCommand (.NET Core 3.1) 
gantt
    title Execution time (ms) FakeDbCommand (.NET Core 3.1)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (117ms)  : 113, 122
    master - mean (119ms)  : 115, 123

    section Bailout
    This PR (8310) - mean (118ms)  : 116, 121
    master - mean (120ms)  : 117, 123

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (767ms)  : 742, 792
    master - mean (769ms)  : 738, 800
Loading
FakeDbCommand (.NET 6) 
gantt
    title Execution time (ms) FakeDbCommand (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (104ms)  : 101, 107
    master - mean (105ms)  : 102, 108

    section Bailout
    This PR (8310) - mean (105ms)  : 103, 108
    master - mean (106ms)  : 103, 108

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (755ms)  : 719, 790
    master - mean (749ms)  : 716, 782
Loading
FakeDbCommand (.NET 8) 
gantt
    title Execution time (ms) FakeDbCommand (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (104ms)  : 101, 107
    master - mean (104ms)  : 101, 108

    section Bailout
    This PR (8310) - mean (105ms)  : 103, 107
    master - mean (106ms)  : 102, 109

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (691ms)  : 670, 713
    master - mean (695ms)  : 670, 719
Loading
HttpMessageHandler (.NET Framework 4.8) 
gantt
    title Execution time (ms) HttpMessageHandler (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (194ms)  : 189, 199
    master - mean (194ms)  : 191, 198

    section Bailout
    This PR (8310) - mean (196ms)  : 194, 199
    master - mean (198ms)  : 194, 202

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (1,149ms)  : 1103, 1196
    master - mean (1,152ms)  : 1106, 1199
Loading
HttpMessageHandler (.NET Core 3.1) 
gantt
    title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (277ms)  : 272, 281
    master - mean (279ms)  : 275, 283

    section Bailout
    This PR (8310) - mean (277ms)  : 272, 282
    master - mean (278ms)  : 274, 282

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (902ms)  : 870, 935
    master - mean (907ms)  : 877, 937
Loading
HttpMessageHandler (.NET 6) 
gantt
    title Execution time (ms) HttpMessageHandler (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (270ms)  : 266, 275
    master - mean (272ms)  : 265, 278

    section Bailout
    This PR (8310) - mean (270ms)  : 266, 274
    master - mean (271ms)  : 267, 275

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (939ms)  : 912, 966
    master - mean (946ms)  : 911, 980
Loading
HttpMessageHandler (.NET 8) 
gantt
    title Execution time (ms) HttpMessageHandler (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8310) - mean (270ms)  : 263, 276
    master - mean (269ms)  : 264, 274

    section Bailout
    This PR (8310) - mean (271ms)  : 267, 274
    master - mean (269ms)  : 264, 274

    section CallTarget+Inlining+NGEN
    This PR (8310) - mean (838ms)  : 816, 860
    master - mean (835ms)  : 814, 857
Loading

@dudikeleti
dudikeleti force-pushed the fix/di-snapshot-stackoverflow branch from a9eb98c to cc2c348 Compare March 17, 2026 00:45
@dudikeleti
dudikeleti enabled auto-merge (squash) March 17, 2026 10:58
@dudikeleti
dudikeleti merged commit abcc5ad into master Mar 17, 2026
143 checks passed
@dudikeleti
dudikeleti deleted the fix/di-snapshot-stackoverflow branch March 17, 2026 12:24
@github-actions github-actions Bot added this to the vNext-v3 milestone Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucaspimentel lucaspimentel lucaspimentel approved these changes
@tylfin tylfin tylfin approved these changes
@andrewlock andrewlock andrewlock approved these changes

Assignees

No one assigned

Labels

area:debugger

Projects

None yet

Milestone

3.40.0

Development

Successfully merging this pull request may close these issues.

4 participants

@dudikeleti @lucaspimentel @tylfin @andrewlock