[PATCH] D33396: [LV] Report multiple reasons for not vectorizing under allowExtraAnalysis

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:5211
   // which may limit our maximum vectorization factor, so just return true with
   // no restrictions.
+  if (ORE->allowExtraAnalysis())
----------------
fhahn wrote:
> I think the comment also needs updating, mentioning the ORE->allowExtraAnalysis() case
OK, sure.

Additional documentation that needs updating is docs/Vectorizers.rst.

Fixed both and uploaded updated version.


https://reviews.llvm.org/D33396