Skip to content

Commit

Permalink
Caveat for spl case dominator in “Dominators” doc (#8903)
Browse files Browse the repository at this point in the history
Fixes #3346
  • Loading branch information
Himanshu authored Sep 15, 2021
1 parent cca867c commit 235e7e9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion files/en-us/tools/memory/dominators/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ <h2 id="Dominators">Dominators</h2>

<p><img alt="" src="memory-graph-immediate-dominator.svg" style="display: block; margin-left: auto; margin-right: auto;"></p>

<p><a id="multiple-paths">One slight subtlety here is that if an object A is referenced by two other objects B and C, then neither object is its dominator</a>, because you could remove either B or C from the graph, and A would still be retained by its other referrer. Instead, the immediate dominator of A would be its first common ancestor:<br>
<p><a id="multiple-paths">One slight subtlety here is that if an object A is referenced by two other unrelated objects B and C, then neither object is its dominator</a>, because you could remove either B or C from the graph, and A would still be retained by its other referrer. Instead, the immediate dominator of A would be its first common ancestor:<br>
<img alt="" src="memory-graph-dominator-multiple-references.svg" style="display: block; margin-left: auto; margin-right: auto;"></p>

<h2 id="See_also">See also</h2>
Expand Down

0 comments on commit 235e7e9

Please sign in to comment.