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

[Adam Nemet via Phabricator via llvm-commits]

anemet added inline comments.


================
Comment at: docs/Vectorizers.rst:101-103
 ``-Rpass-analysis=loop-vectorize`` identifies the statements that caused
-vectorization to fail.
+vectorization to fail. If in addition ``-fsave-optimization-record`` is
+provided, multiple causes of vectorization failure may be listed.
----------------
hfinkel wrote:
> anemet wrote:
> > I am curious what others think but I am not sure we want to advertise this, i.e. this is more of a bug than a feature.  After PR32352 this should work even without -fsave-optimization-record. 
> I agree (it is more of a bug than a feature), but on the other hand, documenting it leads to fewer surprises (hopefully). We might add, "(this behavior might change in the future)" to make this clear.
SGTM


https://reviews.llvm.org/D33396