[llvm-commits] CVS: llvm/tools/gccld/GenerateCode.cpp

[Brian Gaeke]

Changes in directory llvm/tools/gccld:

GenerateCode.cpp updated: 1.13 -> 1.14

---
Log message:

Add a -verify option to verify the results of gccld passes.
Add a -disable-opt option to turn off gccld optimization passes.


---
Diffs of the changes:  (+68 -48)

Index: llvm/tools/gccld/GenerateCode.cpp
diff -u llvm/tools/gccld/GenerateCode.cpp:1.13 llvm/tools/gccld/GenerateCode.cpp:1.14
--- llvm/tools/gccld/GenerateCode.cpp:1.13	Tue Nov 11 16:41:34 2003
+++ llvm/tools/gccld/GenerateCode.cpp	Sun Nov 16 17:07:28 2003
@@ -17,6 +17,7 @@
 #include "llvm/Module.h"
 #include "llvm/PassManager.h"
 #include "llvm/Analysis/LoadValueNumbering.h"
+#include "llvm/Analysis/Verifier.h"
 #include "llvm/Bytecode/WriteBytecodePass.h"
 #include "llvm/Target/TargetData.h"
 #include "llvm/Transforms/IPO.h"
@@ -30,10 +31,25 @@
 namespace {
   cl::opt<bool>
   DisableInline("disable-inlining", cl::desc("Do not run the inliner pass"));
+
+  cl::opt<bool>
+  Verify("verify", cl::desc("Verify intermediate results of all passes"));
+
+  cl::opt<bool>
+  DisableOptimizations("disable-opt",
+                       cl::desc("Do not run any optimization passes"));
 }
 
 namespace llvm {
 
+static inline void addPass(PassManager &PM, Pass *P) {
+  // Add the pass to the pass manager...
+  PM.add(P);
+  
+  // If we are verifying all of the intermediate steps, add the verifier...
+  if (Verify) PM.add(createVerifierPass());
+}
+
 /// GenerateBytecode - generates a bytecode file from the specified module.
 ///
 /// Inputs:
@@ -53,60 +69,64 @@
   // a little bit.  Do this now.
   PassManager Passes;
 
-  // Add an appropriate TargetData instance for this module...
-  Passes.add(new TargetData("gccld", M));
+  if (Verify) Passes.add(createVerifierPass());
 
-  // Linking modules together can lead to duplicated global constants, only keep
-  // one copy of each constant...
-  Passes.add(createConstantMergePass());
-
-  // If the -s command line option was specified, strip the symbols out of the
-  // resulting program to make it smaller.  -s is a GCC option that we are
-  // supporting.
-  if (Strip)
-    Passes.add(createSymbolStrippingPass());
-
-  // Often if the programmer does not specify proper prototypes for the
-  // functions they are calling, they end up calling a vararg version of the
-  // function that does not get a body filled in (the real function has typed
-  // arguments).  This pass merges the two functions.
-  Passes.add(createFunctionResolvingPass());
-
-  if (Internalize) {
-    // Now that composite has been compiled, scan through the module, looking
-    // for a main function.  If main is defined, mark all other functions
-    // internal.
-    Passes.add(createInternalizePass());
-  }
-
-  // Propagate constants at call sites into the functions they call.
-  Passes.add(createIPConstantPropagationPass());
-
-  // Remove unused arguments from functions...
-  Passes.add(createDeadArgEliminationPass());
-
-  if (!DisableInline)
-    Passes.add(createFunctionInliningPass());   // Inline small functions
-
-  // Run a few AA driven optimizations here and now, to cleanup the code.
-  // Eventually we should put an IP AA in place here.
-
-  Passes.add(createLICMPass());                 // Hoist loop invariants
-  Passes.add(createLoadValueNumberingPass());   // GVN for load instructions
-  Passes.add(createGCSEPass());                 // Remove common subexprs
+  // Add an appropriate TargetData instance for this module...
+  addPass (Passes, new TargetData("gccld", M));
 
-  // The FuncResolve pass may leave cruft around if functions were prototyped
-  // differently than they were defined.  Remove this cruft.
-  Passes.add(createInstructionCombiningPass());
+  if (!DisableOptimizations) {
+    // Linking modules together can lead to duplicated global constants, only
+    // keep one copy of each constant...
+    addPass (Passes, createConstantMergePass());
+
+    // If the -s command line option was specified, strip the symbols out of the
+    // resulting program to make it smaller.  -s is a GCC option that we are
+    // supporting.
+    if (Strip)
+      addPass (Passes, createSymbolStrippingPass());
+
+    // Often if the programmer does not specify proper prototypes for the
+    // functions they are calling, they end up calling a vararg version of the
+    // function that does not get a body filled in (the real function has typed
+    // arguments).  This pass merges the two functions.
+    addPass (Passes, createFunctionResolvingPass());
+
+    if (Internalize) {
+      // Now that composite has been compiled, scan through the module, looking
+      // for a main function.  If main is defined, mark all other functions
+      // internal.
+      addPass (Passes, createInternalizePass());
+    }
+
+    // Propagate constants at call sites into the functions they call.
+    addPass (Passes, createIPConstantPropagationPass());
+
+    // Remove unused arguments from functions...
+    addPass (Passes, createDeadArgEliminationPass());
+
+    if (!DisableInline)
+      addPass (Passes, createFunctionInliningPass()); // Inline small functions
+
+    // Run a few AA driven optimizations here and now, to cleanup the code.
+    // Eventually we should put an IP AA in place here.
+
+    addPass (Passes, createLICMPass());               // Hoist loop invariants
+    addPass (Passes, createLoadValueNumberingPass()); // GVN for load instrs
+    addPass (Passes, createGCSEPass());               // Remove common subexprs
+
+    // The FuncResolve pass may leave cruft around if functions were prototyped
+    // differently than they were defined.  Remove this cruft.
+    addPass (Passes, createInstructionCombiningPass());
 
-  // Delete basic blocks, which optimization passes may have killed...
-  Passes.add(createCFGSimplificationPass());
+    // Delete basic blocks, which optimization passes may have killed...
+    addPass (Passes, createCFGSimplificationPass());
 
-  // Now that we have optimized the program, discard unreachable functions...
-  Passes.add(createGlobalDCEPass());
+    // Now that we have optimized the program, discard unreachable functions...
+    addPass (Passes, createGlobalDCEPass());
+  }
 
   // Add the pass that writes bytecode to the output file...
-  Passes.add(new WriteBytecodePass(Out));
+  addPass (Passes, new WriteBytecodePass(Out));
 
   // Run our queue of passes all at once now, efficiently.
   Passes.run(*M);