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

is_empty is very slow? #45

Open
epage opened this issue Aug 17, 2023 · 0 comments
Open

is_empty is very slow? #45

epage opened this issue Aug 17, 2023 · 0 comments

Comments

@epage
Copy link

epage commented Aug 17, 2023

I run very basic benchmarks for various string implementations. I arbitrarily chose is_empty for my "access" test to see how expensive it was to use the underlying &str and SmartString is ridicuously slow compared to the others.

Looks like SmartString hand implements is_empty rather than relying on the Deref. I'm assuming its some lack of inlining.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant