[llvm-commits] CVS: llvm/lib/Transforms/Instrumentation/BlockProfiling.cpp EdgeProfiling.cpp EmitFunctions.cpp TraceBasicBlocks.cpp

Chris Lattner lattner at cs.uiuc.edu
Sun Sep 19 21:44:12 PDT 2004



Changes in directory llvm/lib/Transforms/Instrumentation:

BlockProfiling.cpp updated: 1.9 -> 1.10
EdgeProfiling.cpp updated: 1.3 -> 1.4
EmitFunctions.cpp updated: 1.18 -> 1.19
TraceBasicBlocks.cpp updated: 1.8 -> 1.9
---
Log message:

'Pass' should now not be derived from by clients.  Instead, they should derive
from ModulePass.  Instead of implementing Pass::run, then should implement 
ModulePass::runOnModule.


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

Index: llvm/lib/Transforms/Instrumentation/BlockProfiling.cpp
diff -u llvm/lib/Transforms/Instrumentation/BlockProfiling.cpp:1.9 llvm/lib/Transforms/Instrumentation/BlockProfiling.cpp:1.10
--- llvm/lib/Transforms/Instrumentation/BlockProfiling.cpp:1.9	Sat Jul 17 19:21:14 2004
+++ llvm/lib/Transforms/Instrumentation/BlockProfiling.cpp	Sun Sep 19 23:44:01 2004
@@ -29,15 +29,15 @@
 using namespace llvm;
 
 namespace {
-  class FunctionProfiler : public Pass {
-    bool run(Module &M);
+  class FunctionProfiler : public ModulePass {
+    bool runOnModule(Module &M);
   };
 
   RegisterOpt<FunctionProfiler> X("insert-function-profiling",
                                "Insert instrumentation for function profiling");
 }
 
-bool FunctionProfiler::run(Module &M) {
+bool FunctionProfiler::runOnModule(Module &M) {
   Function *Main = M.getMainFunction();
   if (Main == 0) {
     std::cerr << "WARNING: cannot insert function profiling into a module"
@@ -69,15 +69,15 @@
 
 
 namespace {
-  class BlockProfiler : public Pass {
-    bool run(Module &M);
+  class BlockProfiler : public ModulePass {
+    bool runOnModule(Module &M);
   };
 
   RegisterOpt<BlockProfiler> Y("insert-block-profiling",
                                "Insert instrumentation for block profiling");
 }
 
-bool BlockProfiler::run(Module &M) {
+bool BlockProfiler::runOnModule(Module &M) {
   Function *Main = M.getMainFunction();
   if (Main == 0) {
     std::cerr << "WARNING: cannot insert block profiling into a module"


Index: llvm/lib/Transforms/Instrumentation/EdgeProfiling.cpp
diff -u llvm/lib/Transforms/Instrumentation/EdgeProfiling.cpp:1.3 llvm/lib/Transforms/Instrumentation/EdgeProfiling.cpp:1.4
--- llvm/lib/Transforms/Instrumentation/EdgeProfiling.cpp:1.3	Sat Jul 17 19:21:14 2004
+++ llvm/lib/Transforms/Instrumentation/EdgeProfiling.cpp	Sun Sep 19 23:44:01 2004
@@ -28,15 +28,15 @@
 using namespace llvm;
 
 namespace {
-  class EdgeProfiler : public Pass {
-    bool run(Module &M);
+  class EdgeProfiler : public ModulePass {
+    bool runOnModule(Module &M);
   };
 
   RegisterOpt<EdgeProfiler> X("insert-edge-profiling",
                               "Insert instrumentation for edge profiling");
 }
 
-bool EdgeProfiler::run(Module &M) {
+bool EdgeProfiler::runOnModule(Module &M) {
   Function *Main = M.getMainFunction();
   if (Main == 0) {
     std::cerr << "WARNING: cannot insert edge profiling into a module"


Index: llvm/lib/Transforms/Instrumentation/EmitFunctions.cpp
diff -u llvm/lib/Transforms/Instrumentation/EmitFunctions.cpp:1.18 llvm/lib/Transforms/Instrumentation/EmitFunctions.cpp:1.19
--- llvm/lib/Transforms/Instrumentation/EmitFunctions.cpp:1.18	Sat Jul 17 19:21:14 2004
+++ llvm/lib/Transforms/Instrumentation/EmitFunctions.cpp	Sun Sep 19 23:44:01 2004
@@ -27,8 +27,8 @@
     BLACK
   };
   
-  struct EmitFunctionTable : public Pass {
-    bool run(Module &M);
+  struct EmitFunctionTable : public ModulePass {
+    bool runOnModule(Module &M);
   };
   
   RegisterOpt<EmitFunctionTable>
@@ -64,7 +64,7 @@
 }
 
 // Per Module pass for inserting function table
-bool EmitFunctionTable::run(Module &M){
+bool EmitFunctionTable::runOnModule(Module &M){
   std::vector<const Type*> vType;
  
   std::vector<Constant *> vConsts;


Index: llvm/lib/Transforms/Instrumentation/TraceBasicBlocks.cpp
diff -u llvm/lib/Transforms/Instrumentation/TraceBasicBlocks.cpp:1.8 llvm/lib/Transforms/Instrumentation/TraceBasicBlocks.cpp:1.9
--- llvm/lib/Transforms/Instrumentation/TraceBasicBlocks.cpp:1.8	Wed Sep  1 17:55:36 2004
+++ llvm/lib/Transforms/Instrumentation/TraceBasicBlocks.cpp	Sun Sep 19 23:44:01 2004
@@ -25,8 +25,8 @@
 using namespace llvm;
 
 namespace {
-  class TraceBasicBlocks : public Pass {
-    bool run(Module &M);
+  class TraceBasicBlocks : public ModulePass {
+    bool runOnModule(Module &M);
   };
 
   RegisterOpt<TraceBasicBlocks> X("trace-basic-blocks",
@@ -52,7 +52,7 @@
   Instruction *InstrCall = new CallInst (InstrFn, Args, "", InsertPos);
 }
 
-bool TraceBasicBlocks::run(Module &M) {
+bool TraceBasicBlocks::runOnModule(Module &M) {
   Function *Main = M.getMainFunction();
   if (Main == 0) {
     std::cerr << "WARNING: cannot insert basic-block trace instrumentation"






More information about the llvm-commits mailing list