[Xtensa] Fix LLVM argument size for bool vectors.

maciej-czekaj · maciej-czekaj · commit 3dc231944615 · 2024-01-15T10:24:34.000Z
Commit d4f4b2f [clang] Fix sizeof of boolean vector changed the result of ASTContext::getTypeSize() for bool vector to at least 8 bits. This commit fixes the Xtensa bool vector size calculation.
diff --git a/clang/lib/CodeGen/Targets/Xtensa.cpp b/clang/lib/CodeGen/Targets/Xtensa.cpp
@@ -101,8 +101,12 @@ ABIArgInfo XtensaABIInfo::classifyArgumentType(QualType Ty,
 
   // xtbool
   if (getTarget().hasFeature("bool") && Size <= 8 && Ty->isVectorType()) {
+    // The type size is rounded up to the power of two and at least 8 bits,
+    // so we need to get the "true" size from num of vector elements
+    const VectorType *VT = Ty->getAs<VectorType>();
+    unsigned NumBits = VT->getNumElements();
     llvm::Type *ResType =
-        llvm::FixedVectorType::get(llvm::Type::getInt1Ty(getVMContext()), Size);
+        llvm::FixedVectorType::get(llvm::Type::getInt1Ty(getVMContext()), NumBits);
     return ABIArgInfo::getDirect(ResType);
   }
   // Vector arguments
