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

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 18 08:19:36 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)",
----------------
david-arm wrote:

Yeah it's a good point and I thought about that too. It doesn't look like there is always consistency throughout for all of the reports. I agree it would be good to standardise them. Not sure if @fhahn has any thoughts?

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


More information about the llvm-commits mailing list