diff options
author | Owen Anderson <resistor@mac.com> | 2007-04-14 23:57:00 +0000 |
---|---|---|
committer | Owen Anderson <resistor@mac.com> | 2007-04-14 23:57:00 +0000 |
commit | e59b36defa22db98f0640cddb7550f0fdf8e2efe (patch) | |
tree | 51c58bdbc051bda81e76215e558181b7b1a3952a /llvm/lib | |
parent | 78cecc817fd471a266d1a4c7ab9d956eb9875579 (diff) | |
download | bcm5719-llvm-e59b36defa22db98f0640cddb7550f0fdf8e2efe.tar.gz bcm5719-llvm-e59b36defa22db98f0640cddb7550f0fdf8e2efe.zip |
Fix some unsafe code. Also, tabs -> spaces.
llvm-svn: 36035
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/VMCore/Dominators.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/llvm/lib/VMCore/Dominators.cpp b/llvm/lib/VMCore/Dominators.cpp index da8d36d9021..0959b07b47a 100644 --- a/llvm/lib/VMCore/Dominators.cpp +++ b/llvm/lib/VMCore/Dominators.cpp @@ -929,13 +929,13 @@ ETNode *ETForest::getNodeForBlock(BasicBlock *BB) { // Haven't calculated this node yet? Get or calculate the node for the // immediate dominator. - BasicBlock *IDom = getAnalysis<DominatorTree>().getNode(BB)->getIDom()->getBlock(); + DominatorTree::Node *node= getAnalysis<DominatorTree>().getNode(BB); // If we are unreachable, we may not have an immediate dominator. - if (!IDom) + if (!node || !node->getIDom()) return BBNode = new ETNode(BB); else { - ETNode *IDomNode = getNodeForBlock(IDom); + ETNode *IDomNode = getNodeForBlock(node->getIDom()->getBlock()); // Add a new tree node for this BasicBlock, and link it as a child of // IDomNode @@ -953,9 +953,9 @@ void ETForest::calculate(const DominatorTree &DT) { Function *F = Root->getParent(); // Loop over all of the reachable blocks in the function... for (Function::iterator I = F->begin(), E = F->end(); I != E; ++I) { - DominatorTree::Node* node = DT.getNode(I); + DominatorTree::Node* node = DT.getNode(I); if (node && node->getIDom()) { // Reachable block. - BasicBlock* ImmDom = node->getIDom()->getBlock(); + BasicBlock* ImmDom = node->getIDom()->getBlock(); ETNode *&BBNode = Nodes[I]; if (!BBNode) { // Haven't calculated this node yet? // Get or calculate the node for the immediate dominator @@ -967,7 +967,7 @@ void ETForest::calculate(const DominatorTree &DT) { BBNode->setFather(IDomNode); } } - } + } // Make sure we've got nodes around for every block for (Function::iterator I = F->begin(), E = F->end(); I != E; ++I) { |