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

[Michael Zolotukhin via Phabricator via llvm-commits]

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).

Michael



================
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.


https://reviews.llvm.org/D28831