[PATCH] D36220: [Loop Vectorize] Added a separate metadata

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

> Added a separate metadata to indicate when the loop has already been vectorized instead of setting width and count to 1.

What's the motivation?



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:1320
                << "loop not vectorized: vectorization and interleaving are "
-                  "explicitly disabled, or vectorize width and interleave "
-                  "count are both set to 1");
+                  "explicitly disabled, or IsVectorized "
+                  "is set to 1");
----------------
These messages are read by users so referring to internal variables is not appropriate.  How about, "or the loop has already been vectorized".



https://reviews.llvm.org/D36220