[PATCH] D83089: DomTree: Extract (mostly) read-only logic into type-erased base classes

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/Support/GenericDomTree.h:81-82
 
-  iterator begin() { return Children.begin(); }
-  iterator end() { return Children.end(); }
+  iterator generic_begin() { return Children.begin(); }
+  iterator generic_end() { return Children.end(); }
   const_iterator begin() const { return Children.begin(); }
----------------
Iterating over "generic" seems like a strange naming choice? The generic_children range is a bit better, but this should probably match


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83089