[PATCH] D25963: [LoopUnroll] Implement profile-based loop peeling

Gerolf Hoflehner via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 27 19:49:51 PDT 2016


Gerolf added a reviewer: anemet.
Gerolf added inline comments.


================
Comment at: lib/Transforms/Utils/LoopUnrollPeel.cpp:47
+    cl::desc("Force a peel count regardless of profiling information."));
+
+// Check whether we are capable of peeling this loop.
----------------
Please add an option to turn it off by default.


================
Comment at: lib/Transforms/Utils/LoopUnrollPeel.cpp:125
+                            ValueToValueMapTy &LVMap, LoopInfo *LI) {
+
+  BasicBlock *Header = L->getHeader();
----------------
Could this be implemented as
- distribute k: N-k (assuming trip count N)
- complete unroll first k iterations?


https://reviews.llvm.org/D25963





More information about the llvm-commits mailing list