-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Perf] Linux/x64: 1 Regression on 8/20/2023 5:37:20 PM #91281
Comments
The diff points to #88394 cc @MihaZupan. But I'm not 100% convinced. |
That PR is adding new APIs, but they're not consumed anywhere yet, so I doubt it's the cause. |
The first spike points to d267844...374b111. That seems to be hardly related as well. |
Also seems to be only Linux/Ubuntu 22.04 x64. Others are unaffected. |
Tagging subscribers to this area: @dotnet/area-system-memory Issue DetailsRun Information
Regressions in System.Memory.Span<Int32>
ReproGeneral Docs link: https://github.com/dotnet/performance/blob/main/docs/benchmarking-workflow-dotnet-runtime.md git clone https://github.com/dotnet/performance.git
python3 .\performance\scripts\benchmarks_ci.py -f net8.0 --filter 'System.Memory.Span<Int32>*' PayloadsSystem.Memory.Span<Int32>.IndexOfAnyTwoValues(Size: 4)ETL FilesHistogramDescription of detection logic
JIT DisasmsDocsProfiling workflow for dotnet/runtime repository
|
Adding new methods can change the alignment of other methods. It would be great to compare the disassembly to see if anything has actually changed. |
This is a 1 nanosecond change for a scenario taking in 4 inputs in an API that has undergone multiple other improvements balancing APIs . I'm closing this as "by design" given the regression is so small, that there is no clear indicator of what caused the regression, and that such small timings can be influenced by almost any part of the system. |
Run Information
Regressions in System.Memory.Span<Int32>
Test Report
Repro
General Docs link: https://github.com/dotnet/performance/blob/main/docs/benchmarking-workflow-dotnet-runtime.md
Payloads
Baseline
Compare
System.Memory.Span<Int32>.IndexOfAnyTwoValues(Size: 4)
ETL Files
Histogram
Description of detection logic
JIT Disasms
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
The text was updated successfully, but these errors were encountered: