-
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
Incorporate Araq's suggestions #32
Comments
sso branch benchmark results:
|
Used mem: jsonpak: 256KiB |
Using separate strings, numbers hash tables, branch literals:
|
For reference, these are the current timings:
jsonpak alone used Mem: 302.281KiB |
And here's branch 'short' with the suggest changes:
|
Bugs fixed, holy shit. |
Summary of branches:
|
Here's the bench report of the main branch by just changing the bitable implementation to not store hashes.
|
Observation that was missed: not caching hashes might be the reason that the literals branch doesn't perform as well for extract, but the size would increase as well. Sso branch results are still unreliable and might be caused by errors. |
Also worth noting VM performance of main versus literals. literals may take less time to execute but results in more instructions. master:
literals:
|
From the forum reply
The text was updated successfully, but these errors were encountered: