From 4bb6e929027fff9fcf8d5a09770b19b1e7aa972b Mon Sep 17 00:00:00 2001
From: Anders Carlsson <andersca@mac.com>
Date: Wed, 21 Apr 2010 20:28:29 +0000
Subject: [PATCH] Diagnose access to fields with private constructors.

llvm-svn: 102025
---
 clang/include/clang/Basic/DiagnosticSemaKinds.td | 3 +++
 clang/lib/Sema/SemaAccess.cpp                    | 7 +++++++
 2 files changed, 10 insertions(+)

diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td
index a5be82df8609..eb53d6ca3847 100644
--- a/clang/include/clang/Basic/DiagnosticSemaKinds.td
+++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td
@@ -462,6 +462,9 @@ def err_access_ctor :
 def err_access_ctor_base :
     Error<"base class %0 has %select{private|protected}1 constructor">,
     NoSFINAE;
+def err_access_ctor_field :
+    Error<"field of type %1 has %select{private|protected}2 constructor">,
+    NoSFINAE;
 def err_access_ctor_vbase :
     Error<"inherited virtual base class %0 has "
     "%select{private|protected}1 constructor">,
diff --git a/clang/lib/Sema/SemaAccess.cpp b/clang/lib/Sema/SemaAccess.cpp
index 1a96c26be19c..f71d36ad7f33 100644
--- a/clang/lib/Sema/SemaAccess.cpp
+++ b/clang/lib/Sema/SemaAccess.cpp
@@ -1150,6 +1150,13 @@ Sema::AccessResult Sema::CheckConstructorAccess(SourceLocation UseLoc,
                           << Entity.getBaseSpecifier()->getType());
     break;
   }
+  
+  case InitializedEntity::EK_Member: {
+    const FieldDecl *Field = cast<FieldDecl>(Entity.getDecl());
+    AccessEntity.setDiag(PDiag(diag::err_access_ctor_field)
+                          << Field->getDeclName() << Field->getType());
+    break;
+  }
 
   }