[PATCH] D18237: [SLPVectorizer] Try to vectorize in the range from MaxVecRegSize to MinVecRegSize

[Jongwon Lee via llvm-commits]

JongwonLee added inline comments.

================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4323
@@ +4322,3 @@
+      if (NumElts > 1) {
+        bool SuccessToVectorizeList = false;
+        for (unsigned VecRegSize = MaxVecRegSize; VecRegSize >= MinVecRegSize;
----------------
mcrosier wrote:
> No that this logic is cleaned up you don't need the temporary bool.
> 
>   for (unsigned VecRegSize = MaxVecRegSize; VecRegSize >= MinVecRegSize; VecRegSize /= 2) {
>     if (tryToVectorizeList(makeArrayRef(IncIt, NumElts), R, None, false, VecRegSize)) {
>       // Success start over because instructions might have been changed.
>       HaveVectorizedPhiNodes = true;
>       Changed = true;
>       break;
>     }
>   }
Removed the temporary bool.


http://reviews.llvm.org/D18237