This repository has been archived by the owner on Mar 18, 2022. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
updated to W64 to W64+K for additional optimization as pointed by @potuz, to save 64 adds and 64 mem loads
Also mystery of variable performance on various machines was resolved:
test/bench.js
, was generating the random buffer inside the bench loop, causing interference of that as it also involvedMath.Random
, multiplications and rounds off.pre W+K optimization digest64 performance with bench fixed : ~ 24% faster than normal digest
post W+K optimization: ~ 29% faster than normal digest
GAIN: ~ 5%
@dapplion @wemeetagain
ps: this branch doesn't include uint8 optimization gains to be added by @tuyennhv PR!