[PATCH] D33457: [LV] Update type in cost model for scalarization

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7159
+                           TTI.getNumberOfParts(VectorTy) < VF &&
+                           VectorTy->isVectorTy();
   return VectorizationCostTy(C, TypeNotScalarized);
----------------
isVectorTy() implies !isVoidTy(), so checking the latter becomes redundant. Does the VF > 1 check also become redundant?


https://reviews.llvm.org/D33457