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

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 22 15:09:32 PDT 2017


hfinkel 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.
----------------
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.


https://reviews.llvm.org/D33396





More information about the llvm-commits mailing list