diff options
author | David Majnemer <david.majnemer@gmail.com> | 2015-02-03 23:04:06 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2015-02-03 23:04:06 +0000 |
commit | fd1e739a442b09a3f84eedfffda10b2263d76faa (patch) | |
tree | 5c84bb749b6375cbbf3040bbf663b5e553216633 /clang/lib/CodeGen/CGClass.cpp | |
parent | 2a96704b339699d60dd8849b4a73f4e133879354 (diff) | |
download | llvm-fd1e739a442b09a3f84eedfffda10b2263d76faa.zip llvm-fd1e739a442b09a3f84eedfffda10b2263d76faa.tar.gz llvm-fd1e739a442b09a3f84eedfffda10b2263d76faa.tar.bz2 |
CodeGen: Copy-ctorm must obey the destination's alignment requirement
We would synthesize memcpy intrinsics when emitting calls to trivial C++
constructors but we wouldn't take into account the alignment of the
destination.
llvm-svn: 228061
Diffstat (limited to 'clang/lib/CodeGen/CGClass.cpp')
-rw-r--r-- | clang/lib/CodeGen/CGClass.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/clang/lib/CodeGen/CGClass.cpp b/clang/lib/CodeGen/CGClass.cpp index 2e2b4207..524697c 100644 --- a/clang/lib/CodeGen/CGClass.cpp +++ b/clang/lib/CodeGen/CGClass.cpp @@ -1747,9 +1747,10 @@ void CodeGenFunction::EmitCXXConstructorCall(const CXXConstructorDecl *D, "trivial 1-arg ctor not a copy/move ctor"); const Expr *Arg = E->getArg(0); - QualType Ty = Arg->getType(); + QualType SrcTy = Arg->getType(); llvm::Value *Src = EmitLValue(Arg).getAddress(); - EmitAggregateCopy(This, Src, Ty); + QualType DestTy = getContext().getTypeDeclType(D->getParent()); + EmitAggregateCopyCtor(This, Src, DestTy, SrcTy); return; } @@ -1789,7 +1790,9 @@ CodeGenFunction::EmitSynthesizedCXXCopyCtorCall(const CXXConstructorDecl *D, assert(E->getNumArgs() == 1 && "unexpected argcount for trivial ctor"); assert(D->isCopyOrMoveConstructor() && "trivial 1-arg ctor not a copy/move ctor"); - EmitAggregateCopy(This, Src, E->arg_begin()->getType()); + EmitAggregateCopyCtor(This, Src, + getContext().getTypeDeclType(D->getParent()), + E->arg_begin()->getType()); return; } llvm::Value *Callee = CGM.getAddrOfCXXStructor(D, StructorType::Complete); |