]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: intrinsify math/bits.Sub on amd64
authorKeith Randall <khr@google.com>
Tue, 23 Oct 2018 21:38:22 +0000 (14:38 -0700)
committerKeith Randall <khr@golang.org>
Thu, 25 Oct 2018 19:47:27 +0000 (19:47 +0000)
commitdd789550a74817e88466cdb583ae86c4c1426380
treef3e51cfdc5678f9bcf4f2eb78531e5e3595a0a58
parent899f3a2892b0a6e06a2e5f54c0a196d15944b99e
cmd/compile: intrinsify math/bits.Sub on amd64

name             old time/op  new time/op  delta
Sub-8            1.12ns ± 1%  1.17ns ± 1%   +5.20%          (p=0.008 n=5+5)
Sub32-8          1.11ns ± 0%  1.11ns ± 0%     ~     (all samples are equal)
Sub64-8          1.12ns ± 0%  1.18ns ± 1%   +5.00%          (p=0.016 n=4+5)
Sub64multiple-8  4.10ns ± 1%  0.86ns ± 1%  -78.93%          (p=0.008 n=5+5)

Fixes #28273

Change-Id: Ibcb6f2fd32d987c3bcbae4f4cd9d335a3de98548
Reviewed-on: https://go-review.googlesource.com/c/144258
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
src/cmd/compile/internal/amd64/ssa.go
src/cmd/compile/internal/gc/ssa.go
src/cmd/compile/internal/ssa/gen/AMD64.rules
src/cmd/compile/internal/ssa/gen/AMD64Ops.go
src/cmd/compile/internal/ssa/gen/genericOps.go
src/cmd/compile/internal/ssa/opGen.go
src/cmd/compile/internal/ssa/rewriteAMD64.go
test/codegen/mathbits.go