diff options
| author | Bryan C. Mills <bcmills@google.com> | 2019-09-23 13:22:13 +0000 |
|---|---|---|
| committer | Bryan C. Mills <bcmills@google.com> | 2019-09-23 15:20:12 +0000 |
| commit | 34fe8295c5ac099109d7f8a1f967edec96f2a71d (patch) | |
| tree | fc4c1da2c1b8f7960c7e64d802ab0332533b7d5e /test/codegen | |
| parent | 37c033b0a5433ce4132e8694705ad66d4650f670 (diff) | |
| download | go-34fe8295c5ac099109d7f8a1f967edec96f2a71d.tar.xz | |
Revert "compile: prefer an AND instead of SHR+SHL instructions"
This reverts CL 194297.
Reason for revert: introduced register allocation failures on PPC64LE builders.
Updates #33826
Updates #32781
Updates #34468
Change-Id: I7d0b55df8cdf8e7d2277f1814299b083c2692e48
Reviewed-on: https://go-review.googlesource.com/c/go/+/196957
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Martin Möhrmann <moehrmann@google.com>
Diffstat (limited to 'test/codegen')
| -rw-r--r-- | test/codegen/math.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/codegen/math.go b/test/codegen/math.go index 8aa652595b..36252710d1 100644 --- a/test/codegen/math.go +++ b/test/codegen/math.go @@ -81,7 +81,7 @@ func abs32(x float32) float32 { // Check that it's using integer registers func copysign(a, b, c float64) { - // amd64:"BTRQ\t[$]63","ANDQ","ORQ" + // amd64:"BTRQ\t[$]63","SHRQ\t[$]63","SHLQ\t[$]63","ORQ" // s390x:"CPSDR",-"MOVD" (no integer load/store) // ppc64:"FCPSGN" // ppc64le:"FCPSGN" @@ -100,7 +100,7 @@ func copysign(a, b, c float64) { // s390x:"LNDFR\t",-"MOVD\t" (no integer load/store) sink64[2] = math.Float64frombits(math.Float64bits(a) | 1<<63) - // amd64:"ANDQ","ORQ" + // amd64:-"SHLQ\t[$]1",-"SHRQ\t[$]1","SHRQ\t[$]63","SHLQ\t[$]63","ORQ" // s390x:"CPSDR\t",-"MOVD\t" (no integer load/store) // ppc64:"FCPSGN" // ppc64le:"FCPSGN" |