From 3567937f16643b80f995ee9c9cd3e3f75027d460 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sat, 21 Feb 2004 00:30:28 +0000 Subject: [PATCH] Instead of cloning the globals for main into the globals graph at the end of BU propagation, clone the globals into the GG of EACH FUNCTION that finishes processing! The GlobalsGraph *must* include all globals and effects from all functions in the program. Fixing this makes pool allocation work better on 175.vpr, but it still ultimately crashes. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11686 91177308-0d34-0410-b5e6-96231b3b80d8 --- .../DataStructure/BottomUpClosure.cpp | 28 ++++++++----------- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a/lib/Analysis/DataStructure/BottomUpClosure.cpp b/lib/Analysis/DataStructure/BottomUpClosure.cpp index fceb1ded59a..2125064030e 100644 --- a/lib/Analysis/DataStructure/BottomUpClosure.cpp +++ b/lib/Analysis/DataStructure/BottomUpClosure.cpp @@ -57,23 +57,6 @@ bool BUDataStructures::run(Module &M) { NumCallEdges += ActualCallees.size(); - - // At the end of the BU phase, clone the BU graph for main into the globals - // graph to make sure it has everything. - if (MainFunc) { - DSGraph &MainGraph = getOrCreateGraph(MainFunc); - DSScalarMap &MainSM = MainGraph.getScalarMap(); - ReachabilityCloner RC(*GlobalsGraph, MainGraph, DSGraph::StripAllocaBit); - - // Clone everything reachable from globals in the "main" graph into the - // globals graph. - for (DSScalarMap::global_iterator I = MainSM.global_begin(), - E = MainSM.global_end(); I != E; ++I) - RC.getClonedNH(MainSM[*I]); - - - } - // At the end of the bottom-up pass, the globals graph becomes complete. // FIXME: This is not the right way to do this, but it is sorta better than // nothing! In particular, externally visible globals and unresolvable call @@ -329,5 +312,16 @@ void BUDataStructures::calculateGraph(DSGraph &Graph) { // reach live nodes as live. Graph.removeDeadNodes(DSGraph::KeepUnreachableGlobals); + // When this graph is finalized, clone the globals in the graph into the + // globals graph to make sure it has everything, from all graphs. + DSScalarMap &MainSM = Graph.getScalarMap(); + ReachabilityCloner RC(*GlobalsGraph, Graph, DSGraph::StripAllocaBit); + + // Clone everything reachable from globals in the "main" graph into the + // globals graph. + for (DSScalarMap::global_iterator I = MainSM.global_begin(), + E = MainSM.global_end(); I != E; ++I) + RC.getClonedNH(MainSM[*I]); + //Graph.writeGraphToFile(std::cerr, "bu_" + F.getName()); }