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.

Sign up for GitHub

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

Jump to bottom

Remove the branches from len_utf8 #125129

Closed
wants to merge 2 commits into from
Closed

Remove the branches from len_utf8 #125129

wants to merge 2 commits into from

Commits on May 14, 2024

  1. Remove the branches from len_utf8 

    This changes `len_utf8` to add all of the range comparisons together,
rather than branching on each one. We should definitely test performance
though, because it's possible that this will pessimize mostly-ascii
inputs that would have had a short branch-predicted path before.
    @cuviper
    cuviper committed May 14, 2024
    Configuration menu
    Copy the full SHA
    38f14be View commit details
    Browse the repository at this point in the history

Commits on May 15, 2024

  1. Try len_utf8 with one branch for ASCII's sake

    @cuviper
    cuviper committed May 15, 2024
    Configuration menu
    Copy the full SHA
    ba2f5a9 View commit details
    Browse the repository at this point in the history