[PATCH] D36258: Disable loop peeling during full unrolling pass.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 2 22:33:47 PDT 2017


davidxl added a comment.

Perhaps it is cleaner to

1. pass the information whether it is a full unroll to tryToUnroll
2. add a internal option FullUnrollAllowPeeling and make it off by default?  This will be similar to UnrollAllowPeeling flag, but takes precedence if it is full unroll.



================
Comment at: lib/Transforms/Scalar/LoopUnrollPass.cpp:204
     UP.UpperBound = *UserUpperBound;
+  if (UserAllowPeeling.hasValue())
+    UP.AllowPeeling = *UserAllowPeeling;
----------------
Move this closer to line 188 above?


https://reviews.llvm.org/D36258





More information about the llvm-commits mailing list