[polly] r200208 - Allow Polly at all optimization levels

Tobias Grosser tobias at grosser.es
Mon Jan 27 02:44:21 PST 2014


Author: grosser
Date: Mon Jan 27 04:44:21 2014
New Revision: 200208

URL: http://llvm.org/viewvc/llvm-project?rev=200208&view=rev
Log:
Allow Polly at all optimization levels

Restricting Polly to -O3 does not make a lot of sense as it is opt-in anyway
and users who specifically request it should get it. If this causes performance
problems we should rather address them by scheduling the right cleanup passes
then just prevent the user from trying.

Also restricting Polly to -O3 made bugpoint not work with the -O3 flag and polly
enabled.

Modified:
    polly/trunk/lib/RegisterPasses.cpp

Modified: polly/trunk/lib/RegisterPasses.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/RegisterPasses.cpp?rev=200208&r1=200207&r2=200208&view=diff
==============================================================================
--- polly/trunk/lib/RegisterPasses.cpp (original)
+++ polly/trunk/lib/RegisterPasses.cpp Mon Jan 27 04:44:21 2014
@@ -307,16 +307,6 @@ registerPollyEarlyAsPossiblePasses(const
   if (!shouldEnablePolly(Builder.OptLevel))
     return;
 
-  // We only run Polly at optimization level '-O3'.
-  //
-  // This is to ensure that scalar overhead that may be introduced by Polly is
-  // properly cleaned up by LLVM later on. We may reinvestigate this decision
-  // later on.
-  if (Builder.OptLevel != 3) {
-    errs() << "Polly should only be run with -O3. Disabling Polly.\n";
-    return;
-  }
-
   registerPollyPasses(PM);
 }
 





More information about the llvm-commits mailing list