]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.13] cmd/compile: rename poset method dominates to reaches
authorzdjones <zachj1@gmail.com>
Fri, 30 Aug 2019 13:41:09 +0000 (14:41 +0100)
committerAlexander Rakoczy <alex@golang.org>
Mon, 14 Oct 2019 19:21:29 +0000 (19:21 +0000)
commitc16e37eab9666dc1905f8d5668392510a5b23134
treec8ffb1f6fab6e119f489a71be8d84aa39708cc5b
parent2018d431c7e865d6db909cc0aa0d1f6d01e54583
[release-branch.go1.13] cmd/compile: rename poset method dominates to reaches

The partially ordered set uses a method named 'dominates' to determine whether
two nodes are partially ordered. Dominates does a depth-first search of the
DAG, beginning at the source node, and returns true as soon as it finds a path
to the target node. In the context of the forest-of-DAGs that makes up the
poset, dominates is not necessarily checking dominance, but is checking
reachability. See the issue tracker for a more detailed discussion of the
difference.

Fortunately, reachability is logically correct everywhere dominates is currently
used in poset.go. Reachability within a DAG is sufficient to establish the
partial ordering (source < target).

This CL changes the name of the method (dominates -> reaches) and updates
all the comments in the file accordingly.

Updates #34807

Change-Id: Ia3a34f7b14b363801d75b05099cfc686035f7d96
Reviewed-on: https://go-review.googlesource.com/c/go/+/192617
Reviewed-by: Giovanni Bajo <rasky@develer.com>
Run-TryBot: Giovanni Bajo <rasky@develer.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-on: https://go-review.googlesource.com/c/go/+/201059
Run-TryBot: Alexander Rakoczy <alex@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/ssa/poset.go