[llvm-commits] CVS: llvm/lib/VMCore/PassManager.cpp

[Devang Patel]

Changes in directory llvm/lib/VMCore:

PassManager.cpp updated: 1.48 -> 1.49
---
Log message:

Top level manages schedule the pass immediately at the time of insertion.


---
Diffs of the changes:  (+8 -10)

 PassManager.cpp |   18 ++++++++----------
 1 files changed, 8 insertions(+), 10 deletions(-)


Index: llvm/lib/VMCore/PassManager.cpp
diff -u llvm/lib/VMCore/PassManager.cpp:1.48 llvm/lib/VMCore/PassManager.cpp:1.49
--- llvm/lib/VMCore/PassManager.cpp:1.48	Thu Dec  7 15:27:23 2006
+++ llvm/lib/VMCore/PassManager.cpp	Thu Dec  7 15:32:57 2006
@@ -346,7 +346,9 @@
   /// PassManager_X is destroyed, the pass will be destroyed as well, so
   /// there is no need to delete the pass. (TODO delete passes.)
   /// This implies that all passes MUST be allocated with 'new'.
-  void add(Pass *P) { /* TODO*/  }
+  void add(Pass *P) { 
+    schedulePass(P, this);
+  }
 
   /// Add pass into the pass manager queue.
   bool addPass(Pass *P);
@@ -407,7 +409,8 @@
 };
 
 /// PassManager_New manages ModulePassManagers
-class PassManagerImpl_New : public PMDataManager,
+class PassManagerImpl_New : public Pass,
+                            public PMDataManager,
                             public PMTopLevelManager {
 
 public:
@@ -416,7 +419,9 @@
   /// the Pass to the PassManager.  When the PassManager is destroyed, the pass
   /// will be destroyed as well, so there is no need to delete the pass.  This
   /// implies that all passes MUST be allocated with 'new'.
-  void add(Pass *P);
+  void add(Pass *P) {
+    schedulePass(P, this);
+  }
  
   /// run - Execute all of the passes scheduled for execution.  Keep track of
   /// whether any of the passes modifies the module, and if so, return true.
@@ -870,13 +875,6 @@
   return P;
 }
 
-/// Add pass P to the queue of passes to run.
-void PassManagerImpl_New::add(Pass *P) {
-  // Do not process Analysis now. Analysis is process while scheduling
-  // the pass vector.
-  addPassToManager(P, false);
-}
-
 // PassManager_New implementation
 /// Add P into active pass manager or use new module pass manager to
 /// manage it.