From 5965359d8f404844b5a211834fd7ac26a31accda Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 25 Apr 2005 19:16:17 +0000 Subject: [PATCH] Don't mess up SCC traversal when a node has null edges out of it. llvm-svn: 21536 --- llvm/lib/Analysis/DataStructure/DataStructure.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/llvm/lib/Analysis/DataStructure/DataStructure.cpp b/llvm/lib/Analysis/DataStructure/DataStructure.cpp index a95d2522a0a2..904d8b10b769 100644 --- a/llvm/lib/Analysis/DataStructure/DataStructure.cpp +++ b/llvm/lib/Analysis/DataStructure/DataStructure.cpp @@ -1430,11 +1430,12 @@ VisitForSCCs(const DSNode *N) { // Otherwise, check all successors. bool AnyDirectSuccessorsReachClonedNodes = false; for (DSNode::const_edge_iterator EI = N->edge_begin(), EE = N->edge_end(); - EI != EE; ++EI) { - std::pair &SuccInfo = VisitForSCCs(EI->getNode()); - if (SuccInfo.first < Min) Min = SuccInfo.first; - AnyDirectSuccessorsReachClonedNodes |= SuccInfo.second; - } + EI != EE; ++EI) + if (DSNode *Succ = EI->getNode()) { + std::pair &SuccInfo = VisitForSCCs(Succ); + if (SuccInfo.first < Min) Min = SuccInfo.first; + AnyDirectSuccessorsReachClonedNodes |= SuccInfo.second; + } if (Min != MyId) return ThisNodeInfo; // Part of a large SCC. Leave self on stack.