[PATCH] D62507: [Dominators] PR42041: Skip nullpointer successors

Jakub Kuderski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 28 08:27:03 PDT 2019


kuhar added inline comments.


================
Comment at: clang/include/clang/Analysis/Analyses/Dominators.h:190
+    clang::CFGBlock *N, std::integral_constant<bool, false>) {
+  auto RChildren = reverse(children<NodePtr>(N));
+  ResultTy Ret(RChildren.begin(), RChildren.end());
----------------
Shouldn't one of the cases use `inverse_children`?


Repository:
  rC Clang

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

https://reviews.llvm.org/D62507





More information about the cfe-commits mailing list