diff options
| author | Brian Kessler <brian.m.kessler@gmail.com> | 2018-09-06 12:26:31 -0600 |
|---|---|---|
| committer | Robert Griesemer <gri@golang.org> | 2018-12-10 22:50:48 +0000 |
| commit | 1ccb66d1ef031ad70345d2ef1983a5814f405295 (patch) | |
| tree | f679ff2649264764b3984c9e20d2ad77349783db /src/hash | |
| parent | ba69df63ebbf9101a35fb11a0df1e1912d285da4 (diff) | |
| download | go-1ccb66d1ef031ad70345d2ef1983a5814f405295.tar.gz go-1ccb66d1ef031ad70345d2ef1983a5814f405295.zip | |
hash/fnv: use bits.Mul64 for 128-bit hash
Replace the 128-bit multiplication in 4 parts with bits.Mul64
and two single-width multiplications. This simplifies the code
and increases throughput by ~50% on amd64.
name old time/op new time/op delta
Fnv128KB-4 9.64µs ± 0% 6.09µs ± 0% -36.89% (p=0.016 n=4+5)
Fnv128aKB-4 9.11µs ± 0% 6.17µs ± 5% -32.32% (p=0.008 n=5+5)
name old speed new speed delta
Fnv128KB-4 106MB/s ± 0% 168MB/s ± 0% +58.44% (p=0.016 n=4+5)
Fnv128aKB-4 112MB/s ± 0% 166MB/s ± 5% +47.85% (p=0.008 n=5+5)
Change-Id: Id752f2a20ea3de23a41e08db89eecf2bb60b7e6d
Reviewed-on: https://go-review.googlesource.com/c/133936
Run-TryBot: Matt Layher <mdlayher@gmail.com>
Run-TryBot: Robert Griesemer <gri@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Matt Layher <mdlayher@gmail.com>
Reviewed-by: Robert Griesemer <gri@golang.org>
Diffstat (limited to 'src/hash')
| -rw-r--r-- | src/hash/fnv/fnv.go | 33 |
1 files changed, 11 insertions, 22 deletions
diff --git a/src/hash/fnv/fnv.go b/src/hash/fnv/fnv.go index 7662315d43..0fce177cb3 100644 --- a/src/hash/fnv/fnv.go +++ b/src/hash/fnv/fnv.go @@ -15,6 +15,7 @@ package fnv import ( "errors" "hash" + "math/bits" ) type ( @@ -137,18 +138,12 @@ func (s *sum64a) Write(data []byte) (int, error) { func (s *sum128) Write(data []byte) (int, error) { for _, c := range data { - // Compute the multiplication in 4 parts to simplify carrying - s1l := (s[1] & 0xffffffff) * prime128Lower - s1h := (s[1] >> 32) * prime128Lower - s0l := (s[0]&0xffffffff)*prime128Lower + (s[1]&0xffffffff)<<prime128Shift - s0h := (s[0]>>32)*prime128Lower + (s[1]>>32)<<prime128Shift - // Carries - s1h += s1l >> 32 - s0l += s1h >> 32 - s0h += s0l >> 32 + // Compute the multiplication + s0, s1 := bits.Mul64(prime128Lower, s[1]) + s0 += s[1]<<prime128Shift + prime128Lower*s[0] // Update the values - s[1] = (s1l & 0xffffffff) + (s1h << 32) - s[0] = (s0l & 0xffffffff) + (s0h << 32) + s[1] = s1 + s[0] = s0 s[1] ^= uint64(c) } return len(data), nil @@ -157,18 +152,12 @@ func (s *sum128) Write(data []byte) (int, error) { func (s *sum128a) Write(data []byte) (int, error) { for _, c := range data { s[1] ^= uint64(c) - // Compute the multiplication in 4 parts to simplify carrying - s1l := (s[1] & 0xffffffff) * prime128Lower - s1h := (s[1] >> 32) * prime128Lower - s0l := (s[0]&0xffffffff)*prime128Lower + (s[1]&0xffffffff)<<prime128Shift - s0h := (s[0]>>32)*prime128Lower + (s[1]>>32)<<prime128Shift - // Carries - s1h += s1l >> 32 - s0l += s1h >> 32 - s0h += s0l >> 32 + // Compute the multiplication + s0, s1 := bits.Mul64(prime128Lower, s[1]) + s0 += s[1]<<prime128Shift + prime128Lower*s[0] // Update the values - s[1] = (s1l & 0xffffffff) + (s1h << 32) - s[0] = (s0l & 0xffffffff) + (s0h << 32) + s[1] = s1 + s[0] = s0 } return len(data), nil } |