-
-
Notifications
You must be signed in to change notification settings - Fork 110
perf: use simdutf for idna #960
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
Conversation
Good work. Whether it is worth it depends on the input strings, evidently. :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add this flag info to readme as well?
Can you open the same pull-request to ada-idna first, and later update this pull-request with ada-idea version and then add cmakelist changes? |
This is ready to land except:
|
github workflow ubuntu.yml (Ubuntu 22.04) updated, and README updated with new option too. |
@anonrig shared this idea with me and I wanted to see the results
before
after