maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Allow elision of invocations of move constructors from temporary objects. 

llvm-svn: 124455
This commit is contained in:
Douglas Gregor 2011-01-27 23:24:55 +00:00
parent e9f2f0cb0b
commit 3fb22baddb
2 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clang/lib/Sema/SemaDeclCXX.cpp
View File

@ -5551,7 +5551,7 @@ Sema::BuildCXXConstructExpr(SourceLocation ConstructLoc, QualType DeclInitType,
// can be omitted by constructing the temporary object
// directly into the target of the omitted copy/move
if (ConstructKind == CXXConstructExpr::CK_Complete &&
Constructor->isCopyConstructor() && ExprArgs.size() >= 1) {
Constructor->isCopyOrMoveConstructor() && ExprArgs.size() >= 1) {
Expr *SubExpr = ((Expr **)ExprArgs.get())[0];
Elidable = SubExpr->isTemporaryObject(Context, Constructor->getParent());
}

9
clang/test/CodeGenCXX/rvalue-references.cpp
View File

@ -62,13 +62,10 @@ C test();
// CHECK: define void @_Z15elide_copy_initv
void elide_copy_init() {
ok = false;
// FIXME: We're doing an extra move here, when we shouldn't be!
// CHECK: call void @_Z4testv(%class.C* sret %ref.tmp)
// CHECK: call void @_ZN1CC1EOS_(%class.C* %a, %class.C* %ref.tmp)
// CHECK: call void @_ZN1CD1Ev(%class.C* %ref.tmp)
// CHECK: call void @_Z4testv
C a = test();
// CHECK: call void @_ZN1CD1Ev(%class.C* %a)
// CHECK: ret void
// CHECK-NEXT: call void @_ZN1CD1Ev
// CHECK-NEXT: ret void
}
// CHECK: define void @_Z16test_move_returnv