-
Notifications
You must be signed in to change notification settings - Fork 1
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
Stdlib encoding performance #60
Comments
Adding the top-secret™ |
Just look at that
|
The base 10 case is ironically made 2x faster by omitting the fast paths in the binary integer init methods. Well, I won’t do that as it should only matter when they are unspecialized, which for my purposes means never. Using a more efficient form of type testing (#71) might have a similar effect, however. |
I’ve not kept track of what changes are responsible, but:
|
Adding custom smart-shifts reduces base |
I'm pretty sure (#88) sent the base 10 case to an outer ring of Tuple's Inferno™
|
Given that the package provides alternative coding methods via the
ANKBigEndianTextCodable
protocol, the following is more vexing than problematic. Having said that, if somebody wants to take a deep dive into Protocol Witness Hell ™️ and figure out how to make the stdlib encoding method perform - that'd be cool. As you can see in the list below, there's some funky business going on inString.init(some BinaryInteger, radix: Int, uppercase: Bool)
:The text was updated successfully, but these errors were encountered: