Revert "Revert "enable the SLP Vectorizer optimization pass by defaul…

…t"" (JuliaLang#28344)
Liozou · Aug 1, 2018 · 0ef8826 · 0ef8826
1 parent fe9a075
commit 0ef8826
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -776,6 +776,9 @@ Compiler/Runtime improvements
  * Inference now propagates constants inter-procedurally, and can compute
  various constants expressions at compile-time ([#24362]).
 
+ * The LLVM SLP Vectorizer optimization pass is now enabled at the default
+ optimization level.
+
 Deprecated or removed
 ---------------------
 

diff --git a/src/jitlayers.cpp b/src/jitlayers.cpp
@@ -257,14 +257,9 @@ void addOptimizationPasses(legacy::PassManagerBase *PM, int opt_level, bool dump
  PM->add(createLoopIdiomPass());
  PM->add(createLoopDeletionPass()); // Delete dead loops
  PM->add(createJumpThreadingPass()); // Thread jumps
-
- if (opt_level >= 3) {
- PM->add(createSLPVectorizerPass()); // Vectorize straight-line code
- }
-
+ PM->add(createSLPVectorizerPass()); // Vectorize straight-line code
  PM->add(createAggressiveDCEPass()); // Delete dead instructions
- if (opt_level >= 3)
- PM->add(createInstructionCombiningPass()); // Clean up after SLP loop vectorizer
+ PM->add(createInstructionCombiningPass()); // Clean up after SLP loop vectorizer
  PM->add(createLoopVectorizePass()); // Vectorize loops
  PM->add(createInstructionCombiningPass()); // Clean up after loop vectorizer
  // LowerPTLS removes an indirect call. As a result, it is likely to trigger