[PATCH] D83092: DomTree: Add findSiblingOfUncle helper

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 3 08:34:26 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/lib/Support/GenericDomTree.cpp:217-219
+/// findSiblingOfUncle - Under the assumption that \p Uncle is the sibling
+/// of some ancestor of \p A in the tree, find that ancestor. Also handles
+/// the degenerate case where \p A itself is a sibling of \p Uncle.
----------------
Should go in header?


================
Comment at: llvm/lib/Support/GenericDomTree.cpp:220
+/// the degenerate case where \p A itself is a sibling of \p Uncle.
+const GenericDomTreeNodeBase *GenericDominatorTreeBase::findSiblingOfUncle(
+    const GenericDomTreeNodeBase *A,
----------------
I'm not sure these are the right family analogies. This could also find a great uncle, or the same parent.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83092/new/

https://reviews.llvm.org/D83092





More information about the llvm-commits mailing list