[llvm] [X86] Correct an assertion message (NFC) (PR #149386)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 17 12:21:35 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-x86

Author: Trevor Gross (tgross35)

<details>
<summary>Changes</summary>

I introduced this in a78a0f8d2043 ("[X86] Align f128 and i128 to 16 bytes"). Correct the message here.

---
Full diff: https://github.com/llvm/llvm-project/pull/149386.diff


1 Files Affected:

- (modified) llvm/lib/Target/X86/X86CallingConv.cpp (+1-1) 


``````````diff
diff --git a/llvm/lib/Target/X86/X86CallingConv.cpp b/llvm/lib/Target/X86/X86CallingConv.cpp
index 82e8ce4e0bd7c..5d5a705893242 100644
--- a/llvm/lib/Target/X86/X86CallingConv.cpp
+++ b/llvm/lib/Target/X86/X86CallingConv.cpp
@@ -389,7 +389,7 @@ static bool CC_X86_32_I128_FP128(unsigned &ValNo, MVT &ValVT, MVT &LocVT,
   if (!ArgFlags.isInConsecutiveRegsLast())
     return true;
 
-  assert(PendingMembers.size() == 4 && "Should have two parts");
+  assert(PendingMembers.size() == 4 && "Should have four parts");
 
   int64_t Offset = State.AllocateStack(16, Align(16));
   PendingMembers[0].convertToMem(Offset);

``````````

</details>


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


More information about the llvm-commits mailing list