[llvm-commits] [poolalloc] r117542 - in /poolalloc/trunk/lib/DSA: BottomUpClosure.cpp Local.cpp

[Arushi Aggarwal]

Author: aggarwa4
Date: Thu Oct 28 06:21:34 2010
New Revision: 117542

URL: http://llvm.org/viewvc/llvm-project?rev=117542&view=rev
Log:
Changes to build Better call graph. 

Modified:
    poolalloc/trunk/lib/DSA/BottomUpClosure.cpp
    poolalloc/trunk/lib/DSA/Local.cpp

Modified: poolalloc/trunk/lib/DSA/BottomUpClosure.cpp
URL: http://llvm.org/viewvc/llvm-project/poolalloc/trunk/lib/DSA/BottomUpClosure.cpp?rev=117542&r1=117541&r2=117542&view=diff
==============================================================================
--- poolalloc/trunk/lib/DSA/BottomUpClosure.cpp (original)
+++ poolalloc/trunk/lib/DSA/BottomUpClosure.cpp Thu Oct 28 06:21:34 2010
@@ -149,6 +149,7 @@
     if (!(F->isDeclaration())){
       DSGraph *Graph  = getOrCreateGraph(F);
       cloneGlobalsInto(Graph);
+      Graph->buildCallGraph(callgraph, filterCallees);
       Graph->maskIncompleteMarkers();
       Graph->markIncompleteNodes(DSGraph::MarkFormalArgs |
                                    DSGraph::IgnoreGlobals);

Modified: poolalloc/trunk/lib/DSA/Local.cpp
URL: http://llvm.org/viewvc/llvm-project/poolalloc/trunk/lib/DSA/Local.cpp?rev=117542&r1=117541&r2=117542&view=diff
==============================================================================
--- poolalloc/trunk/lib/DSA/Local.cpp (original)
+++ poolalloc/trunk/lib/DSA/Local.cpp Thu Oct 28 06:21:34 2010
@@ -44,7 +44,7 @@
 STATISTIC(NumDirectCall,    "Number of direct calls added");
 STATISTIC(NumIndirectCall,  "Number of indirect calls added");
 STATISTIC(NumAsmCall,       "Number of asm calls collapsed/seen");
-STATISTIC(NumBoringCall,    "Number of pointer-free direct calls ignored");
+//STATISTIC(NumBoringCall,    "Number of pointer-free direct calls ignored");
 STATISTIC(NumIntrinsicCall, "Number of intrinsics called");
 
 RegisterPass<LocalDataStructures>
@@ -972,11 +972,12 @@
     return;
   }
 
+  //Removed so that call graph is correct. Keep information about call sites.
   //uninteresting direct call
-  if (isa<Function>(Callee) && !DSCallGraph::hasPointers(CS)) {
+  /*if (isa<Function>(Callee) && !DSCallGraph::hasPointers(CS)) {
     ++NumBoringCall;
     return;
-  }
+  }*/
 
   // Set up the return value...
   DSNodeHandle RetVal;