]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.21] cmd/compile: fix findIndVar so it does not match disjointed...
authorJorropo <jorropo.pgm@gmail.com>
Sun, 5 Nov 2023 21:40:01 +0000 (22:40 +0100)
committerGopher Robot <gobot@golang.org>
Tue, 28 Nov 2023 20:12:59 +0000 (20:12 +0000)
commitf7a79cb5fc0a57cabcf2cf2651f4e7573b4be1de
treeebb15fbcb86c327962a2cb996a2d276ce809c7aa
parent3684d19c20d73baba3ea2ae7e86d002465a24508
[release-branch.go1.21] cmd/compile: fix findIndVar so it does not match disjointed loop headers

Fix #63984

parseIndVar, prove and maybe more are on the assumption that the loop header
is a single block. This can be wrong, ensure we don't match theses cases we
don't know how to handle.

In the future we could update them so that they know how to handle such cases
but theses cases seems rare so I don't think the value would be really high.
We could also run a loop canonicalization pass first which could handle this.

The repro case looks weird because I massaged it so it would crash with the
previous compiler.

Change-Id: I4aa8afae9e90a17fa1085832250fc1139c97faa6
Reviewed-on: https://go-review.googlesource.com/c/go/+/539977
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
(cherry picked from commit 8b4e1259d0e82c8fe38a1456f997a4e9d63573a2)
Reviewed-on: https://go-review.googlesource.com/c/go/+/540535
Reviewed-by: Jorropo <jorropo.pgm@gmail.com>
Reviewed-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
src/cmd/compile/internal/ssa/loopbce.go
test/fixedbugs/issue63955.go [new file with mode: 0644]