[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 13:58:08 PDT 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:5089
 bool LoopVectorizationLegality::canVectorize() {
+  bool Result = true;
   // We must have a loop in canonical form. Loops with indirectbr in them cannot
----------------
Please add a comment here explaining that we use this for the allowExtraAnalysis case where we don't early-exit.


https://reviews.llvm.org/D33396





More information about the llvm-commits mailing list