diff options
author | Anders Carlsson <andersca@mac.com> | 2010-05-03 01:20:20 +0000 |
---|---|---|
committer | Anders Carlsson <andersca@mac.com> | 2010-05-03 01:20:20 +0000 |
commit | 0d7c583a4b4d0f57c6b69c66fd73babec4ef3799 (patch) | |
tree | a56db60d31a54f09308db200c191eccff9fec76c /lib/CodeGen/CGExprAgg.cpp | |
parent | 3855a07db05677883f4a9534e324da22ed1c6f9a (diff) |
Don't copy or initialize empty classes. Fixes PR7012.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@102891 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/CGExprAgg.cpp')
-rw-r--r-- | lib/CodeGen/CGExprAgg.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/lib/CodeGen/CGExprAgg.cpp b/lib/CodeGen/CGExprAgg.cpp index 84841bf5b9..96274cb6e7 100644 --- a/lib/CodeGen/CGExprAgg.cpp +++ b/lib/CodeGen/CGExprAgg.cpp @@ -738,6 +738,14 @@ void CodeGenFunction::EmitAggregateCopy(llvm::Value *DestPtr, bool isVolatile) { assert(!Ty->isAnyComplexType() && "Shouldn't happen for complex"); + // Ignore empty classes in C++. + if (getContext().getLangOptions().CPlusPlus) { + if (const RecordType *RT = Ty->getAs<RecordType>()) { + if (cast<CXXRecordDecl>(RT->getDecl())->isEmpty()) + return; + } + } + // Aggregate assignment turns into llvm.memcpy. This is almost valid per // C99 6.5.16.1p3, which states "If the value being stored in an object is // read from another object that overlaps in anyway the storage of the first |