diff options
| author | Evgeniy Stepanov <eugeni.stepanov@gmail.com> | 2016-12-17 01:53:15 +0000 |
|---|---|---|
| committer | Evgeniy Stepanov <eugeni.stepanov@gmail.com> | 2016-12-17 01:53:15 +0000 |
| commit | 95294127d0ef8264274a1cd9767b15295454af63 (patch) | |
| tree | a2ffe51bab4eb916622e0e9d1be8af96f4f3742d /llvm/lib/Transforms/IPO/PassManagerBuilder.cpp | |
| parent | 982c6b24b04f5dfa28aeedfb3b2e370f2359348f (diff) | |
| download | bcm5719-llvm-95294127d0ef8264274a1cd9767b15295454af63.tar.gz bcm5719-llvm-95294127d0ef8264274a1cd9767b15295454af63.zip | |
Revert "[GVNHoist] Move GVNHoist to function simplification part of pipeline."
This reverts r289696, which caused TSan perf regression.
See PR31382.
llvm-svn: 290030
Diffstat (limited to 'llvm/lib/Transforms/IPO/PassManagerBuilder.cpp')
| -rw-r--r-- | llvm/lib/Transforms/IPO/PassManagerBuilder.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/IPO/PassManagerBuilder.cpp b/llvm/lib/Transforms/IPO/PassManagerBuilder.cpp index b0e4b113f8e..a663eedc4f4 100644 --- a/llvm/lib/Transforms/IPO/PassManagerBuilder.cpp +++ b/llvm/lib/Transforms/IPO/PassManagerBuilder.cpp @@ -249,6 +249,8 @@ void PassManagerBuilder::populateFunctionPassManager( FPM.add(createCFGSimplificationPass()); FPM.add(createSROAPass()); FPM.add(createEarlyCSEPass()); + if(EnableGVNHoist) + FPM.add(createGVNHoistPass()); FPM.add(createLowerExpectIntrinsicPass()); } @@ -293,8 +295,6 @@ void PassManagerBuilder::addFunctionSimplificationPasses( // Break up aggregate allocas, using SSAUpdater. MPM.add(createSROAPass()); MPM.add(createEarlyCSEPass()); // Catch trivial redundancies - if(EnableGVNHoist) - MPM.add(createGVNHoistPass()); // Speculative execution if the target has divergent branches; otherwise nop. MPM.add(createSpeculativeExecutionIfHasBranchDivergencePass()); MPM.add(createJumpThreadingPass()); // Thread jumps. |
