diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2011-11-18 00:28:11 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2011-11-18 00:28:11 +0000 |
| commit | 9f061a37bb6e27314cb95c1750704d68403971f9 (patch) | |
| tree | 299dc873f15542f0db309d60c7063e07efdab0fa | |
| parent | 841dd8861ad46c7efc00e9e8121f4af204c36e96 (diff) | |
| download | bcm5719-llvm-9f061a37bb6e27314cb95c1750704d68403971f9.tar.gz bcm5719-llvm-9f061a37bb6e27314cb95c1750704d68403971f9.zip | |
Ignore empty unions in argument lowering on x86-32. From gcc struct layout tests.
llvm-svn: 144944
| -rw-r--r-- | clang/lib/CodeGen/TargetInfo.cpp | 4 | ||||
| -rw-r--r-- | clang/test/CodeGen/x86_32-arguments-darwin.c | 4 |
2 files changed, 6 insertions, 2 deletions
diff --git a/clang/lib/CodeGen/TargetInfo.cpp b/clang/lib/CodeGen/TargetInfo.cpp index c3d5ee19d68..b23ae304a82 100644 --- a/clang/lib/CodeGen/TargetInfo.cpp +++ b/clang/lib/CodeGen/TargetInfo.cpp @@ -700,8 +700,8 @@ ABIArgInfo X86_32ABIInfo::classifyArgumentType(QualType Ty) const { return getIndirectResult(Ty); } - // Ignore empty structs. - if (Ty->isStructureType() && getContext().getTypeSize(Ty) == 0) + // Ignore empty structs/unions. + if (Ty->isRecordType() && getContext().getTypeSize(Ty) == 0) return ABIArgInfo::getIgnore(); // Expand small (<= 128-bit) record types when we know that the stack layout diff --git a/clang/test/CodeGen/x86_32-arguments-darwin.c b/clang/test/CodeGen/x86_32-arguments-darwin.c index c3af7fbc047..b31262fc96c 100644 --- a/clang/test/CodeGen/x86_32-arguments-darwin.c +++ b/clang/test/CodeGen/x86_32-arguments-darwin.c @@ -280,3 +280,7 @@ void f56(char a0, struct s56_0 a1, // CHECK: call void @f57( struct s57 { _Complex int x; }; void f57(struct s57 x) {} void f57a(void) { f57((struct s57){1}); } + +union u58 {}; +void f58(union u58 x) {} +// CHECK: define void @f58() |
