Uh oh!
There was an error while loading. Please reload this page.
Revert "Eliminate backtracking in the interpreter for patterns with .… - #56034
Revert "Eliminate backtracking in the interpreter for patterns with .…#56034pgovind wants to merge 1 commit into
Conversation
ghost
commented
Jul 20, 2021
Tagging subscribers to this area: @eerhardt, @dotnet/area-system-text-regularexpressions |
stephentoub
commented
Jul 20, 2021
#56031 is 37 minutes ahead ;-) |
pgovind
commented
Jul 20, 2021
Ah, I wasn't tagged on there, so I never got the notification for it. Thanks. I'll close this |
stephentoub
commented
Jul 20, 2021
Oh, interesting. Do you not get notifications when area-System.Text.RegularExpressions is added to something or when @dotnet/area-system-text-regularexpressions is tagged? I'm wondering if we have a broader issue to be fixed. |
pgovind
commented
Jul 20, 2021
My bad, I was wrong. I do get the notifications for area-System.Text.RegularExpressions. I did get this notification too. It came in right after the P7 one you had open, and the titles were the same, so I assumed all of them were related to the P7 PR :) |
stephentoub
commented
Jul 20, 2021
Ok, cool :-) |
…* (#51508)"
This reverts commit 7eb749c.
cc @BrennanConroy@danmoseley@jeffhandley@tannergooding
I'll add the regression as a unit test in a separate PR and investigate a fix locally (will probably only target .NET7P1 at this point for the optimization changes)