diff options
| author | Vedant Kumar <vsk@apple.com> | 2018-03-12 20:49:42 +0000 |
|---|---|---|
| committer | Vedant Kumar <vsk@apple.com> | 2018-03-12 20:49:42 +0000 |
| commit | 3a408538f0913f99f5c726734c684710a4304707 (patch) | |
| tree | 6550129b741e5ef16c7cbcab98794193eab319b6 /llvm/lib/Passes | |
| parent | a3d8ef0f0821f674a24587f0a477cf33e15ae06e (diff) | |
| download | bcm5719-llvm-3a408538f0913f99f5c726734c684710a4304707.tar.gz bcm5719-llvm-3a408538f0913f99f5c726734c684710a4304707.zip | |
Remove the LoopInstSimplify pass (-loop-instsimplify)
LoopInstSimplify is unused and untested. Reading through the commit
history the pass also seems to have a high maintenance burden.
It would be best to retire the pass for now. It should be easy to
recover if we need something similar in the future.
Differential Revision: https://reviews.llvm.org/D44053
llvm-svn: 327329
Diffstat (limited to 'llvm/lib/Passes')
| -rw-r--r-- | llvm/lib/Passes/PassBuilder.cpp | 6 | ||||
| -rw-r--r-- | llvm/lib/Passes/PassRegistry.def | 1 |
2 files changed, 2 insertions, 5 deletions
diff --git a/llvm/lib/Passes/PassBuilder.cpp b/llvm/lib/Passes/PassBuilder.cpp index 89570f7d153..02889541650 100644 --- a/llvm/lib/Passes/PassBuilder.cpp +++ b/llvm/lib/Passes/PassBuilder.cpp @@ -110,7 +110,6 @@ #include "llvm/Transforms/Scalar/LoopDeletion.h" #include "llvm/Transforms/Scalar/LoopDistribute.h" #include "llvm/Transforms/Scalar/LoopIdiomRecognize.h" -#include "llvm/Transforms/Scalar/LoopInstSimplify.h" #include "llvm/Transforms/Scalar/LoopLoadElimination.h" #include "llvm/Transforms/Scalar/LoopPassManager.h" #include "llvm/Transforms/Scalar/LoopPredication.h" @@ -391,9 +390,8 @@ PassBuilder::buildFunctionSimplificationPipeline(OptimizationLevel Level, // FIXME: Currently this is split into two loop pass pipelines because we run // some function passes in between them. These can and should be replaced by // loop pass equivalenst but those aren't ready yet. Specifically, - // `SimplifyCFGPass` and `InstCombinePass` are used. We have - // `LoopSimplifyCFGPass` which isn't yet powerful enough, and the closest to - // the other we have is `LoopInstSimplify`. + // `SimplifyCFGPass` and `InstCombinePass` are used. We just have + // `LoopSimplifyCFGPass` which isn't yet powerful enough. LoopPassManager LPM1(DebugLogging), LPM2(DebugLogging); // Rotate Loop - disable header duplication at -Oz diff --git a/llvm/lib/Passes/PassRegistry.def b/llvm/lib/Passes/PassRegistry.def index bebeb8f2a7e..916b1d54b8a 100644 --- a/llvm/lib/Passes/PassRegistry.def +++ b/llvm/lib/Passes/PassRegistry.def @@ -230,7 +230,6 @@ LOOP_ANALYSIS("ivusers", IVUsersAnalysis()) LOOP_PASS("invalidate<all>", InvalidateAllAnalysesPass()) LOOP_PASS("licm", LICMPass()) LOOP_PASS("loop-idiom", LoopIdiomRecognizePass()) -LOOP_PASS("loop-instsimplify", LoopInstSimplifyPass()) LOOP_PASS("rotate", LoopRotatePass()) LOOP_PASS("no-op-loop", NoOpLoopPass()) LOOP_PASS("print", PrintLoopPass(dbgs())) |
