[llvm-commits] CVS: llvm/lib/Analysis/DataStructure/DataStructure.cpp

[Chris Lattner]

Changes in directory llvm/lib/Analysis/DataStructure:

DataStructure.cpp updated: 1.140 -> 1.141

---
Log message:

Further reduce the number of nodes cloned with getClonedNH, using merge instead.
This reduces the number of nodes allocated, then immediately merged and DNE'd
from 2193852 to 1298049.  unfortunately this only speeds DSA up by ~1.5s (of 
53s), because it's spending most of its time waddling through the scalar map :(



---
Diffs of the changes:  (+1 -1)

Index: llvm/lib/Analysis/DataStructure/DataStructure.cpp
diff -u llvm/lib/Analysis/DataStructure/DataStructure.cpp:1.140 llvm/lib/Analysis/DataStructure/DataStructure.cpp:1.141
--- llvm/lib/Analysis/DataStructure/DataStructure.cpp:1.140	Tue Jan 27 20:05:05 2004
+++ llvm/lib/Analysis/DataStructure/DataStructure.cpp	Tue Jan 27 20:11:49 2004
@@ -1070,7 +1070,7 @@
       if (InlinedGlobals.count(GV) == 0) { // GNode is not up-to-date
         ScalarMapTy::iterator It = GlobalsGraph->ScalarMap.find(GV);
         if (It != GlobalsGraph->ScalarMap.end())
-          RC.getClonedNH(It->second);
+          RC.merge(I->second, It->second);
       }
   
   // Merging global nodes leaves behind unused nodes: get rid of them now.