[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