From: Chris Lattner Date: Mon, 1 Apr 2002 00:13:56 +0000 (+0000) Subject: Allow merging of identical call nodes. Make the shadow node pointed to X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=f30185fdfbb61bd63999b819bb117a8f9776296a;p=oota-llvm.git Allow merging of identical call nodes. Make the shadow node pointed to by the call node noncritical before the call is destroyed. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2082 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Analysis/DataStructure/EliminateNodes.cpp b/lib/Analysis/DataStructure/EliminateNodes.cpp index 7dd5b3317b0..767f7fe006d 100644 --- a/lib/Analysis/DataStructure/EliminateNodes.cpp +++ b/lib/Analysis/DataStructure/EliminateNodes.cpp @@ -53,6 +53,23 @@ static void DestroyFirstNodeOfPair(DSNode *N1, DSNode *N2) { assert(RanOnce && "Node on user set but cannot find the use!"); } + // If we are about to eliminate a call node that returns a pointer, make the + // shadow node it points to not be critical anymore! + // + if (isa(N1) && N1->getNumLinks()) { + assert(N1->getNumLinks() == 1 && "Call node can only return one pointer!"); + PointerValSet &PVS = N1->getLink(0); + + for (unsigned i = 0, e = PVS.size(); i != e; ++i) + if (ShadowDSNode *Shad = dyn_cast(PVS[i].Node)) + if (Shad->isCriticalNode()) { + Shad->resetCriticalMark(); // Only unmark _ONE_ node.. + break; + } + + } + + N1->removeAllIncomingEdges(); delete N1; } @@ -170,10 +187,7 @@ bool FunctionDSGraph::UnlinkUndistinguishableNodes() { return removeIndistinguishableNodes(AllocNodes) | removeIndistinguishableNodes(ShadowNodes) | - //FIXME: We cannot naively remove call nodes here because if there is a - // shadow node that is the result of the call, we have to make sure to - // merge the shadow node as well!!! - // removeIndistinguishableNodePairs(CallNodes) | + removeIndistinguishableNodePairs(CallNodes) | removeIndistinguishableNodePairs(GlobalNodes); }