[PATCH] D62507: [Dominators] PR42041: Skip nullpointer successors
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 27 22:16:31 PDT 2019
MaskRay added inline comments.
================
Comment at: llvm/include/llvm/Support/GenericDomTreeConstruction.h:242
+ // unreachable), skip them.
+ if (!Succ)
+ continue;
----------------
I hope we can find an approach that doesn't need the defensive check here. In post dominator tree nullptr represents the virtual root, this check will cause some confusion. I also don't know if a simple check here is sufficient, I guess somewhere else might need similar checks.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62507/new/
https://reviews.llvm.org/D62507
More information about the llvm-commits
mailing list