diff options
author | Fariborz Jahanian <fjahanian@apple.com> | 2010-10-22 22:05:03 +0000 |
---|---|---|
committer | Fariborz Jahanian <fjahanian@apple.com> | 2010-10-22 22:05:03 +0000 |
commit | 8a97005f97a2a93fc2cd942c040668c5d4df7537 (patch) | |
tree | ecc9046e5be317ce39de7135a98a833b9a93ba39 /lib/CodeGen/CGExprAgg.cpp | |
parent | cee55018570c0c46262c6e4ef7beaa707c93fc86 (diff) |
After discussion with Doug and John, I am reverting
the patch.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@117159 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/CGExprAgg.cpp')
-rw-r--r-- | lib/CodeGen/CGExprAgg.cpp | 19 |
1 files changed, 1 insertions, 18 deletions
diff --git a/lib/CodeGen/CGExprAgg.cpp b/lib/CodeGen/CGExprAgg.cpp index 99b46d41a8..31ba3373f0 100644 --- a/lib/CodeGen/CGExprAgg.cpp +++ b/lib/CodeGen/CGExprAgg.cpp @@ -147,23 +147,6 @@ public: /// represents a value lvalue, this method emits the address of the lvalue, /// then loads the result into DestPtr. void AggExprEmitter::EmitAggLoadOfLValue(const Expr *E) { - if (CGF.getContext().getLangOptions().CPlusPlus) { - if (const CXXConstructExpr *CE = Dest.getCtorExpr()) { - // Perform copy initialization of Src into Dest. - const CXXConstructorDecl *CD = CE->getConstructor(); - CXXCtorType Type = - (CE->getConstructionKind() == CXXConstructExpr::CK_Complete) - ? Ctor_Complete : Ctor_Base; - bool ForVirtualBase = - CE->getConstructionKind() == CXXConstructExpr::CK_VirtualBase; - // Call the constructor. - const Stmt * S = dyn_cast<Stmt>(E); - clang::ConstExprIterator BegExp(&S); - CGF.EmitCXXConstructorCall(CD, Type, ForVirtualBase, Dest.getAddr(), - BegExp, (BegExp+1)); - return; - } - } LValue LV = CGF.EmitLValue(E); EmitFinalDestCopy(E, LV); } @@ -221,6 +204,7 @@ void AggExprEmitter::EmitFinalDestCopy(const Expr *E, RValue Src, bool Ignore) { CGF.CGM.getLangOptions().CPlusPlus || (IgnoreResult && Ignore)) return; + // If the source is volatile, we must read from it; to do that, we need // some place to put it. Dest = CGF.CreateAggTemp(E->getType(), "agg.tmp"); @@ -238,7 +222,6 @@ void AggExprEmitter::EmitFinalDestCopy(const Expr *E, RValue Src, bool Ignore) { SizeVal); return; } - // If the result of the assignment is used, copy the LHS there also. // FIXME: Pass VolatileDest as well. I think we also need to merge volatile // from the source as well, as we can't eliminate it if either operand |