cmd/compile: reset memcombine correctly between basic blocks

Not sure why this bug didn't cause a complete failure, but it
certainly makes for doing a lot more work than is necessary.

Fixes #62604

Change-Id: If0be4acb6eafc3d7eeb42d2f4263c21b4e6e1c7d
Reviewed-on: https://go-review.googlesource.com/c/go/+/527699
Reviewed-by: David Chase <drchase@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>

diff --git a/src/cmd/compile/internal/ssa/memcombine.go b/src/cmd/compile/internal/ssa/memcombine.go
index 5c26fec7fe2c76416c5f268c02ad881253022f00..8e3db5a378a53dae762ca86cd4527dcfb00f300f 100644 (file)
--- a/src/cmd/compile/internal/ssa/memcombine.go
+++ b/src/cmd/compile/internal/ssa/memcombine.go
@@ -41,6 +41,7 @@ func memcombineLoads(f *Func) {
                }
        }
        for _, b := range f.Blocks {
+               order = order[:0]
                for _, v := range b.Values {
                        if v.Op != OpOr16 && v.Op != OpOr32 && v.Op != OpOr64 {
                                continue