]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: more fix on boolean ops on ARM64
authorCherry Mui <cherryyz@google.com>
Mon, 9 May 2022 16:57:31 +0000 (12:57 -0400)
committerCherry Mui <cherryyz@google.com>
Mon, 9 May 2022 17:29:53 +0000 (17:29 +0000)
commit90a11e921ba3b928706854655bd010f309a96458
tree0125257316fe42a47c39b7014d230f9c83feec82
parentf566fe3910593de96b6ab17abc906934de78d27c
cmd/compile: more fix on boolean ops on ARM64

Following CL 405114, the extension rule is also wrong. It is safe
to drop the extension if the value is from a boolean-generating
instruction, but not a boolean-typed Value in general (e.g. a Phi
or a in-register parameter). Fix it.

Updates #52788.

Change-Id: Icf3028fe8e90806f9f57fbe2b38d47da27a97e2a
Reviewed-on: https://go-review.googlesource.com/c/go/+/405115
Reviewed-by: David Chase <drchase@google.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
src/cmd/compile/internal/ssa/gen/ARM64.rules
src/cmd/compile/internal/ssa/rewriteARM64.go
test/fixedbugs/issue52788a.go [new file with mode: 0644]
test/fixedbugs/issue52788a.out [new file with mode: 0644]