[PATCH] D30305: [LV] Consider non-consecutive vectorizable accesses in max VF selection

[Michael Kuperstein via Phabricator via llvm-commits]

mkuper added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6341
+      //
+      if (T->isPointerTy() && !isConsecutiveLoadOrStore(&I) &&
+          !Legal->isAccessInterleaved(&I) && !Legal->isLegalGatherOrScatter(&I))
----------------
Why would this only apply to pointer types, though?  What's special about them?
(It looks like it was a heuristic of some sort, but I'm not sure it makes sense anymore.)


https://reviews.llvm.org/D30305