From ebe772e2f2aadcfa8d73e34c7c59dd601f6641d8 Mon Sep 17 00:00:00 2001
From: Fariborz Jahanian <fjahanian@apple.com>
Date: Fri, 26 Jun 2009 16:08:57 +0000
Subject: [PATCH] Minor change per Doug's comments. - Fariborz

llvm-svn: 74287
---
 clang/lib/Sema/SemaDeclCXX.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp
index f00acdac4117..016cedfe0e37 100644
--- a/clang/lib/Sema/SemaDeclCXX.cpp
+++ b/clang/lib/Sema/SemaDeclCXX.cpp
@@ -1940,7 +1940,9 @@ void Sema::DefineImplicitDefaultConstructor(SourceLocation CurrentLocation,
     }
   }
   if (!err)
-    Constructor->setUsed();  
+    Constructor->setUsed();
+  else
+    Constructor->setInvalidDecl();
 }
 
 void Sema::DefineImplicitOverloadedAssign(SourceLocation CurrentLocation,
@@ -1954,7 +1956,7 @@ void Sema::DefineImplicitOverloadedAssign(SourceLocation CurrentLocation,
     = cast<CXXRecordDecl>(MethodDecl->getDeclContext());
   assert(ClassDecl && "DefineImplicitOverloadedAssign - invalid constructor");
   
-  // C++[class.copy] p210
+  // C++[class.copy] p12
   // Before the implicitly-declared copy assignment operator for a class is
   // implicitly defined, all implicitly-declared copy assignment operators
   // for its direct base classes and its nonstatic data members shall have