-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
Arm64: Perf regressions from using SIMD registers in cp/init blk #68665
Comments
Sort seems to be #68085. |
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
@kunalspathak FYI |
Tagging subscribers to this area: @JulieLeeMSFT Issue DetailsRun Information
Regressions in System.Collections.Tests.Perf_PriorityQueue<Guid, Guid>
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Collections.Tests.Perf_PriorityQueue<Guid, Guid>*' PayloadsHistogramSystem.Collections.Tests.Perf_PriorityQueue<Guid, Guid>.Dequeue_And_Enqueue(Size: 1000)
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
Regressions in System.Collections.Sort<BigStruct>
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Collections.Sort<BigStruct>*' PayloadsHistogramSystem.Collections.Sort<BigStruct>.Array(Size: 512)
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.Numerics.Tests.Perf_Matrix4x4
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Numerics.Tests.Perf_Matrix4x4*' PayloadsHistogramSystem.Numerics.Tests.Perf_Matrix4x4.CreatePerspectiveBenchmark
Description of detection logic
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
|
For Sort and Matrix4x4, I can clearly see the asmdiffs because of use of SIMD registers so I believe that it is because of #68085. The same benchmarks are not affected on Ubuntu. There were also equal wins on other benchmarks in dotnet/perf-autofiling-issues#4992. Looking at the tradeoffs and given the code size improvement, I am inclined to not fix it. The only fix would be to revert this change. Sort: InsertionSort: Matrix4x4: |
Run Information
Regressions in System.Collections.Tests.Perf_PriorityQueue<Guid, Guid>
Test Report
Repro
Payloads
Baseline
Compare
Histogram
System.Collections.Tests.Perf_PriorityQueue<Guid, Guid>.Dequeue_And_Enqueue(Size: 1000)
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
Regressions in System.Collections.Sort<BigStruct>
Test Report
Repro
Payloads
Baseline
Compare
Histogram
System.Collections.Sort<BigStruct>.Array(Size: 512)
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
Run Information
Regressions in System.Numerics.Tests.Perf_Matrix4x4
Test Report
Repro
Payloads
Baseline
Compare
Histogram
System.Numerics.Tests.Perf_Matrix4x4.CreatePerspectiveBenchmark
Description of detection logic
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
The text was updated successfully, but these errors were encountered: