diff options
| author | Alina Sbirlea <asbirlea@google.com> | 2019-05-23 19:07:41 +0000 |
|---|---|---|
| committer | Alina Sbirlea <asbirlea@google.com> | 2019-05-23 19:07:41 +0000 |
| commit | 63729b0c49d29ea96cb89c54187f5a3d2e1a0602 (patch) | |
| tree | 9429459aea980702bee8a863f1708b324d8a9026 | |
| parent | a8e0d49c0ce0f1015cc20d3fcac823facc69df66 (diff) | |
| download | bcm5719-llvm-63729b0c49d29ea96cb89c54187f5a3d2e1a0602.tar.gz bcm5719-llvm-63729b0c49d29ea96cb89c54187f5a3d2e1a0602.zip | |
[SLPVectorizer] Set flag to previous default.
Summary:
The refactoring in r360276 moved the `RunSLPVectorization` flag and added the default explicitly. The default should have been `false`, as before.
The new pass manager used to have SLPVectorization on by default, now it's off in opt, and needs D61617 checked in to enable it in clang.
Reviewers: chandlerc
Subscribers: mehdi_amini, jlebar, eraman, steven_wu, dexonsmith, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D61955
llvm-svn: 361537
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | 2 | ||||
| -rw-r--r-- | llvm/test/Other/new-pm-defaults.ll | 1 | ||||
| -rw-r--r-- | llvm/test/Other/new-pm-thinlto-defaults.ll | 1 |
3 files changed, 1 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index 5b40e5ef1f0..c0c2c85b8f4 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -106,7 +106,7 @@ using namespace slpvectorizer; STATISTIC(NumVectorInstructions, "Number of vector instructions generated"); cl::opt<bool> - llvm::RunSLPVectorization("vectorize-slp", cl::init(true), cl::Hidden, + llvm::RunSLPVectorization("vectorize-slp", cl::init(false), cl::Hidden, cl::desc("Run the SLP vectorization passes")); static cl::opt<int> diff --git a/llvm/test/Other/new-pm-defaults.ll b/llvm/test/Other/new-pm-defaults.ll index 317bffcefdc..d29c127d984 100644 --- a/llvm/test/Other/new-pm-defaults.ll +++ b/llvm/test/Other/new-pm-defaults.ll @@ -243,7 +243,6 @@ ; CHECK-O-NEXT: Running analysis: LoopAccessAnalysis ; CHECK-O-NEXT: Running pass: InstCombinePass ; CHECK-O-NEXT: Running pass: SimplifyCFGPass -; CHECK-O-NEXT: Running pass: SLPVectorizerPass ; CHECK-O-NEXT: Running pass: InstCombinePass ; CHECK-O-NEXT: Running pass: LoopUnrollPass ; CHECK-O-NEXT: Running pass: WarnMissedTransformationsPass diff --git a/llvm/test/Other/new-pm-thinlto-defaults.ll b/llvm/test/Other/new-pm-thinlto-defaults.ll index 079cea9255e..8dc6bf7a239 100644 --- a/llvm/test/Other/new-pm-thinlto-defaults.ll +++ b/llvm/test/Other/new-pm-thinlto-defaults.ll @@ -217,7 +217,6 @@ ; CHECK-POSTLINK-O-NEXT: Running analysis: LoopAccessAnalysis ; CHECK-POSTLINK-O-NEXT: Running pass: InstCombinePass ; CHECK-POSTLINK-O-NEXT: Running pass: SimplifyCFGPass -; CHECK-POSTLINK-O-NEXT: Running pass: SLPVectorizerPass ; CHECK-POSTLINK-O-NEXT: Running pass: InstCombinePass ; CHECK-POSTLINK-O-NEXT: Running pass: LoopUnrollPass ; CHECK-POSTLINK-O-NEXT: Running pass: WarnMissedTransformationsPass |
