]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: eliminate unnecessary type conversions in TrailingZeros(16|8) for arm
authorTobias Klauser <tklauser@distanz.ch>
Fri, 15 Mar 2019 07:49:38 +0000 (08:49 +0100)
committerTobias Klauser <tobias.klauser@gmail.com>
Fri, 15 Mar 2019 18:37:22 +0000 (18:37 +0000)
commit156c830bea6795d57ef9eb9bfe66197413c00fce
treedb40d5b5fc0b950082f7f849a59daab5d48d662a
parentd9db9e32e924a60bbfbb15cc0dd7cfaaf8a62a3b
cmd/compile: eliminate unnecessary type conversions in TrailingZeros(16|8) for arm

This follows CL 156999 which did the same for arm64.

name               old time/op  new time/op  delta
TrailingZeros-4    7.30ns ± 1%  7.30ns ± 0%     ~     (p=0.413 n=9+9)
TrailingZeros8-4   8.32ns ± 0%  7.17ns ± 0%  -13.77%  (p=0.000 n=10+9)
TrailingZeros16-4  8.30ns ± 0%  7.18ns ± 0%  -13.50%  (p=0.000 n=9+10)
TrailingZeros32-4  6.46ns ± 1%  6.47ns ± 1%     ~     (p=0.325 n=10+10)
TrailingZeros64-4  16.3ns ± 0%  16.2ns ± 0%   -0.61%  (p=0.000 n=7+10)

Change-Id: I7e9e1abf7e30d811aa474d272b2824ec7cbbaa98
Reviewed-on: https://go-review.googlesource.com/c/go/+/167797
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
src/cmd/compile/internal/gc/ssa.go
src/cmd/compile/internal/ssa/gen/ARM.rules
src/cmd/compile/internal/ssa/rewriteARM.go
test/codegen/mathbits.go