You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This looks like 69a7c15 broke the combine-8-byte-loads-into-a-quadword-load optimization. That's what the slowdown is coming from. I'll see if I can figure out why.
No need to have both ops when they do the same thing.
Just declare MOVBload to zero extend and we can get rid
of MOVBQZXload. Same for W and L.
Kind of a followon cleanup for https://go-review.googlesource.com/c/19506/
Should enable an easier fix for #14920
Change-Id: I7cfac909a8ba387f433a6ae75c050740ebb34d42
Reviewed-on: https://go-review.googlesource.com/21004
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Please answer these questions before submitting your issue. Thanks!
go version
)?go version devel +5cdb3d0 2016-03-22 20:35:53 +0000 linux/amd64
go env
)?Linux amd64
BenchmarkXXChecksum64-32 20000000 291 ns/op
BenchmarkXXChecksum64-8 5000000 870 ns/op
I ran a bisect and the offending commit is 69a7c15.
The text was updated successfully, but these errors were encountered: