[PATCH] D61955: [SLPVectorizer] Set flag to previous default.
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 15 11:01:36 PDT 2019
asbirlea created this revision.
asbirlea added a reviewer: chandlerc.
Herald added subscribers: dexonsmith, steven_wu, eraman, jlebar, mehdi_amini.
Herald added a project: LLVM.
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 <https://reviews.llvm.org/D61617> checked in to enable it in clang.
Repository:
rL LLVM
https://reviews.llvm.org/D61955
Files:
lib/Transforms/Vectorize/SLPVectorizer.cpp
test/Other/new-pm-defaults.ll
test/Other/new-pm-thinlto-defaults.ll
Index: test/Other/new-pm-thinlto-defaults.ll
===================================================================
--- test/Other/new-pm-thinlto-defaults.ll
+++ 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
Index: test/Other/new-pm-defaults.ll
===================================================================
--- test/Other/new-pm-defaults.ll
+++ 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
Index: lib/Transforms/Vectorize/SLPVectorizer.cpp
===================================================================
--- lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -106,7 +106,7 @@
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>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61955.199644.patch
Type: text/x-patch
Size: 1756 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190515/9429461b/attachment.bin>
More information about the llvm-commits
mailing list