]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile/internal/ssagen: conditon not need
authorcui <cuiweixie@gmail.com>
Mon, 5 Apr 2021 12:38:12 +0000 (12:38 +0000)
committerKeith Randall <khr@golang.org>
Mon, 5 Apr 2021 17:42:13 +0000 (17:42 +0000)
 n.Op() == ir.OFOR so n.Op() != ir.OFORUNTIL is always true

Change-Id: I97191783c1fb31ef76e0601f626b45af1e8d316e
GitHub-Last-Rev: f68f9fecfb7adf37af060afb77d6615839aabc6e
GitHub-Pull-Request: golang/go#45389
Reviewed-on: https://go-review.googlesource.com/c/go/+/307251
Reviewed-by: Keith Randall <khr@golang.org>
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Trust: Josh Bleecher Snyder <josharian@gmail.com>

src/cmd/compile/internal/ssagen/ssa.go

index 0a49ca214dca3066dc90a5ce3e18c385bac12ff2..8275d2ec9c3d2fcf9113c403ba0342d8614dfa08 100644 (file)
@@ -1803,7 +1803,7 @@ func (s *state) stmt(n ir.Node) {
                                b.AddEdgeTo(bCond)
                                // It can happen that bIncr ends in a block containing only VARKILL,
                                // and that muddles the debugging experience.
-                               if n.Op() != ir.OFORUNTIL && b.Pos == src.NoXPos {
+                               if b.Pos == src.NoXPos {
                                        b.Pos = bCond.Pos
                                }
                        }