Unnamed bit-fields in a union should be laid out with a type that doesn't affect alignment.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@101673 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/CodeGen/CGRecordLayoutBuilder.cpp b/lib/CodeGen/CGRecordLayoutBuilder.cpp
index 60ef7fe..215b39e 100644
--- a/lib/CodeGen/CGRecordLayoutBuilder.cpp
+++ b/lib/CodeGen/CGRecordLayoutBuilder.cpp
@@ -332,10 +332,26 @@
if (FieldSize == 0)
return 0;
+ const llvm::Type *FieldTy;
+
+ if (!Field->getDeclName()) {
+ // This is an unnamed bit-field, which shouldn't affect alignment on the
+ // struct so we use an array of bytes for it.
+
+ FieldTy = llvm::Type::getInt8Ty(Types.getLLVMContext());
+
+ unsigned NumBytesToAppend =
+ llvm::RoundUpToAlignment(FieldSize, 8) / 8;
+
+ if (NumBytesToAppend > 1)
+ FieldTy = llvm::ArrayType::get(FieldTy, NumBytesToAppend);
+ } else
+ FieldTy = Types.ConvertTypeForMemRecursive(Field->getType());
+
// Add the bit field info.
LLVMBitFields.push_back(
LLVMBitFieldInfo(Field, ComputeBitFieldInfo(Types, Field, 0, FieldSize)));
- return Types.ConvertTypeForMemRecursive(Field->getType());
+ return FieldTy;
}
// This is a regular union field.