[llvm-commits] CVS: llvm-poolalloc/lib/PoolAllocate/PoolAllocate.cpp

[Chris Lattner]

Changes in directory llvm-poolalloc/lib/PoolAllocate:

PoolAllocate.cpp updated: 1.114 -> 1.115
---
Log message:

Don't initialize the valuemap for cloning with all of the globals in the 
program.  This is really slow for programs with lots of globals, and the
valuemapper assumes that globals have an identity mapping unless otherwise
specified anyway.  This speeds up the PA on povray from .89s -> .64s.


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

 PoolAllocate.cpp |    8 +-------
 1 files changed, 1 insertion(+), 7 deletions(-)


Index: llvm-poolalloc/lib/PoolAllocate/PoolAllocate.cpp
diff -u llvm-poolalloc/lib/PoolAllocate/PoolAllocate.cpp:1.114 llvm-poolalloc/lib/PoolAllocate/PoolAllocate.cpp:1.115
--- llvm-poolalloc/lib/PoolAllocate/PoolAllocate.cpp:1.114	Sat Apr  2 15:18:18 2005
+++ llvm-poolalloc/lib/PoolAllocate/PoolAllocate.cpp	Sat Apr  2 15:19:54 2005
@@ -352,13 +352,7 @@
   // Populate the value map with all of the globals in the program.
   // FIXME: This should be unnecessary!
   Module &M = *F.getParent();
-{TIME_REGION(X, "ValueMap Construct");
-  for (Module::iterator I = M.begin(), E=M.end(); I!=E; ++I)
-    ValueMap[I] = I;
-  for (Module::global_iterator I = M.global_begin(), E = M.global_end();
-       I != E; ++I)
-    ValueMap[I] = I;
-}
+
   // Perform the cloning.
   std::vector<ReturnInst*> Returns;
 {TIME_REGION(X, "CFI");