[llvm] [VPlan] Fix typo in assertion. NFC (PR #137009)

Luke Lau via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 24 01:07:40 PDT 2025


https://github.com/lukel97 updated https://github.com/llvm/llvm-project/pull/137009

>From 6d8209a24f260a315319a3d7183b884836ff8775 Mon Sep 17 00:00:00 2001
From: Luke Lau <luke at igalia.com>
Date: Thu, 24 Apr 2025 00:09:31 +0800
Subject: [PATCH 1/2] [VPlan] Fix typo in assertion. NFC

---
 llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h b/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h
index f38eb3cc8d43f..0d2d8315240cf 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h
+++ b/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h
@@ -221,8 +221,9 @@ struct Recipe_match {
     if ((!matchRecipeAndOpcode<RecipeTys>(R) && ...))
       return false;
 
-    assert(R->getNumOperands() == std::tuple_size<Ops_t>::value &&
-           "recipe with matched opcode the expected number of operands");
+    assert(
+        R->getNumOperands() == std::tuple_size<Ops_t>::value &&
+        "recipe with matched opcode without the expected number of operands");
 
     auto IdxSeq = std::make_index_sequence<std::tuple_size<Ops_t>::value>();
     if (all_of_tuple_elements(IdxSeq, [R](auto Op, unsigned Idx) {

>From 5362b2f2467e392f61499c31e8e6be1ebec92736 Mon Sep 17 00:00:00 2001
From: Luke Lau <luke at igalia.com>
Date: Thu, 24 Apr 2025 16:07:20 +0800
Subject: [PATCH 2/2] Update comment

---
 llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h b/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h
index 0d2d8315240cf..58865c296ed8a 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h
+++ b/llvm/lib/Transforms/Vectorize/VPlanPatternMatch.h
@@ -221,9 +221,9 @@ struct Recipe_match {
     if ((!matchRecipeAndOpcode<RecipeTys>(R) && ...))
       return false;
 
-    assert(
-        R->getNumOperands() == std::tuple_size<Ops_t>::value &&
-        "recipe with matched opcode without the expected number of operands");
+    assert(R->getNumOperands() == std::tuple_size<Ops_t>::value &&
+           "recipe with matched opcode does not have the expected number of "
+           "operands");
 
     auto IdxSeq = std::make_index_sequence<std::tuple_size<Ops_t>::value>();
     if (all_of_tuple_elements(IdxSeq, [R](auto Op, unsigned Idx) {



More information about the llvm-commits mailing list