diff options
author | Anders Carlsson <andersca@mac.com> | 2007-10-31 22:04:46 +0000 |
---|---|---|
committer | Anders Carlsson <andersca@mac.com> | 2007-10-31 22:04:46 +0000 |
commit | 148fe6772733166c720e28b7bb5084af6e624b44 (patch) | |
tree | 8866e5e144f0ea0120201e3634083f16a94f34ef | |
parent | 19d74e1494fe399f0e2a94e9419c095f8214851b (diff) |
Handle function calls that return aggregate expressions.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@43581 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | CodeGen/CGExprAgg.cpp | 15 | ||||
-rw-r--r-- | test/CodeGen/struct.c | 14 |
2 files changed, 28 insertions, 1 deletions
diff --git a/CodeGen/CGExprAgg.cpp b/CodeGen/CGExprAgg.cpp index 160f491c5e..ea36225ba5 100644 --- a/CodeGen/CGExprAgg.cpp +++ b/CodeGen/CGExprAgg.cpp @@ -67,7 +67,7 @@ public: // case Expr::UnaryOperatorClass: // case Expr::ImplicitCastExprClass: // case Expr::CastExprClass: - // case Expr::CallExprClass: + void VisitCallExpr(const CallExpr *E); void VisitStmtExpr(const StmtExpr *E); void VisitBinaryOperator(const BinaryOperator *BO); void VisitBinAssign(const BinaryOperator *E); @@ -132,6 +132,19 @@ void AggExprEmitter::EmitAggLoadOfLValue(const Expr *E) { // Visitor Methods //===----------------------------------------------------------------------===// +void AggExprEmitter::VisitCallExpr(const CallExpr *E) +{ + RValue RV = CGF.EmitCallExpr(E); + assert(RV.isAggregate() && "Return value must be aggregate value!"); + + // If the result is ignored, don't copy from the value. + if (DestPtr == 0) + // FIXME: If the source is volatile, we must read from it. + return; + + EmitAggregateCopy(DestPtr, RV.getAggregateAddr(), E->getType()); +} + void AggExprEmitter::VisitStmtExpr(const StmtExpr *E) { CGF.EmitCompoundStmt(*E->getSubStmt(), true, DestPtr, VolatileDest); } diff --git a/test/CodeGen/struct.c b/test/CodeGen/struct.c index 443fc33ccb..83ac4d2195 100644 --- a/test/CodeGen/struct.c +++ b/test/CodeGen/struct.c @@ -63,3 +63,17 @@ void f4() { void f5() { (f3())->d1 = 42; } + +/* Function calls */ +typedef struct { + int location; + int length; +} range; + +extern range f6(); +void f7() +{ + range r = f6(); +} + + |