-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Regressions in System.IO.Tests.BinaryReaderTests #93743
Comments
Also dotnet/perf-autofiling-issues#23338 Regressed by #80331 cc @teo-tsirpanis |
Tagging subscribers to this area: @dotnet/area-system-io Issue DetailsRun Information
Regressions in System.IO.Tests.BinaryReaderTests
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.IO.Tests.BinaryReaderTests*' PayloadsSystem.IO.Tests.BinaryReaderTests.ReadHalfETL FilesHistogramDescription of detection logic
JIT DisasmsDocsProfiling workflow for dotnet/runtime repository
|
Weird, on Linux it improved. 🤔 |
It has improved for all OSes on x64, but regressed on arm64. |
I think it's OK to accept this tradeoff. |
Run Information
Regressions in System.IO.Tests.BinaryReaderTests
Test Report
Repro
General Docs link: https://github.com/dotnet/performance/blob/main/docs/benchmarking-workflow-dotnet-runtime.md
Payloads
Baseline
Compare
System.IO.Tests.BinaryReaderTests.ReadHalf
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: