From 6455c51ab6d331dda51a6bf2210bb701e7c7b522 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sat, 18 Sep 2004 21:37:03 +0000 Subject: [PATCH] Fix the inliner to always delete any edges from the external call node to a function being deleted. Due to optimizations done while inlining, there can be edges from the external call node to a function node that were not apparent any longer. This fixes the compiler crash while compiling 175.vpr llvm-svn: 16399 --- llvm/lib/Transforms/IPO/Inliner.cpp | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/llvm/lib/Transforms/IPO/Inliner.cpp b/llvm/lib/Transforms/IPO/Inliner.cpp index 86d343af6a04..9c8d7aa91095 100644 --- a/llvm/lib/Transforms/IPO/Inliner.cpp +++ b/llvm/lib/Transforms/IPO/Inliner.cpp @@ -168,22 +168,21 @@ bool Inliner::doFinalization(CallGraph &CG) { for (CallGraph::iterator I = CG.begin(), E = CG.end(); I != E; ++I) { CallGraphNode *CGN = I->second; if (Function *F = CGN ? CGN->getFunction() : 0) { - // If the only remaining users of the function are dead constants, - // remove them. - bool HadDeadConstantUsers = !F->use_empty(); + // If the only remaining users of the function are dead constants, remove + // them. F->removeDeadConstantUsers(); if ((F->hasLinkOnceLinkage() || F->hasInternalLinkage()) && F->use_empty()) { + // Remove any call graph edges from the function to its callees. while (CGN->begin() != CGN->end()) CGN->removeCallEdgeTo(*(CGN->end()-1)); - // If the function has external linkage (basically if it's a linkonce - // function) remove the edge from the external node to the callee - // node. - if (!F->hasInternalLinkage() || HadDeadConstantUsers) - CG.getExternalCallingNode()->removeCallEdgeTo(CGN); + // Remove any edges from the external node to the function's call graph + // node. These edges might have been made irrelegant due to + // optimization of the program. + CG.getExternalCallingNode()->removeAnyCallEdgeTo(CGN); // Removing the node for callee from the call graph and delete it. FunctionsToRemove.insert(CGN);