Move the EnableFullLoadPRE flag from a separate command-line option to an

argument of createGVNPass and set it automatically for -O3.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@97245 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Bob Wilson 2010-02-26 19:09:47 +00:00
parent ad597f99d9
commit 8561dcd730
3 changed files with 9 additions and 7 deletions

View File

@ -138,7 +138,7 @@ namespace llvm {
PM->add(createLoopUnrollPass()); // Unroll small loops PM->add(createLoopUnrollPass()); // Unroll small loops
PM->add(createInstructionCombiningPass()); // Clean up after the unroller PM->add(createInstructionCombiningPass()); // Clean up after the unroller
if (OptimizationLevel > 1) if (OptimizationLevel > 1)
PM->add(createGVNPass()); // Remove redundancies PM->add(createGVNPass(OptimizationLevel > 2)); // Remove redundancies
PM->add(createMemCpyOptPass()); // Remove memcpy / form memset PM->add(createMemCpyOptPass()); // Remove memcpy / form memset
PM->add(createSCCPPass()); // Constant prop with SCCP PM->add(createSCCPPass()); // Constant prop with SCCP

View File

@ -263,7 +263,8 @@ extern const PassInfo *const LCSSAID;
// GVN - This pass performs global value numbering and redundant load // GVN - This pass performs global value numbering and redundant load
// elimination cotemporaneously. // elimination cotemporaneously.
// //
FunctionPass *createGVNPass(bool NoLoads = false); FunctionPass *createGVNPass(bool EnableFullLoadPRE = true,
bool NoLoads = false);
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
// //

View File

@ -60,7 +60,6 @@ STATISTIC(NumPRELoad, "Number of loads PRE'd");
static cl::opt<bool> EnablePRE("enable-pre", static cl::opt<bool> EnablePRE("enable-pre",
cl::init(true), cl::Hidden); cl::init(true), cl::Hidden);
static cl::opt<bool> EnableLoadPRE("enable-load-pre", cl::init(true)); static cl::opt<bool> EnableLoadPRE("enable-load-pre", cl::init(true));
static cl::opt<bool> EnableFullLoadPRE("enable-full-load-pre", cl::init(false));
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
// ValueTable Class // ValueTable Class
@ -662,10 +661,12 @@ namespace {
bool runOnFunction(Function &F); bool runOnFunction(Function &F);
public: public:
static char ID; // Pass identification, replacement for typeid static char ID; // Pass identification, replacement for typeid
explicit GVN(bool noloads = false) explicit GVN(bool fullloadpre = true, bool noloads = false)
: FunctionPass(&ID), NoLoads(noloads), MD(0) { } : FunctionPass(&ID), EnableFullLoadPRE(fullloadpre),
NoLoads(noloads), MD(0) { }
private: private:
bool EnableFullLoadPRE;
bool NoLoads; bool NoLoads;
MemoryDependenceAnalysis *MD; MemoryDependenceAnalysis *MD;
DominatorTree *DT; DominatorTree *DT;
@ -710,8 +711,8 @@ namespace {
} }
// createGVNPass - The public interface to this file... // createGVNPass - The public interface to this file...
FunctionPass *llvm::createGVNPass(bool NoLoads) { FunctionPass *llvm::createGVNPass(bool EnableFullLoadPRE, bool NoLoads) {
return new GVN(NoLoads); return new GVN(EnableFullLoadPRE, NoLoads);
} }
static RegisterPass<GVN> X("gvn", static RegisterPass<GVN> X("gvn",