Patch to improve ir-gen for constructors with default argument

expressions and a test case.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@78213 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Sema/Sema.h b/lib/Sema/Sema.h
index 6cad4b2f1442dbef92152914170e935db5bb54b0..ca7e02bcfd46689f275ba044fd0cd233d6bdcb76 100644 (file)
--- a/lib/Sema/Sema.h
+++ b/lib/Sema/Sema.h
@@ -1672,6 +1672,14 @@ public:
                                     CXXConstructorDecl *Constructor,
                                     QualType DeclInitType, 
                                     Expr **Exprs, unsigned NumExprs);
+  
+  /// BuildCXXConstructExpr - Creates a complete call to a constructor,
+  /// including handling of its default argument expressions.
+  Expr * BuildCXXConstructExpr(ASTContext &C,
+                               QualType DeclInitType,
+                               CXXConstructorDecl *Constructor,
+                               bool Elidable,
+                               Expr **Exprs, unsigned NumExprs);
 
   /// FinalizeVarWithDestructor - Prepare for calling destructor on the
   /// constructed variable.

diff --git a/lib/Sema/SemaDeclCXX.cpp b/lib/Sema/SemaDeclCXX.cpp
index 0e42a71c597aa05c8788366ad2b4cf9052a0e196..8c1b880b86912664099e197a77e2a3909e189c9a 100644 (file)
--- a/lib/Sema/SemaDeclCXX.cpp
+++ b/lib/Sema/SemaDeclCXX.cpp
@@ -2356,13 +2356,16 @@ void Sema::DefineImplicitCopyConstructor(SourceLocation CurrentLocation,
   CopyConstructor->setUsed();
 }
 
-void Sema::InitializeVarWithConstructor(VarDecl *VD, 
-                                        CXXConstructorDecl *Constructor,
-                                        QualType DeclInitType, 
-                                        Expr **Exprs, unsigned NumExprs) {
-  CXXConstructExpr *Temp = CXXConstructExpr::Create(Context, DeclInitType, 
+/// BuildCXXConstructExpr - Creates a complete call to a constructor,
+/// including handling of its default argument expressions.
+Expr *Sema::BuildCXXConstructExpr(ASTContext &C,
+                                  QualType DeclInitType, 
+                                  CXXConstructorDecl *Constructor,
+                                  bool Elidable,
+                                  Expr **Exprs, unsigned NumExprs) {
+  CXXConstructExpr *Temp = CXXConstructExpr::Create(C, DeclInitType, 
                                                     Constructor, 
-                                                    false, Exprs, NumExprs);
+                                                    Elidable, Exprs, NumExprs);
   // default arguments must be added to constructor call expression.
   FunctionDecl *FDecl = cast<FunctionDecl>(Constructor);
   unsigned NumArgsInProto = FDecl->param_size();
@@ -2379,10 +2382,19 @@ void Sema::InitializeVarWithConstructor(VarDecl *VD,
       for (unsigned I = 0, N = E->getNumTemporaries(); I != N; ++I)
         ExprTemporaries.push_back(E->getTemporary(I));
     }
-    Expr *Arg = new (Context) CXXDefaultArgExpr(FDecl->getParamDecl(j));
+    Expr *Arg = new (C) CXXDefaultArgExpr(FDecl->getParamDecl(j));
     Temp->setArg(j, Arg);
   }
-  
+  return Temp;
+}
+
+void Sema::InitializeVarWithConstructor(VarDecl *VD, 
+                                        CXXConstructorDecl *Constructor,
+                                        QualType DeclInitType, 
+                                        Expr **Exprs, unsigned NumExprs) {
+  Expr *Temp = BuildCXXConstructExpr(Context,
+                                     DeclInitType, Constructor, 
+                                     false, Exprs, NumExprs);  
   MarkDeclarationReferenced(VD->getLocation(), Constructor);
   VD->setInit(Context, Temp);
 }

