From 08cdfd06ed1ca21f3b8c95361506ab94c429b622 Mon Sep 17 00:00:00 2001 From: Keith Randall Date: Tue, 12 Sep 2023 16:15:59 -0700 Subject: [PATCH] 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 LUCI-TryBot-Result: Go LUCI Reviewed-by: Heschi Kreinick --- src/cmd/compile/internal/ssa/memcombine.go | 1 + 1 file changed, 1 insertion(+) diff --git a/src/cmd/compile/internal/ssa/memcombine.go b/src/cmd/compile/internal/ssa/memcombine.go index 5c26fec7fe2c76..8e3db5a378a53d 100644 --- 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