]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: update bottomUpVisitor.visit comments
authorCuong Manh Le <cuong.manhle.vn@gmail.com>
Fri, 9 Sep 2022 19:36:59 +0000 (02:36 +0700)
committerGopher Robot <gobot@golang.org>
Fri, 23 Sep 2022 15:09:44 +0000 (15:09 +0000)
Change-Id: I83a62b15c5946cfe61afc53c2c528aa3a62f815e
Reviewed-on: https://go-review.googlesource.com/c/go/+/429975
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com>

src/cmd/compile/internal/ir/scc.go

index a42951c1dda2de5676c496e1b9cfe45971e54ed5..b222939a1e923b51b096e397a9fab231dc84fbae 100644 (file)
@@ -103,16 +103,13 @@ func (v *bottomUpVisitor) visit(n *Func) uint32 {
        if (min == id || min == id+1) && !n.IsHiddenClosure() {
                // This node is the root of a strongly connected component.
 
-               // The original min passed to visitcodelist was v.nodeID[n]+1.
-               // If visitcodelist found its way back to v.nodeID[n], then this
-               // block is a set of mutually recursive functions.
-               // Otherwise it's just a lone function that does not recurse.
+               // The original min was id+1. If the bottomUpVisitor found its way
+               // back to id, then this block is a set of mutually recursive functions.
+               // Otherwise, it's just a lone function that does not recurse.
                recursive := min == id
 
-               // Remove connected component from stack.
-               // Mark walkgen so that future visits return a large number
-               // so as not to affect the caller's min.
-
+               // Remove connected component from stack and mark v.nodeID so that future
+               // visits return a large number, which will not affect the caller's min.
                var i int
                for i = len(v.stack) - 1; i >= 0; i-- {
                        x := v.stack[i]
@@ -122,7 +119,7 @@ func (v *bottomUpVisitor) visit(n *Func) uint32 {
                        }
                }
                block := v.stack[i:]
-               // Run escape analysis on this set of functions.
+               // Call analyze on this set of functions.
                v.stack = v.stack[:i]
                v.analyze(block, recursive)
        }