diff --git a/lib/Sema/SemaExprCXX.cpp b/lib/Sema/SemaExprCXX.cpp
index 2ebd7b184daed4e010667bd7785f216b382220da..7e0422e432be14b678abde43ac062ad6ac5c97e5 100644 (file)
--- a/lib/Sema/SemaExprCXX.cpp
+++ b/lib/Sema/SemaExprCXX.cpp
@@ -911,8 +911,8 @@ Sema::PerformImplicitConversion(Expr *&From, QualType ToType,
     assert(!ToType->isReferenceType());
     
     // FIXME: Keep track of whether the copy constructor is elidable or not.
-    From = CXXConstructExpr::Create(Context, ToType, 
-                                    SCS.CopyConstructor, false, &From, 1);
+    From = BuildCXXConstructExpr(Context,
+                                 ToType, SCS.CopyConstructor, false, &From, 1);
     return false;
   }
 

diff --git a/lib/Sema/SemaInit.cpp b/lib/Sema/SemaInit.cpp
index c976c115c30008cfbf480f4ed32c6c75682a1a74..a92f7f4a5341c4a76dc49d941e54e5b29020c73a 100644 (file)
--- a/lib/Sema/SemaInit.cpp
+++ b/lib/Sema/SemaInit.cpp
@@ -177,8 +177,8 @@ bool Sema::CheckInitializerTypes(Expr *&Init, QualType &DeclType,
         if (!Constructor)
           return true;
         
-        Init = CXXConstructExpr::Create(Context, DeclType, Constructor, false, 
-                                        &Init, 1);
+        Init = BuildCXXConstructExpr(Context,
+                                     DeclType, Constructor, false, &Init, 1);
         return false;
       }
       

diff --git a/lib/Sema/SemaTemplateInstantiateExpr.cpp b/lib/Sema/SemaTemplateInstantiateExpr.cpp
index 3578dafbbb0fcee2214513ff97c55250938c6a45..3169998fb3835d02c4b1c6c0b53f77529481fa5b 100644 (file)
--- a/lib/Sema/SemaTemplateInstantiateExpr.cpp
+++ b/lib/Sema/SemaTemplateInstantiateExpr.cpp
@@ -1073,11 +1073,11 @@ TemplateExprInstantiator::VisitCXXConstructExpr(CXXConstructExpr *E) {
     Args.push_back(ArgInst.takeAs<Expr>());
   }
 
-  return SemaRef.Owned(CXXConstructExpr::Create(SemaRef.Context, T,
-                                                E->getConstructor(), 
-                                                E->isElidable(),
-                                                Args.takeAs<Expr>(), 
-                                                Args.size()));
+  return SemaRef.Owned(SemaRef.BuildCXXConstructExpr(SemaRef.Context, T,
+                                             E->getConstructor(), 
+                                             E->isElidable(),
+                                             Args.takeAs<Expr>(), 
+                                             Args.size()));
 }
 
 Sema::OwningExprResult 

diff --git a/test/CodeGenCXX/constructor-default-arg.cpp b/test/CodeGenCXX/constructor-default-arg.cpp
new file mode 100644 (file)
index 0000000..c87ec91
--- /dev/null
+++ b/test/CodeGenCXX/constructor-default-arg.cpp
@@ -0,0 +1,30 @@
+// RUN: clang-cc %s -emit-llvm -o %t &&
+// RUN: grep 'call void @_ZN1XC1ERK1Xiii' %t | count 3
+
+extern "C" int printf(...);
+
+
+struct C {
+       C() : iC(6) {}
+       int iC;
+};
+
+int foo() {
+  return 6;
+};
+
+class X { // ...
+public: 
+       X(int) {}
+       X(const X&, int i = 1, int j = 2, int k = foo()) {
+               printf("X(const X&, %d, %d, %d)\n", i, j, k);
+       }
+};
+
+int main()
+{
+       X a(1);
+       X b(a, 2);
+       X c = b;
+       X d(a, 5, 6);
+}