-
-
Notifications
You must be signed in to change notification settings - Fork 339
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(parser): optimize lexing strings #2366
Conversation
Current dependencies on/for this PR: This stack of pull requests is managed by Graphite. |
CodSpeed Performance ReportMerging #2366 will improve performances by 4.83%Comparing Summary
Benchmarks breakdown
|
44b0d84
to
8544f1f
Compare
4aeb664
to
38b7d79
Compare
This is as good as I can do for now. Still a bit disappointing, but ready for merge now if you're willing. As discussed on #2353, the increasing use of macros may be a concern. But could we go with it for now? If we introduce SIMD, or implement #2368, the macro will change a lot, and it'd be easier to keep those changes in one place (the macro) until it's fairly locked. At that point, as you said, if the macros slow down compilation we could expand them all manually. |
Of course! Improve then refactor. |
Thanks Boshen! |
Optimize lexing strings a bit.
Optimize lexing strings a bit.