-
Notifications
You must be signed in to change notification settings - Fork 8
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
Reproducible benchmark code? #3
Comments
Needs access to the Debian porter boxes, and probably updates for changes there, to run the entire thing; but can also run locally ( |
Not sure how you'd test that differently, though, you do need to run the function a lot to get usable results. |
Thanks for the pointers. I just wanted to see how unpredictable inputs would change the results. To make it fair and avoid adding too much overhead to the loop, I was thinking the best way might be to pre-compute an array of randomly selected (known) words and use |
Won't that be affected too much by data caches? |
It'll increase cache pressure I guess, but not by much for a small array. It'd certainly be much closer to a real world scenario than repeating the same input a million times. |
Is the code used to produce the numbers in the readme available somewhere? Based on the methodology described, it seems likely that triehash is benefiting significantly from (unrealistically favourable) branch prediction:
The text was updated successfully, but these errors were encountered: