From 1c56ace4df1117d30eae6112f10f194d8f674817 Mon Sep 17 00:00:00 2001 From: Devang Patel Date: Tue, 27 May 2008 22:45:40 +0000 Subject: [PATCH] Check first member alignment and uses packed struct if required. llvm-svn: 51619 --- clang/lib/CodeGen/CodeGenTypes.cpp | 2 ++ clang/test/CodeGen/struct.c | 3 +++ 2 files changed, 5 insertions(+) diff --git a/clang/lib/CodeGen/CodeGenTypes.cpp b/clang/lib/CodeGen/CodeGenTypes.cpp index aa0d2ed0531f..5efde77a7d22 100644 --- a/clang/lib/CodeGen/CodeGenTypes.cpp +++ b/clang/lib/CodeGen/CodeGenTypes.cpp @@ -516,6 +516,8 @@ void RecordOrganizer::layoutStructFields(const ASTRecordLayout &RL) { unsigned Align = CGT.getTargetData().getABITypeAlignment(Ty) * 8; if (llvmSize % Align) packedStruct = true; + else if (offset == 0 && RL.getAlignment() % Align) + packedStruct = true; llvmSize += size; CGT.addFieldInfo(FD, LLVMFields.size()); diff --git a/clang/test/CodeGen/struct.c b/clang/test/CodeGen/struct.c index 4c88c2539fdf..f17a7af76c83 100644 --- a/clang/test/CodeGen/struct.c +++ b/clang/test/CodeGen/struct.c @@ -160,3 +160,6 @@ int f14(int i, ...) { a13 b = __builtin_va_arg(l, a13); return b.b; } + +/* Attribute packed */ +struct __attribute__((packed)) S2839 { double a[19]; signed char b; } s2839[5];