diff options
| author | Martin Möhrmann <moehrmann@google.com> | 2019-09-09 07:22:14 +0000 |
|---|---|---|
| committer | Martin Möhrmann <moehrmann@google.com> | 2019-09-09 07:33:25 +0000 |
| commit | 5bb59b6d1645144dade71e8d19ccf39338788a0f (patch) | |
| tree | 70123e60523d412876f2d4c37375283f94618397 /test/codegen | |
| parent | 9ec7074a946b7c2812a1a044e84276a36f46d14d (diff) | |
| download | go-5bb59b6d1645144dade71e8d19ccf39338788a0f.tar.xz | |
Revert "compile: prefer an AND instead of SHR+SHL instructions"
This reverts commit 9ec7074a946b7c2812a1a044e84276a36f46d14d.
Reason for revert: broke s390x (copysign, abs) and arm64 (bitfield) tests.
Change-Id: I16c1b389c062e8c4aa5de079f1d46c9b25b0db52
Reviewed-on: https://go-review.googlesource.com/c/go/+/193850
Run-TryBot: Martin Möhrmann <moehrmann@google.com>
Reviewed-by: Agniva De Sarker <agniva.quicksilver@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
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" |