diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-05-10 02:35:44 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-05-10 02:35:44 +0000 |
commit | 12c91dc4c86861ff6289664a501b3271ee469fee (patch) | |
tree | 5bac4edded86f36b6de1fb0923d63d402639a513 /llvm/lib/Analysis/ValueTracking.cpp | |
parent | d47f42435ae9fe8a9f4941fd79ac315d4ea46bcf (diff) | |
download | bcm5719-llvm-12c91dc4c86861ff6289664a501b3271ee469fee.tar.gz bcm5719-llvm-12c91dc4c86861ff6289664a501b3271ee469fee.zip |
[ValueTracking] Use guards to prove non-nullness of a value
Reviewers: apilipenko, majnemer, reames
Subscribers: mcrosier, llvm-commits
Differential Revision: http://reviews.llvm.org/D20044
llvm-svn: 269008
Diffstat (limited to 'llvm/lib/Analysis/ValueTracking.cpp')
-rw-r--r-- | llvm/lib/Analysis/ValueTracking.cpp | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp index 52b1e444731..19fef59e1f3 100644 --- a/llvm/lib/Analysis/ValueTracking.cpp +++ b/llvm/lib/Analysis/ValueTracking.cpp @@ -3212,17 +3212,19 @@ static bool isKnownNonNullFromDominatingCondition(const Value *V, continue; for (auto *CmpU : U->users()) { - const BranchInst *BI = dyn_cast<BranchInst>(CmpU); - if (!BI) - continue; - - assert(BI->isConditional() && "uses a comparison!"); + if (const BranchInst *BI = dyn_cast<BranchInst>(CmpU)) { + assert(BI->isConditional() && "uses a comparison!"); - BasicBlock *NonNullSuccessor = - BI->getSuccessor(Pred == ICmpInst::ICMP_EQ ? 1 : 0); - BasicBlockEdge Edge(BI->getParent(), NonNullSuccessor); - if (Edge.isSingleEdge() && DT->dominates(Edge, CtxI->getParent())) + BasicBlock *NonNullSuccessor = + BI->getSuccessor(Pred == ICmpInst::ICMP_EQ ? 1 : 0); + BasicBlockEdge Edge(BI->getParent(), NonNullSuccessor); + if (Edge.isSingleEdge() && DT->dominates(Edge, CtxI->getParent())) + return true; + } else if (Pred == ICmpInst::ICMP_NE && + match(CmpU, m_Intrinsic<Intrinsic::experimental_guard>()) && + DT->dominates(cast<Instruction>(CmpU), CtxI)) { return true; + } } } |