Skip to content
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] Regressions in System.Text.RegularExpressions.Tests.Perf_Regex_Common #56018

Closed
DrewScoggins opened this issue Jul 20, 2021 · 7 comments
Closed
Labels
arch-x64 area-System.Text.RegularExpressions os-windows tenet-performance Performance related issue tenet-performance-benchmarks Issue from performance benchmark untriaged New issue has not been triaged by the area owner

Comments

@DrewScoggins
Copy link
Member

Run Information

Architecture x64
OS Windows 10.0.18362
Baseline 96ce6b35359b3c159ef3e685dd67cf30bb46769b
Compare c5edf0e09541e2c9c4441c0abdf23f712507596c
Diff Diff

Regressions in System.Text.RegularExpressions.Tests.Perf_Regex_Common

Benchmark Baseline Test Test/Base Test Quality Edge Detector Baseline IR Compare IR IR Ratio Baseline ETL Compare ETL
OneNodeBacktracking - Duration of single invocation 1.78 μs 2.23 μs 1.25 0.02 False
CtorInvoke - Duration of single invocation 138.34 μs 154.34 μs 1.12 0.00 False
Date_IsMatch - Duration of single invocation 79.03 ns 87.87 ns 1.11 0.03 False
Ctor - Duration of single invocation 29.66 μs 34.19 μs 1.15 0.03 False

graph
graph
graph
graph
Historical Data in Reporting System

Repro

git clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f netcoreapp5.0 --filter 'System.Text.RegularExpressions.Tests.Perf_Regex_Common*'

Payloads

Baseline
Compare

Histogram

System.Text.RegularExpressions.Tests.Perf_Regex_Common.OneNodeBacktracking(Options: Compiled)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.CtorInvoke(Options: Compiled)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.Date_IsMatch(Options: Compiled)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.Ctor(Options: Compiled)


Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository

@DrewScoggins DrewScoggins added os-windows tenet-performance Performance related issue tenet-performance-benchmarks Issue from performance benchmark arch-x64 labels Jul 20, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added area-System.Text.RegularExpressions untriaged New issue has not been triaged by the area owner labels Jul 20, 2021
@ghost
Copy link

ghost commented Jul 20, 2021

Tagging subscribers to this area: @eerhardt, @dotnet/area-system-text-regularexpressions
See info in area-owners.md if you want to be subscribed.

Issue Details

Run Information

Architecture x64
OS Windows 10.0.18362
Baseline 96ce6b35359b3c159ef3e685dd67cf30bb46769b
Compare c5edf0e09541e2c9c4441c0abdf23f712507596c
Diff Diff

Regressions in System.Text.RegularExpressions.Tests.Perf_Regex_Common

Benchmark Baseline Test Test/Base Test Quality Edge Detector Baseline IR Compare IR IR Ratio Baseline ETL Compare ETL
OneNodeBacktracking - Duration of single invocation 1.78 μs 2.23 μs 1.25 0.02 False
CtorInvoke - Duration of single invocation 138.34 μs 154.34 μs 1.12 0.00 False
Date_IsMatch - Duration of single invocation 79.03 ns 87.87 ns 1.11 0.03 False
Ctor - Duration of single invocation 29.66 μs 34.19 μs 1.15 0.03 False

graph
graph
graph
graph
Historical Data in Reporting System

Repro

git clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f netcoreapp5.0 --filter 'System.Text.RegularExpressions.Tests.Perf_Regex_Common*'

Payloads

Baseline
Compare

Histogram

System.Text.RegularExpressions.Tests.Perf_Regex_Common.OneNodeBacktracking(Options: Compiled)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.CtorInvoke(Options: Compiled)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.Date_IsMatch(Options: Compiled)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.Ctor(Options: Compiled)


Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository

Author: DrewScoggins
Assignees: -
Labels:

arch-x64, area-System.Text.RegularExpressions, os-windows, tenet-performance, tenet-performance-benchmarks, untriaged

Milestone: -

@DrewScoggins
Copy link
Member Author

Seems related to #51508

@DrewScoggins
Copy link
Member Author

Improvements from this change:

Run Information

Architecture x64
OS Windows 10.0.18362
Baseline 96ce6b35359b3c159ef3e685dd67cf30bb46769b
Compare c5edf0e09541e2c9c4441c0abdf23f712507596c
Diff Diff

Improvemnts in System.Text.RegularExpressions.Tests.Perf_Regex_Common

Benchmark Baseline Test Test/Base Test Quality Edge Detector Baseline IR Compare IR IR Ratio Baseline ETL Compare ETL
Backtracking - Duration of single invocation 1.85 μs 268.47 ns 0.15 0.03 True
Backtracking - Duration of single invocation 471.42 ns 176.56 ns 0.37 0.01 True

graph
graph
Historical Data in Reporting System

Repro

git clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f netcoreapp5.0 --filter 'System.Text.RegularExpressions.Tests.Perf_Regex_Common*'

Payloads

Baseline
Compare

Histogram

System.Text.RegularExpressions.Tests.Perf_Regex_Common.Backtracking(Options: None)


System.Text.RegularExpressions.Tests.Perf_Regex_Common.Backtracking(Options: Compiled)


Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository

@stephentoub
Copy link
Member

cc: @pgovind

@pgovind
Copy link
Contributor

pgovind commented Jul 20, 2021

Will look into this after #55960 (comment)

@stephentoub
Copy link
Member

#51508 was reverted. Presumably/hopefully this will go away as well.

@pgovind
Copy link
Contributor

pgovind commented Jul 21, 2021

Yup, we reverted #51508. I'd be very very surprised if the regressions/improvements here still remain. I'm going to close this issue for now. We can re-open if the regressions still exist.

@pgovind pgovind closed this as completed Jul 21, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Aug 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-x64 area-System.Text.RegularExpressions os-windows tenet-performance Performance related issue tenet-performance-benchmarks Issue from performance benchmark untriaged New issue has not been triaged by the area owner
Projects
None yet
Development

No branches or pull requests

3 participants