[llvm] [LoopVectorize] Use new single string variant of reportVectorizationFailure (PR #120414)

Hari Limaye via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 18 05:23:44 PST 2024


================
@@ -666,7 +666,6 @@ bool LoopVectorizationLegality::canVectorizeOuterLoop() {
   // Check whether we are able to set up outer loop induction.
   if (!setupOuterLoopInductions()) {
     reportVectorizationFailure("Unsupported outer loop Phi(s)",
-                               "Unsupported outer loop Phi(s)",
----------------
hazzlim wrote:

Small thought - do we want to unify the style for all `reportVectorizationFailure` calls to either capitalize or not-capitalize the first word of the string? As currently it looks like it's a mix between both.

Otherwise LGTM, once clang-format is happy.

https://github.com/llvm/llvm-project/pull/120414


More information about the llvm-commits mailing list