[PATCH] D28831: Introduce -unroll-partial-threshold to separate PartialThreshold from Threshold in loop unorller.

Michael Zolotukhin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 17 15:26:14 PST 2017

mzolotukhin accepted this revision.
mzolotukhin added a comment.
This revision is now accepted and ready to land.

LGTM, but I'd like to wait for Eli's reply too (considering that he suggested a different naming scheme).


Comment at: lib/Transforms/Scalar/LoopUnrollPass.cpp:47
     UnrollThreshold("unroll-threshold", cl::Hidden,
-                    cl::desc("The baseline cost threshold for loop unrolling"));
+                    cl::desc("The cost threshold for loop unrolling"));
Please update the description to reflect that this is for full unrolling.


More information about the llvm-commits mailing list