[PATCH] D27734: [LoopUnroll] Enable PGO-based loop peeling by default

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 16:39:17 PST 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL295796: [LoopUnroll] Enable PGO-based loop peeling by default. (authored by mkuper).

Changed prior to commit:
  https://reviews.llvm.org/D27734?vs=81299&id=89302#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D27734

Files:
  llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp
  llvm/trunk/test/Transforms/LoopUnroll/peel-loop-pgo.ll


Index: llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp
===================================================================
--- llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp
+++ llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp
@@ -110,7 +110,7 @@
              "aggressively unrolled."));
 
 static cl::opt<bool>
-    UnrollAllowPeeling("unroll-allow-peeling", cl::Hidden,
+    UnrollAllowPeeling("unroll-allow-peeling", cl::init(true), cl::Hidden,
                        cl::desc("Allows loops to be peeled when the dynamic "
                                 "trip count is known to be low."));
 
@@ -155,7 +155,7 @@
   UP.AllowExpensiveTripCount = false;
   UP.Force = false;
   UP.UpperBound = false;
-  UP.AllowPeeling = false;
+  UP.AllowPeeling = true;
 
   // Override with any target specific settings
   TTI.getUnrollingPreferences(L, UP);
Index: llvm/trunk/test/Transforms/LoopUnroll/peel-loop-pgo.ll
===================================================================
--- llvm/trunk/test/Transforms/LoopUnroll/peel-loop-pgo.ll
+++ llvm/trunk/test/Transforms/LoopUnroll/peel-loop-pgo.ll
@@ -1,4 +1,4 @@
-; RUN: opt < %s -S -debug-only=loop-unroll -loop-unroll -unroll-allow-peeling 2>&1 | FileCheck %s
+; RUN: opt < %s -S -debug-only=loop-unroll -loop-unroll 2>&1 | FileCheck %s
 ; REQUIRES: asserts
 
 ; Make sure we use the profile information correctly to peel-off 3 iterations


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27734.89302.patch
Type: text/x-patch
Size: 1404 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170222/226d44ce/attachment.bin>


More information about the llvm-commits mailing list