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 :(

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@10992 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Analysis/DataStructure/DataStructure.cpp b/lib/Analysis/DataStructure/DataStructure.cpp
index c0be41331ce548799383de635f6e5c6a66aba48c..881e8d6d4cc7ae21a4baa87f5f5cad4a5863399a 100644 (file)
--- a/lib/Analysis/DataStructure/DataStructure.cpp
+++ b/lib/Analysis/DataStructure/DataStructure.cpp
@@ -1070,7 +1070,7 @@ void DSGraph::updateFromGlobalGraph() {
       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.