Darwin x86-32: Multi-dimensional arrays were not handled correctly,
spotted by Eli!
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@71490 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/CodeGen/CGCall.cpp b/lib/CodeGen/CGCall.cpp
index 36b6b92..c46e790 100644
--- a/lib/CodeGen/CGCall.cpp
+++ b/lib/CodeGen/CGCall.cpp
@@ -166,10 +166,9 @@
return true;
QualType FT = FD->getType();
- // Arrays of empty records count as empty.
- if (const ConstantArrayType *AT = Context.getAsConstantArrayType(FT))
- if (isEmptyRecord(Context, AT->getElementType()))
- return true;
+ // Constant arrays of empty records count as empty, strip them off.
+ while (const ConstantArrayType *AT = Context.getAsConstantArrayType(FT))
+ FT = AT->getElementType();
return isEmptyRecord(Context, FT);
}
@@ -218,14 +217,18 @@
if (isEmptyField(Context, FD))
continue;
- // Treat single element arrays as the element
- if (const ConstantArrayType *AT = Context.getAsConstantArrayType(FT))
- if (AT->getSize().getZExtValue() == 1)
- FT = AT->getElementType();
-
+ // If we already found an element then this isn't a single-element
+ // struct.
if (Found)
return 0;
+ // Treat single element arrays as the element.
+ while (const ConstantArrayType *AT = Context.getAsConstantArrayType(FT)) {
+ if (AT->getSize().getZExtValue() != 1)
+ break;
+ FT = AT->getElementType();
+ }
+
if (!CodeGenFunction::hasAggregateLLVMType(FT)) {
Found = FT.getTypePtr();
} else {