[PATCH] D84713: [DominatorTree] Simplify ChildrenGetter.

Mon Jul 27 17:34:48 PDT 2020

dblaikie added a comment.

In D84713#2177408 <https://reviews.llvm.org/D84713#2177408>, @kuhar wrote:

> LGTM.
>
> One tiny nit: the function name `ChildrenGet` sounds kind of backwards to me, but it seems like the other direction is already taken.

If there are both "ChildrenGet" and "GetChildren" in the same scope, seems like we probably should address that rather than having one awkwardly named & the ensuing lack of clarity.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84713