From: Rafael Espindola <rafael.espindola@gmail.com>
Date: Thu, 1 Aug 2013 23:56:42 +0000 (+0000)
Subject: Revert "If -fslp-vectorize or -fno-slp-vectorize are given, honor this selection... 
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f818ef4c37e5db740549f86c05accf1bcd5481fc;p=platform%2Fupstream%2Fllvm.git

Revert "If -fslp-vectorize or -fno-slp-vectorize are given, honor this selection. If no flag is given, enable it for -O3."

This reverts commit r187630.

Looks like it is breaking the bots.

llvm-svn: 187633
---

diff --git a/clang/lib/Driver/Tools.cpp b/clang/lib/Driver/Tools.cpp
index b6f6c3d..3a23732 100644
--- a/clang/lib/Driver/Tools.cpp
+++ b/clang/lib/Driver/Tools.cpp
@@ -1974,12 +1974,6 @@ static bool isOptimizationLevelFast(const ArgList &Args) {
   return false;
 }
 
-static bool isOptimizationLevel3(const ArgList &Args) {
-  if (Arg *A = Args.getLastArg(options::OPT_O_Group))
-    return StringRef(A->getValue()).equals("3");
-  return false;
-}
-
 void Clang::ConstructJob(Compilation &C, const JobAction &JA,
                          const InputInfo &Output,
                          const InputInfoList &Inputs,
@@ -3458,17 +3452,9 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
                    options::OPT_fno_vectorize, true))
     CmdArgs.push_back("-vectorize-loops");
 
-  bool EnableSLP = false;
-  // If -fslp-vectorize or -fno-slp-vectorize are given, honor this selection.
-  if (Arg *A = Args.getLastArg(options::OPT_fslp_vectorize,
-                      options::OPT_fno_slp_vectorize)) {
-    EnableSLP = A->getOption().matches(options::OPT_fslp_vectorize);
-  } else {
-    // Also enable SLP vectorization on O3 or OFast
-    EnableSLP = isOptimizationLevel3(Args) || isOptimizationLevelFast(Args);
-  }
-
-  if (EnableSLP)
+  // -fslp-vectorize is default.
+  if (Args.hasFlag(options::OPT_fslp_vectorize,
+                   options::OPT_fno_slp_vectorize, true))
     CmdArgs.push_back("-vectorize-slp");
 
   // -fno-slp-vectorize-aggressive is default.