SemaObjCProperty.cpp revision 5bf0e3522fac400cf7bf49a775554f0fc942b336
19d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//===--- SemaObjCProperty.cpp - Semantic Analysis for ObjC @property ------===//
29d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//
39d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//                     The LLVM Compiler Infrastructure
49d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//
59d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// This file is distributed under the University of Illinois Open Source
69d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// License. See LICENSE.TXT for details.
79d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//
89d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//===----------------------------------------------------------------------===//
99d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//
109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//  This file implements semantic analysis for Objective C @property and
119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//  @synthesize declarations.
129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//
139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//===----------------------------------------------------------------------===//
149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
152d88708cbe4e4ec5e04e4acb6bd7f5be68557379John McCall#include "clang/Sema/SemaInternal.h"
16e737f5041a36d0befb39ffeed8d50ba15916d3daDouglas Gregor#include "clang/Sema/Initialization.h"
177cd088e519d7e6caa4c4c12db52e0e4ae35d25c2John McCall#include "clang/AST/DeclObjC.h"
1817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian#include "clang/AST/ExprObjC.h"
1957e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian#include "clang/AST/ExprCXX.h"
20c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis#include "clang/AST/ASTMutationListener.h"
21edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian#include "clang/Lex/Lexer.h"
22edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian#include "clang/Basic/SourceManager.h"
2350df6ae41f232612e5e88b19e0db9900d08d2f6cJohn McCall#include "llvm/ADT/DenseSet.h"
248fe83e1df954d72c0f4ffc15d20a5222ec151c21Benjamin Kramer#include "llvm/ADT/SmallString.h"
259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekusing namespace clang;
279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
2828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===//
2928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek// Grammar actions.
3028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===//
3128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
32265941bc308d65cc270d5c4de5806f37ce405606John McCall/// getImpliedARCOwnership - Given a set of property attributes and a
33265941bc308d65cc270d5c4de5806f37ce405606John McCall/// type, infer an expected lifetime.  The type's ownership qualification
34265941bc308d65cc270d5c4de5806f37ce405606John McCall/// is not considered.
35265941bc308d65cc270d5c4de5806f37ce405606John McCall///
36265941bc308d65cc270d5c4de5806f37ce405606John McCall/// Returns OCL_None if the attributes as stated do not imply an ownership.
37265941bc308d65cc270d5c4de5806f37ce405606John McCall/// Never returns OCL_Autoreleasing.
38265941bc308d65cc270d5c4de5806f37ce405606John McCallstatic Qualifiers::ObjCLifetime getImpliedARCOwnership(
39265941bc308d65cc270d5c4de5806f37ce405606John McCall                               ObjCPropertyDecl::PropertyAttributeKind attrs,
40265941bc308d65cc270d5c4de5806f37ce405606John McCall                                                QualType type) {
41265941bc308d65cc270d5c4de5806f37ce405606John McCall  // retain, strong, copy, weak, and unsafe_unretained are only legal
42265941bc308d65cc270d5c4de5806f37ce405606John McCall  // on properties of retainable pointer type.
43265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (attrs & (ObjCPropertyDecl::OBJC_PR_retain |
44265941bc308d65cc270d5c4de5806f37ce405606John McCall               ObjCPropertyDecl::OBJC_PR_strong |
45265941bc308d65cc270d5c4de5806f37ce405606John McCall               ObjCPropertyDecl::OBJC_PR_copy)) {
465fa065b55bd622f20656e0de3b1f122951146784Fariborz Jahanian    return type->getObjCARCImplicitLifetime();
47265941bc308d65cc270d5c4de5806f37ce405606John McCall  } else if (attrs & ObjCPropertyDecl::OBJC_PR_weak) {
48265941bc308d65cc270d5c4de5806f37ce405606John McCall    return Qualifiers::OCL_Weak;
49265941bc308d65cc270d5c4de5806f37ce405606John McCall  } else if (attrs & ObjCPropertyDecl::OBJC_PR_unsafe_unretained) {
50265941bc308d65cc270d5c4de5806f37ce405606John McCall    return Qualifiers::OCL_ExplicitNone;
51265941bc308d65cc270d5c4de5806f37ce405606John McCall  }
52265941bc308d65cc270d5c4de5806f37ce405606John McCall
53265941bc308d65cc270d5c4de5806f37ce405606John McCall  // assign can appear on other types, so we have to check the
54265941bc308d65cc270d5c4de5806f37ce405606John McCall  // property type.
55265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (attrs & ObjCPropertyDecl::OBJC_PR_assign &&
56265941bc308d65cc270d5c4de5806f37ce405606John McCall      type->isObjCRetainableType()) {
57265941bc308d65cc270d5c4de5806f37ce405606John McCall    return Qualifiers::OCL_ExplicitNone;
58265941bc308d65cc270d5c4de5806f37ce405606John McCall  }
59265941bc308d65cc270d5c4de5806f37ce405606John McCall
60265941bc308d65cc270d5c4de5806f37ce405606John McCall  return Qualifiers::OCL_None;
61265941bc308d65cc270d5c4de5806f37ce405606John McCall}
62265941bc308d65cc270d5c4de5806f37ce405606John McCall
63f85e193739c953358c865005855253af4f68a497John McCall/// Check the internal consistency of a property declaration.
64f85e193739c953358c865005855253af4f68a497John McCallstatic void checkARCPropertyDecl(Sema &S, ObjCPropertyDecl *property) {
65f85e193739c953358c865005855253af4f68a497John McCall  if (property->isInvalidDecl()) return;
66f85e193739c953358c865005855253af4f68a497John McCall
67f85e193739c953358c865005855253af4f68a497John McCall  ObjCPropertyDecl::PropertyAttributeKind propertyKind
68f85e193739c953358c865005855253af4f68a497John McCall    = property->getPropertyAttributes();
69f85e193739c953358c865005855253af4f68a497John McCall  Qualifiers::ObjCLifetime propertyLifetime
70f85e193739c953358c865005855253af4f68a497John McCall    = property->getType().getObjCLifetime();
71f85e193739c953358c865005855253af4f68a497John McCall
72f85e193739c953358c865005855253af4f68a497John McCall  // Nothing to do if we don't have a lifetime.
73f85e193739c953358c865005855253af4f68a497John McCall  if (propertyLifetime == Qualifiers::OCL_None) return;
74f85e193739c953358c865005855253af4f68a497John McCall
75265941bc308d65cc270d5c4de5806f37ce405606John McCall  Qualifiers::ObjCLifetime expectedLifetime
76265941bc308d65cc270d5c4de5806f37ce405606John McCall    = getImpliedARCOwnership(propertyKind, property->getType());
77265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (!expectedLifetime) {
78f85e193739c953358c865005855253af4f68a497John McCall    // We have a lifetime qualifier but no dominating property
79265941bc308d65cc270d5c4de5806f37ce405606John McCall    // attribute.  That's okay, but restore reasonable invariants by
80265941bc308d65cc270d5c4de5806f37ce405606John McCall    // setting the property attribute according to the lifetime
81265941bc308d65cc270d5c4de5806f37ce405606John McCall    // qualifier.
82265941bc308d65cc270d5c4de5806f37ce405606John McCall    ObjCPropertyDecl::PropertyAttributeKind attr;
83265941bc308d65cc270d5c4de5806f37ce405606John McCall    if (propertyLifetime == Qualifiers::OCL_Strong) {
84265941bc308d65cc270d5c4de5806f37ce405606John McCall      attr = ObjCPropertyDecl::OBJC_PR_strong;
85265941bc308d65cc270d5c4de5806f37ce405606John McCall    } else if (propertyLifetime == Qualifiers::OCL_Weak) {
86265941bc308d65cc270d5c4de5806f37ce405606John McCall      attr = ObjCPropertyDecl::OBJC_PR_weak;
87265941bc308d65cc270d5c4de5806f37ce405606John McCall    } else {
88265941bc308d65cc270d5c4de5806f37ce405606John McCall      assert(propertyLifetime == Qualifiers::OCL_ExplicitNone);
89265941bc308d65cc270d5c4de5806f37ce405606John McCall      attr = ObjCPropertyDecl::OBJC_PR_unsafe_unretained;
90265941bc308d65cc270d5c4de5806f37ce405606John McCall    }
91265941bc308d65cc270d5c4de5806f37ce405606John McCall    property->setPropertyAttributes(attr);
92f85e193739c953358c865005855253af4f68a497John McCall    return;
93f85e193739c953358c865005855253af4f68a497John McCall  }
94f85e193739c953358c865005855253af4f68a497John McCall
95f85e193739c953358c865005855253af4f68a497John McCall  if (propertyLifetime == expectedLifetime) return;
96f85e193739c953358c865005855253af4f68a497John McCall
97f85e193739c953358c865005855253af4f68a497John McCall  property->setInvalidDecl();
98f85e193739c953358c865005855253af4f68a497John McCall  S.Diag(property->getLocation(),
99b8b0313e84700b5c6d597b3be4de41c97b7550f1Argyrios Kyrtzidis         diag::err_arc_inconsistent_property_ownership)
100f85e193739c953358c865005855253af4f68a497John McCall    << property->getDeclName()
101265941bc308d65cc270d5c4de5806f37ce405606John McCall    << expectedLifetime
102f85e193739c953358c865005855253af4f68a497John McCall    << propertyLifetime;
103f85e193739c953358c865005855253af4f68a497John McCall}
104f85e193739c953358c865005855253af4f68a497John McCall
105d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallDecl *Sema::ActOnProperty(Scope *S, SourceLocation AtLoc,
10677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                          SourceLocation LParenLoc,
107d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          FieldDeclarator &FD,
108d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          ObjCDeclSpec &ODS,
109d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          Selector GetterSel,
110d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          Selector SetterSel,
111d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          bool *isOverridingProperty,
1124a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                          tok::ObjCKeywordKind MethodImplKind,
1134a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                          DeclContext *lexicalDC) {
11428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  unsigned Attributes = ODS.getPropertyAttributes();
115f85e193739c953358c865005855253af4f68a497John McCall  TypeSourceInfo *TSI = GetTypeForDeclarator(FD.D, S);
116f85e193739c953358c865005855253af4f68a497John McCall  QualType T = TSI->getType();
1174e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if ((getLangOpts().getGC() != LangOptions::NonGC &&
118f85e193739c953358c865005855253af4f68a497John McCall       T.isObjCGCWeak()) ||
1194e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      (getLangOpts().ObjCAutoRefCount &&
120f85e193739c953358c865005855253af4f68a497John McCall       T.getObjCLifetime() == Qualifiers::OCL_Weak))
121f85e193739c953358c865005855253af4f68a497John McCall    Attributes |= ObjCDeclSpec::DQ_PR_weak;
122f85e193739c953358c865005855253af4f68a497John McCall
12328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  bool isReadWrite = ((Attributes & ObjCDeclSpec::DQ_PR_readwrite) ||
12428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                      // default is readwrite!
12528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                      !(Attributes & ObjCDeclSpec::DQ_PR_readonly));
12628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // property is defaulted to 'assign' if it is readwrite and is
12728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // not retain or copy
12828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  bool isAssign = ((Attributes & ObjCDeclSpec::DQ_PR_assign) ||
12928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                   (isReadWrite &&
13028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                    !(Attributes & ObjCDeclSpec::DQ_PR_retain) &&
131f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_strong) &&
132f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_copy) &&
133f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) &&
134f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_weak)));
13514086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian
136e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Proceed with constructing the ObjCPropertDecls.
137a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian  ObjCContainerDecl *ClassDecl = cast<ObjCContainerDecl>(CurContext);
138e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
13928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (ObjCCategoryDecl *CDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl))
140ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian    if (CDecl->IsClassExtension()) {
14177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian      Decl *Res = HandlePropertyInClassExtension(S, AtLoc, LParenLoc,
142ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           FD, GetterSel, SetterSel,
143ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           isAssign, isReadWrite,
144ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           Attributes,
145dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                           ODS.getPropertyAttributes(),
146ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           isOverridingProperty, TSI,
147ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           MethodImplKind);
148f85e193739c953358c865005855253af4f68a497John McCall      if (Res) {
149ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian        CheckObjCPropertyAttributes(Res, AtLoc, Attributes);
1504e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        if (getLangOpts().ObjCAutoRefCount)
151f85e193739c953358c865005855253af4f68a497John McCall          checkARCPropertyDecl(*this, cast<ObjCPropertyDecl>(Res));
152f85e193739c953358c865005855253af4f68a497John McCall      }
153ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian      return Res;
154ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian    }
155ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian
15677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian  ObjCPropertyDecl *Res = CreatePropertyDecl(S, ClassDecl, AtLoc, LParenLoc, FD,
157f85e193739c953358c865005855253af4f68a497John McCall                                             GetterSel, SetterSel,
158f85e193739c953358c865005855253af4f68a497John McCall                                             isAssign, isReadWrite,
159dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                             Attributes,
160dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                             ODS.getPropertyAttributes(),
161dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                             TSI, MethodImplKind);
1624a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek  if (lexicalDC)
1634a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek    Res->setLexicalDeclContext(lexicalDC);
1644a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek
165842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  // Validate the attributes on the @property.
166842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  CheckObjCPropertyAttributes(Res, AtLoc, Attributes);
167f85e193739c953358c865005855253af4f68a497John McCall
1684e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().ObjCAutoRefCount)
169f85e193739c953358c865005855253af4f68a497John McCall    checkARCPropertyDecl(*this, Res);
170f85e193739c953358c865005855253af4f68a497John McCall
171842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  return Res;
172e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek}
173e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
174b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidisstatic ObjCPropertyDecl::PropertyAttributeKind
175b98ffded10453d80369951f33f3892f35d747c95Argyrios KyrtzidismakePropertyAttributesAsWritten(unsigned Attributes) {
176b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  unsigned attributesAsWritten = 0;
177b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
178b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_readonly;
179b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_readwrite)
180b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_readwrite;
181b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_getter)
182b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_getter;
183b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_setter)
184b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_setter;
185b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_assign)
186b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_assign;
187b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_retain)
188b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_retain;
189b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_strong)
190b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_strong;
191b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_weak)
192b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_weak;
193b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_copy)
194b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_copy;
195b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
196b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_unsafe_unretained;
197b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)
198b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_nonatomic;
199b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_atomic)
200b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_atomic;
201b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis
202b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  return (ObjCPropertyDecl::PropertyAttributeKind)attributesAsWritten;
203b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis}
204b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis
2055bf0e3522fac400cf7bf49a775554f0fc942b336Fariborz Jahanianstatic bool LocPropertyAttribute( ASTContext &Context, const char *attrName,
206edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian                                 SourceLocation LParenLoc, SourceLocation &Loc) {
207edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  if (LParenLoc.isMacroID())
208edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    return false;
209edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian
210edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  SourceManager &SM = Context.getSourceManager();
211edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  std::pair<FileID, unsigned> locInfo = SM.getDecomposedLoc(LParenLoc);
212edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  // Try to load the file buffer.
213edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  bool invalidTemp = false;
214edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  StringRef file = SM.getBufferData(locInfo.first, &invalidTemp);
215edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  if (invalidTemp)
216edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    return false;
217edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  const char *tokenBegin = file.data() + locInfo.second;
218edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian
219edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  // Lex from the start of the given location.
220edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  Lexer lexer(SM.getLocForStartOfFile(locInfo.first),
221edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian              Context.getLangOpts(),
222edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian              file.begin(), tokenBegin, file.end());
223edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  Token Tok;
224edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  do {
225edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    lexer.LexFromRawLexer(Tok);
226edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    if (Tok.is(tok::raw_identifier) &&
227edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        StringRef(Tok.getRawIdentifierData(), Tok.getLength()) == attrName) {
228edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      Loc = Tok.getLocation();
229edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      return true;
230edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    }
231edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  } while (Tok.isNot(tok::r_paren));
232edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  return false;
233edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian
2342b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian}
2352b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian
236d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallDecl *
237a28948f34817476d02412fa204cae038e228c827Fariborz JahanianSema::HandlePropertyInClassExtension(Scope *S,
23877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                     SourceLocation AtLoc,
23977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                     SourceLocation LParenLoc,
24077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                     FieldDeclarator &FD,
241e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     Selector GetterSel, Selector SetterSel,
242e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     const bool isAssign,
243e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     const bool isReadWrite,
244e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     const unsigned Attributes,
245dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                     const unsigned AttributesAsWritten,
246e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     bool *isOverridingProperty,
24783a230c83a54190366138c1a4f4310ef838b88fcJohn McCall                                     TypeSourceInfo *T,
248e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     tok::ObjCKeywordKind MethodImplKind) {
24958a764940df0cd41767367eb2f4fced6f374176bFariborz Jahanian  ObjCCategoryDecl *CDecl = cast<ObjCCategoryDecl>(CurContext);
250e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Diagnose if this property is already in continuation class.
251a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian  DeclContext *DC = CurContext;
252e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  IdentifierInfo *PropertyId = FD.D.getIdentifier();
2532a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian  ObjCInterfaceDecl *CCPrimary = CDecl->getClassInterface();
2542a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian
2552a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian  if (CCPrimary)
2562a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    // Check for duplicate declaration of this property in current and
2572a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    // other class extensions.
2582a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    for (const ObjCCategoryDecl *ClsExtDecl =
2592a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian         CCPrimary->getFirstClassExtension();
2602a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian         ClsExtDecl; ClsExtDecl = ClsExtDecl->getNextClassExtension()) {
2612a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian      if (ObjCPropertyDecl *prevDecl =
2622a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian          ObjCPropertyDecl::findPropertyDecl(ClsExtDecl, PropertyId)) {
2632a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian        Diag(AtLoc, diag::err_duplicate_property);
2642a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian        Diag(prevDecl->getLocation(), diag::note_property_declare);
2652a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian        return 0;
2662a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian      }
2672a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    }
2682a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian
269e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Create a new ObjCPropertyDecl with the DeclContext being
270e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // the class extension.
27188f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  // FIXME. We should really be using CreatePropertyDecl for this.
272e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  ObjCPropertyDecl *PDecl =
273e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    ObjCPropertyDecl::Create(Context, DC, FD.D.getIdentifierLoc(),
27477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                             PropertyId, AtLoc, LParenLoc, T);
275b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  PDecl->setPropertyAttributesAsWritten(
276dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                          makePropertyAttributesAsWritten(AttributesAsWritten));
27722f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian  if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
27822f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly);
27922f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian  if (Attributes & ObjCDeclSpec::DQ_PR_readwrite)
28022f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite);
28188f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  // Set setter/getter selector name. Needed later.
28288f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  PDecl->setGetterName(GetterSel);
28388f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  PDecl->setSetterName(SetterSel);
28491ae6b43b3a294243698cdb544dd7a18efa4c7a8Douglas Gregor  ProcessDeclAttributes(S, PDecl, FD.D);
285e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  DC->addDecl(PDecl);
286e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
287e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // We need to look in the @interface to see if the @property was
288e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // already declared.
289e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  if (!CCPrimary) {
290e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    Diag(CDecl->getLocation(), diag::err_continuation_class);
291e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    *isOverridingProperty = true;
292d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall    return 0;
293e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  }
294e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
295e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Find the property in continuation class's primary class only.
296e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  ObjCPropertyDecl *PIDecl =
297e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    CCPrimary->FindPropertyVisibleInPrimaryClass(PropertyId);
298e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
299e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  if (!PIDecl) {
300e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // No matching property found in the primary class. Just fall thru
301e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // and add property to continuation class's primary class.
302d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    ObjCPropertyDecl *PrimaryPDecl =
30377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian      CreatePropertyDecl(S, CCPrimary, AtLoc, LParenLoc,
304e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                         FD, GetterSel, SetterSel, isAssign, isReadWrite,
305dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                         Attributes,AttributesAsWritten, T, MethodImplKind, DC);
306e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
307e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // A case of continuation class adding a new property in the class. This
308e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // is not what it was meant for. However, gcc supports it and so should we.
309e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // Make sure setter/getters are declared here.
310d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    ProcessPropertyDecl(PrimaryPDecl, CCPrimary, /* redeclaredProperty = */ 0,
311a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek                        /* lexicalDC = */ CDecl);
312d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    PDecl->setGetterMethodDecl(PrimaryPDecl->getGetterMethodDecl());
313d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    PDecl->setSetterMethodDecl(PrimaryPDecl->getSetterMethodDecl());
314c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis    if (ASTMutationListener *L = Context.getASTMutationListener())
315d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis      L->AddedObjCPropertyInClassExtension(PrimaryPDecl, /*OrigProp=*/0, CDecl);
316d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    return PrimaryPDecl;
317e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  }
318e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian  if (!Context.hasSameType(PIDecl->getType(), PDecl->getType())) {
319e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    bool IncompatibleObjC = false;
320e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    QualType ConvertedType;
321ff2a0ecae1d7692dbb63aa540dba41a8c21d7ee4Fariborz Jahanian    // Relax the strict type matching for property type in continuation class.
322ff2a0ecae1d7692dbb63aa540dba41a8c21d7ee4Fariborz Jahanian    // Allow property object type of continuation class to be different as long
323ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // as it narrows the object type in its primary class property. Note that
324ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // this conversion is safe only because the wider type is for a 'readonly'
325ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // property in primary class and 'narrowed' type for a 'readwrite' property
326ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // in continuation class.
327e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    if (!isa<ObjCObjectPointerType>(PIDecl->getType()) ||
328e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian        !isa<ObjCObjectPointerType>(PDecl->getType()) ||
329e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian        (!isObjCPointerConversion(PDecl->getType(), PIDecl->getType(),
330e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian                                  ConvertedType, IncompatibleObjC))
331e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian        || IncompatibleObjC) {
332e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian      Diag(AtLoc,
333e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian          diag::err_type_mismatch_continuation_class) << PDecl->getType();
334e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian      Diag(PIDecl->getLocation(), diag::note_property_declare);
335e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    }
336a4b984d8396724a8e1137c22186c558c0cb0bf3fFariborz Jahanian  }
337a4b984d8396724a8e1137c22186c558c0cb0bf3fFariborz Jahanian
338e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // The property 'PIDecl's readonly attribute will be over-ridden
339e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // with continuation class's readwrite property attribute!
34080aa1cd7973561889e51c1c152c8990a8de9c953Fariborz Jahanian  unsigned PIkind = PIDecl->getPropertyAttributesAsWritten();
341e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  if (isReadWrite && (PIkind & ObjCPropertyDecl::OBJC_PR_readonly)) {
342e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    unsigned retainCopyNonatomic =
343e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    (ObjCPropertyDecl::OBJC_PR_retain |
344f85e193739c953358c865005855253af4f68a497John McCall     ObjCPropertyDecl::OBJC_PR_strong |
345e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek     ObjCPropertyDecl::OBJC_PR_copy |
346e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek     ObjCPropertyDecl::OBJC_PR_nonatomic);
347e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    if ((Attributes & retainCopyNonatomic) !=
348e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek        (PIkind & retainCopyNonatomic)) {
349e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      Diag(AtLoc, diag::warn_property_attr_mismatch);
350e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      Diag(PIDecl->getLocation(), diag::note_property_declare);
351e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    }
3529944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek    DeclContext *DC = cast<DeclContext>(CCPrimary);
3539944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek    if (!ObjCPropertyDecl::findPropertyDecl(DC,
3549944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek                                 PIDecl->getDeclName().getAsIdentifierInfo())) {
355e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // Protocol is not in the primary class. Must build one for it.
356e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      ObjCDeclSpec ProtocolPropertyODS;
357e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // FIXME. Assuming that ObjCDeclSpec::ObjCPropertyAttributeKind
358e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // and ObjCPropertyDecl::PropertyAttributeKind have identical
359e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // values.  Should consolidate both into one enum type.
360e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      ProtocolPropertyODS.
361e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      setPropertyAttributes((ObjCDeclSpec::ObjCPropertyAttributeKind)
362e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                            PIkind);
363a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian      // Must re-establish the context from class extension to primary
364a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian      // class context.
3657939418cbdf1b610a69e6adf29cd7dc8d0e9eaf8Fariborz Jahanian      ContextRAII SavedContext(*this, CCPrimary);
3667939418cbdf1b610a69e6adf29cd7dc8d0e9eaf8Fariborz Jahanian
367d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      Decl *ProtocolPtrTy =
36877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        ActOnProperty(S, AtLoc, LParenLoc, FD, ProtocolPropertyODS,
369e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                      PIDecl->getGetterName(),
370e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                      PIDecl->getSetterName(),
371a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian                      isOverridingProperty,
3724a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                      MethodImplKind,
3734a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                      /* lexicalDC = */ CDecl);
374d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      PIDecl = cast<ObjCPropertyDecl>(ProtocolPtrTy);
37528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
376e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    PIDecl->makeitReadWriteAttribute();
377e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_retain)
378e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain);
379f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong)
380f85e193739c953358c865005855253af4f68a497John McCall      PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
381e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_copy)
382e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy);
383e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    PIDecl->setSetterName(SetterSel);
384e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  } else {
385788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // Tailor the diagnostics for the common case where a readwrite
386788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // property is declared both in the @interface and the continuation.
387788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // This is a common error where the user often intended the original
388788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // declaration to be readonly.
389788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    unsigned diag =
390788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      (Attributes & ObjCDeclSpec::DQ_PR_readwrite) &&
391788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      (PIkind & ObjCPropertyDecl::OBJC_PR_readwrite)
392788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      ? diag::err_use_continuation_class_redeclaration_readwrite
393788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      : diag::err_use_continuation_class;
394788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    Diag(AtLoc, diag)
395e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      << CCPrimary->getDeclName();
396e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    Diag(PIDecl->getLocation(), diag::note_property_declare);
397e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  }
398e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  *isOverridingProperty = true;
399e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Make sure setter decl is synthesized, and added to primary class's list.
4008254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek  ProcessPropertyDecl(PIDecl, CCPrimary, PDecl, CDecl);
401d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis  PDecl->setGetterMethodDecl(PIDecl->getGetterMethodDecl());
402d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis  PDecl->setSetterMethodDecl(PIDecl->getSetterMethodDecl());
403c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis  if (ASTMutationListener *L = Context.getASTMutationListener())
404c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis    L->AddedObjCPropertyInClassExtension(PDecl, PIDecl, CDecl);
405d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall  return 0;
406e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek}
407e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
408e3d67bcc7b89b335dbcd097a299658057ea31097Ted KremenekObjCPropertyDecl *Sema::CreatePropertyDecl(Scope *S,
409e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           ObjCContainerDecl *CDecl,
410e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           SourceLocation AtLoc,
41177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                           SourceLocation LParenLoc,
412e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           FieldDeclarator &FD,
413e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           Selector GetterSel,
414e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           Selector SetterSel,
415e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           const bool isAssign,
416e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           const bool isReadWrite,
417e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           const unsigned Attributes,
418dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                           const unsigned AttributesAsWritten,
41983a230c83a54190366138c1a4f4310ef838b88fcJohn McCall                                           TypeSourceInfo *TInfo,
42023173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek                                           tok::ObjCKeywordKind MethodImplKind,
42123173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek                                           DeclContext *lexicalDC){
422e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  IdentifierInfo *PropertyId = FD.D.getIdentifier();
42383a230c83a54190366138c1a4f4310ef838b88fcJohn McCall  QualType T = TInfo->getType();
42428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
42528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Issue a warning if property is 'assign' as default and its object, which is
42628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // gc'able conforms to NSCopying protocol
4274e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().getGC() != LangOptions::NonGC &&
42828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      isAssign && !(Attributes & ObjCDeclSpec::DQ_PR_assign))
429c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall    if (const ObjCObjectPointerType *ObjPtrTy =
430c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall          T->getAs<ObjCObjectPointerType>()) {
431c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall      ObjCInterfaceDecl *IDecl = ObjPtrTy->getObjectType()->getInterface();
432c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall      if (IDecl)
433c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall        if (ObjCProtocolDecl* PNSCopying =
434c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall            LookupProtocol(&Context.Idents.get("NSCopying"), AtLoc))
435c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall          if (IDecl->ClassImplementsProtocol(PNSCopying, true))
436c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall            Diag(AtLoc, diag::warn_implements_nscopying) << PropertyId;
43728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
438c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall  if (T->isObjCObjectType())
43928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(FD.D.getIdentifierLoc(), diag::err_statically_allocated_object);
44028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
441e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  DeclContext *DC = cast<DeclContext>(CDecl);
44228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyDecl *PDecl = ObjCPropertyDecl::Create(Context, DC,
44328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                                                     FD.D.getIdentifierLoc(),
44477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                                     PropertyId, AtLoc, LParenLoc, TInfo);
445e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
4469f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek  if (ObjCPropertyDecl *prevDecl =
4479f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek        ObjCPropertyDecl::findPropertyDecl(DC, PropertyId)) {
44828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(PDecl->getLocation(), diag::err_duplicate_property);
449894ae6ad081465a3b168db7e60ab3f94e3730b57Ted Kremenek    Diag(prevDecl->getLocation(), diag::note_property_declare);
45028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setInvalidDecl();
45128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
45223173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek  else {
45328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    DC->addDecl(PDecl);
45423173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek    if (lexicalDC)
45523173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      PDecl->setLexicalDeclContext(lexicalDC);
45623173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek  }
45728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
45828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (T->isArrayType() || T->isFunctionType()) {
45928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(AtLoc, diag::err_property_type) << T;
46028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setInvalidDecl();
46128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
46228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
46328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ProcessDeclAttributes(S, PDecl, FD.D);
46428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
46528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Regardless of setter/getter attribute, we save the default getter/setter
46628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // selector names in anticipation of declaration of setter/getter methods.
46728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  PDecl->setGetterName(GetterSel);
46828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  PDecl->setSetterName(SetterSel);
4690a68dc7f04be1542ce249ff4adb169453698ad91Argyrios Kyrtzidis  PDecl->setPropertyAttributesAsWritten(
470dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                          makePropertyAttributesAsWritten(AttributesAsWritten));
4710a68dc7f04be1542ce249ff4adb169453698ad91Argyrios Kyrtzidis
47228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
47328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly);
47428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
47528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_getter)
47628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_getter);
47728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
47828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_setter)
47928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_setter);
48028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
48128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (isReadWrite)
48228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite);
48328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
48428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_retain)
48528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain);
48628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
487f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_strong)
488f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
489f85e193739c953358c865005855253af4f68a497John McCall
490f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_weak)
491f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_weak);
492f85e193739c953358c865005855253af4f68a497John McCall
49328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_copy)
49428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy);
49528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
496f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
497f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
498f85e193739c953358c865005855253af4f68a497John McCall
49928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (isAssign)
50028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign);
50128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
502265941bc308d65cc270d5c4de5806f37ce405606John McCall  // In the semantic attributes, one of nonatomic or atomic is always set.
50328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)
50428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_nonatomic);
505265941bc308d65cc270d5c4de5806f37ce405606John McCall  else
50645937ae10a0f70f74508165aab4f2b63e18ea747Fariborz Jahanian    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_atomic);
50728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
508f85e193739c953358c865005855253af4f68a497John McCall  // 'unsafe_unretained' is alias for 'assign'.
509f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
510f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign);
511f85e193739c953358c865005855253af4f68a497John McCall  if (isAssign)
512f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
513f85e193739c953358c865005855253af4f68a497John McCall
51428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (MethodImplKind == tok::objc_required)
51528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyImplementation(ObjCPropertyDecl::Required);
51628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  else if (MethodImplKind == tok::objc_optional)
51728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyImplementation(ObjCPropertyDecl::Optional);
51828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
519e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  return PDecl;
52028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek}
52128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
522f85e193739c953358c865005855253af4f68a497John McCallstatic void checkARCPropertyImpl(Sema &S, SourceLocation propertyImplLoc,
523f85e193739c953358c865005855253af4f68a497John McCall                                 ObjCPropertyDecl *property,
524f85e193739c953358c865005855253af4f68a497John McCall                                 ObjCIvarDecl *ivar) {
525f85e193739c953358c865005855253af4f68a497John McCall  if (property->isInvalidDecl() || ivar->isInvalidDecl()) return;
526f85e193739c953358c865005855253af4f68a497John McCall
527f85e193739c953358c865005855253af4f68a497John McCall  QualType ivarType = ivar->getType();
528f85e193739c953358c865005855253af4f68a497John McCall  Qualifiers::ObjCLifetime ivarLifetime = ivarType.getObjCLifetime();
529f85e193739c953358c865005855253af4f68a497John McCall
530265941bc308d65cc270d5c4de5806f37ce405606John McCall  // The lifetime implied by the property's attributes.
531265941bc308d65cc270d5c4de5806f37ce405606John McCall  Qualifiers::ObjCLifetime propertyLifetime =
532265941bc308d65cc270d5c4de5806f37ce405606John McCall    getImpliedARCOwnership(property->getPropertyAttributes(),
533265941bc308d65cc270d5c4de5806f37ce405606John McCall                           property->getType());
534f85e193739c953358c865005855253af4f68a497John McCall
535265941bc308d65cc270d5c4de5806f37ce405606John McCall  // We're fine if they match.
536265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (propertyLifetime == ivarLifetime) return;
537f85e193739c953358c865005855253af4f68a497John McCall
538265941bc308d65cc270d5c4de5806f37ce405606John McCall  // These aren't valid lifetimes for object ivars;  don't diagnose twice.
539265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (ivarLifetime == Qualifiers::OCL_None ||
540265941bc308d65cc270d5c4de5806f37ce405606John McCall      ivarLifetime == Qualifiers::OCL_Autoreleasing)
541265941bc308d65cc270d5c4de5806f37ce405606John McCall    return;
542265941bc308d65cc270d5c4de5806f37ce405606John McCall
543265941bc308d65cc270d5c4de5806f37ce405606John McCall  switch (propertyLifetime) {
544265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_Strong:
545265941bc308d65cc270d5c4de5806f37ce405606John McCall    S.Diag(propertyImplLoc, diag::err_arc_strong_property_ownership)
546265941bc308d65cc270d5c4de5806f37ce405606John McCall      << property->getDeclName()
547265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivar->getDeclName()
548265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivarLifetime;
549265941bc308d65cc270d5c4de5806f37ce405606John McCall    break;
550265941bc308d65cc270d5c4de5806f37ce405606John McCall
551265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_Weak:
552265941bc308d65cc270d5c4de5806f37ce405606John McCall    S.Diag(propertyImplLoc, diag::error_weak_property)
553265941bc308d65cc270d5c4de5806f37ce405606John McCall      << property->getDeclName()
554265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivar->getDeclName();
555265941bc308d65cc270d5c4de5806f37ce405606John McCall    break;
556265941bc308d65cc270d5c4de5806f37ce405606John McCall
557265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_ExplicitNone:
558265941bc308d65cc270d5c4de5806f37ce405606John McCall    S.Diag(propertyImplLoc, diag::err_arc_assign_property_ownership)
559265941bc308d65cc270d5c4de5806f37ce405606John McCall      << property->getDeclName()
560265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivar->getDeclName()
561265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ((property->getPropertyAttributesAsWritten()
562265941bc308d65cc270d5c4de5806f37ce405606John McCall           & ObjCPropertyDecl::OBJC_PR_assign) != 0);
563265941bc308d65cc270d5c4de5806f37ce405606John McCall    break;
564265941bc308d65cc270d5c4de5806f37ce405606John McCall
565265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_Autoreleasing:
566265941bc308d65cc270d5c4de5806f37ce405606John McCall    llvm_unreachable("properties cannot be autoreleasing");
567265941bc308d65cc270d5c4de5806f37ce405606John McCall
568265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_None:
569265941bc308d65cc270d5c4de5806f37ce405606John McCall    // Any other property should be ignored.
570f85e193739c953358c865005855253af4f68a497John McCall    return;
571f85e193739c953358c865005855253af4f68a497John McCall  }
572f85e193739c953358c865005855253af4f68a497John McCall
573f85e193739c953358c865005855253af4f68a497John McCall  S.Diag(property->getLocation(), diag::note_property_declare);
574f85e193739c953358c865005855253af4f68a497John McCall}
575f85e193739c953358c865005855253af4f68a497John McCall
576015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// setImpliedPropertyAttributeForReadOnlyProperty -
577015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// This routine evaludates life-time attributes for a 'readonly'
578015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// property with no known lifetime of its own, using backing
579015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// 'ivar's attribute, if any. If no backing 'ivar', property's
580015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// life-time is assumed 'strong'.
581015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanianstatic void setImpliedPropertyAttributeForReadOnlyProperty(
582015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian              ObjCPropertyDecl *property, ObjCIvarDecl *ivar) {
583015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  Qualifiers::ObjCLifetime propertyLifetime =
584015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    getImpliedARCOwnership(property->getPropertyAttributes(),
585015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                           property->getType());
586015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  if (propertyLifetime != Qualifiers::OCL_None)
587015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    return;
588015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian
589015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  if (!ivar) {
590015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // if no backing ivar, make property 'strong'.
591015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    property->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
592015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    return;
593015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  }
594015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  // property assumes owenership of backing ivar.
595015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  QualType ivarType = ivar->getType();
596015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  Qualifiers::ObjCLifetime ivarLifetime = ivarType.getObjCLifetime();
597015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  if (ivarLifetime == Qualifiers::OCL_Strong)
598015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    property->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
599015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  else if (ivarLifetime == Qualifiers::OCL_Weak)
600015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    property->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_weak);
601015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  return;
602015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian}
60328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
60428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// ActOnPropertyImplDecl - This routine performs semantic checks and
60528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// builds the AST node for a property implementation declaration; declared
60628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// as @synthesize or @dynamic.
60728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek///
608d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallDecl *Sema::ActOnPropertyImplDecl(Scope *S,
609d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  SourceLocation AtLoc,
610d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  SourceLocation PropertyLoc,
611d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  bool Synthesize,
612d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  IdentifierInfo *PropertyId,
613a4ffd85a6684e42f900aad5459e58ad91bb88755Douglas Gregor                                  IdentifierInfo *PropertyIvar,
614a4ffd85a6684e42f900aad5459e58ad91bb88755Douglas Gregor                                  SourceLocation PropertyIvarLoc) {
615e9686572f17968048a5e097254f85c202ea44aecTed Kremenek  ObjCContainerDecl *ClassImpDecl =
61684e0ccff7a6e51d7b81fd58686b493a4880dd44dFariborz Jahanian    dyn_cast<ObjCContainerDecl>(CurContext);
61728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Make sure we have a context for the property implementation declaration.
61828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (!ClassImpDecl) {
61928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(AtLoc, diag::error_missing_property_context);
620d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall    return 0;
62128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
622f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis  if (PropertyIvarLoc.isInvalid())
623f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis    PropertyIvarLoc = PropertyLoc;
624e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman  SourceLocation PropertyDiagLoc = PropertyLoc;
625e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman  if (PropertyDiagLoc.isInvalid())
626e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    PropertyDiagLoc = ClassImpDecl->getLocStart();
62728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyDecl *property = 0;
62828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCInterfaceDecl* IDecl = 0;
62928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Find the class or category class where this property must have
63028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // a declaration.
63128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCImplementationDecl *IC = 0;
63228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCCategoryImplDecl* CatImplClass = 0;
63328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if ((IC = dyn_cast<ObjCImplementationDecl>(ClassImpDecl))) {
63428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IDecl = IC->getClassInterface();
63528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // We always synthesize an interface for an implementation
63628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // without an interface decl. So, IDecl is always non-zero.
63728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    assert(IDecl &&
63828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek           "ActOnPropertyImplDecl - @implementation without @interface");
63928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
64028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // Look for this property declaration in the @implementation's @interface
64128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    property = IDecl->FindPropertyDeclaration(PropertyId);
64228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!property) {
64328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PropertyLoc, diag::error_bad_property_decl) << IDecl->getDeclName();
644d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
64528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
646dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian    unsigned PIkind = property->getPropertyAttributesAsWritten();
64745937ae10a0f70f74508165aab4f2b63e18ea747Fariborz Jahanian    if ((PIkind & (ObjCPropertyDecl::OBJC_PR_atomic |
64845937ae10a0f70f74508165aab4f2b63e18ea747Fariborz Jahanian                   ObjCPropertyDecl::OBJC_PR_nonatomic) ) == 0) {
649dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian      if (AtLoc.isValid())
650dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian        Diag(AtLoc, diag::warn_implicit_atomic_property);
651dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian      else
652dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian        Diag(IC->getLocation(), diag::warn_auto_implicit_atomic_property);
653dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian      Diag(property->getLocation(), diag::note_property_declare);
654dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian    }
655dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian
65628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (const ObjCCategoryDecl *CD =
65728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        dyn_cast<ObjCCategoryDecl>(property->getDeclContext())) {
65828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if (!CD->IsClassExtension()) {
65928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PropertyLoc, diag::error_category_property) << CD->getDeclName();
66028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(property->getLocation(), diag::note_property_declare);
661d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall        return 0;
66228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
66328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
6642b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian
6652b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian    if (Synthesize&&
6662b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian        (PIkind & ObjCPropertyDecl::OBJC_PR_readonly) &&
6672b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian        property->hasAttr<IBOutletAttr>() &&
6682b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian        !AtLoc.isValid()) {
6692b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian      unsigned rwPIKind = (PIkind | ObjCPropertyDecl::OBJC_PR_readwrite);
6702b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian      rwPIKind &= (~ObjCPropertyDecl::OBJC_PR_readonly);
6712b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian      Diag(IC->getLocation(), diag::warn_auto_readonly_iboutlet_property);
6722b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian      Diag(property->getLocation(), diag::note_property_declare);
673edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      SourceLocation readonlyLoc;
6745bf0e3522fac400cf7bf49a775554f0fc942b336Fariborz Jahanian      if (LocPropertyAttribute(Context, "readonly",
675edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian                               property->getLParenLoc(), readonlyLoc)) {
676edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        SourceLocation endLoc =
677edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian          readonlyLoc.getLocWithOffset(strlen("readonly")-1);
678edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        SourceRange ReadonlySourceRange(readonlyLoc, endLoc);
679edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        Diag(property->getLocation(),
680edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian             diag::note_auto_readonly_iboutlet_fixup_suggest) <<
681edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        FixItHint::CreateReplacement(ReadonlySourceRange, "readwrite");
682edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      }
6832b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian    }
6842b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian
68528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else if ((CatImplClass = dyn_cast<ObjCCategoryImplDecl>(ClassImpDecl))) {
68628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Synthesize) {
68728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(AtLoc, diag::error_synthesize_category_decl);
688d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
68928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
69028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IDecl = CatImplClass->getClassInterface();
69128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!IDecl) {
69228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(AtLoc, diag::error_missing_property_interface);
693d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
69428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
69528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    ObjCCategoryDecl *Category =
69628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IDecl->FindCategoryDeclaration(CatImplClass->getIdentifier());
69728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
69828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // If category for this implementation not found, it is an error which
69928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // has already been reported eralier.
70028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!Category)
701d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
70228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // Look for this property declaration in @implementation's category
70328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    property = Category->FindPropertyDeclaration(PropertyId);
70428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!property) {
70528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PropertyLoc, diag::error_bad_category_property_decl)
70628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << Category->getDeclName();
707d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
70828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
70928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else {
71028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(AtLoc, diag::error_bad_property_context);
711d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall    return 0;
71228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
71328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCIvarDecl *Ivar = 0;
714e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman  bool CompleteTypeErr = false;
715744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian  bool compat = true;
71628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Check that we have a valid, previously declared ivar for @synthesize
71728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Synthesize) {
71828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // @synthesize
71928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!PropertyIvar)
72028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      PropertyIvar = PropertyId;
721015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // Check that this is a previously declared 'ivar' in 'IDecl' interface
722015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    ObjCInterfaceDecl *ClassDeclared;
723015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    Ivar = IDecl->lookupInstanceVariable(PropertyIvar, ClassDeclared);
724265941bc308d65cc270d5c4de5806f37ce405606John McCall    QualType PropType = property->getType();
725265941bc308d65cc270d5c4de5806f37ce405606John McCall    QualType PropertyIvarType = PropType.getNonReferenceType();
726e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
727e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    if (RequireCompleteType(PropertyDiagLoc, PropertyIvarType,
728d10099e5c8238fa0327f03921cf2e3c8975c881eDouglas Gregor                            diag::err_incomplete_synthesized_property,
729d10099e5c8238fa0327f03921cf2e3c8975c881eDouglas Gregor                            property->getDeclName())) {
730e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      Diag(property->getLocation(), diag::note_property_declare);
731e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      CompleteTypeErr = true;
732e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    }
733e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
7344e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount &&
735015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian        (property->getPropertyAttributesAsWritten() &
7363efd348d3586b22bb0153f17181128a1b3c41140Fariborz Jahanian         ObjCPropertyDecl::OBJC_PR_readonly) &&
7373efd348d3586b22bb0153f17181128a1b3c41140Fariborz Jahanian        PropertyIvarType->isObjCRetainableType()) {
738015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      setImpliedPropertyAttributeForReadOnlyProperty(property, Ivar);
739015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    }
740015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian
741015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    ObjCPropertyDecl::PropertyAttributeKind kind
742015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      = property->getPropertyAttributes();
743265941bc308d65cc270d5c4de5806f37ce405606John McCall
744265941bc308d65cc270d5c4de5806f37ce405606John McCall    // Add GC __weak to the ivar type if the property is weak.
745265941bc308d65cc270d5c4de5806f37ce405606John McCall    if ((kind & ObjCPropertyDecl::OBJC_PR_weak) &&
7464e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        getLangOpts().getGC() != LangOptions::NonGC) {
7474e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      assert(!getLangOpts().ObjCAutoRefCount);
748265941bc308d65cc270d5c4de5806f37ce405606John McCall      if (PropertyIvarType.isObjCGCStrong()) {
749e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::err_gc_weak_property_strong_type);
750265941bc308d65cc270d5c4de5806f37ce405606John McCall        Diag(property->getLocation(), diag::note_property_declare);
751265941bc308d65cc270d5c4de5806f37ce405606John McCall      } else {
752265941bc308d65cc270d5c4de5806f37ce405606John McCall        PropertyIvarType =
753265941bc308d65cc270d5c4de5806f37ce405606John McCall          Context.getObjCGCQualType(PropertyIvarType, Qualifiers::Weak);
754edc08821d3f73514291b339db29aca1a17485e39Fariborz Jahanian      }
755edc08821d3f73514291b339db29aca1a17485e39Fariborz Jahanian    }
756265941bc308d65cc270d5c4de5806f37ce405606John McCall
75728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!Ivar) {
758265941bc308d65cc270d5c4de5806f37ce405606John McCall      // In ARC, give the ivar a lifetime qualifier based on the
759f85e193739c953358c865005855253af4f68a497John McCall      // property attributes.
7604e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      if (getLangOpts().ObjCAutoRefCount &&
761265941bc308d65cc270d5c4de5806f37ce405606John McCall          !PropertyIvarType.getObjCLifetime() &&
762265941bc308d65cc270d5c4de5806f37ce405606John McCall          PropertyIvarType->isObjCRetainableType()) {
763f85e193739c953358c865005855253af4f68a497John McCall
764265941bc308d65cc270d5c4de5806f37ce405606John McCall        // It's an error if we have to do this and the user didn't
765265941bc308d65cc270d5c4de5806f37ce405606John McCall        // explicitly write an ownership attribute on the property.
766473506bd9dfd84944c2c5ca2c2a38814f46febc6Argyrios Kyrtzidis        if (!property->hasWrittenStorageAttribute() &&
767265941bc308d65cc270d5c4de5806f37ce405606John McCall            !(kind & ObjCPropertyDecl::OBJC_PR_strong)) {
768e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman          Diag(PropertyDiagLoc,
769473506bd9dfd84944c2c5ca2c2a38814f46febc6Argyrios Kyrtzidis               diag::err_arc_objc_property_default_assign_on_object);
770473506bd9dfd84944c2c5ca2c2a38814f46febc6Argyrios Kyrtzidis          Diag(property->getLocation(), diag::note_property_declare);
771265941bc308d65cc270d5c4de5806f37ce405606John McCall        } else {
772265941bc308d65cc270d5c4de5806f37ce405606John McCall          Qualifiers::ObjCLifetime lifetime =
773265941bc308d65cc270d5c4de5806f37ce405606John McCall            getImpliedARCOwnership(kind, PropertyIvarType);
774265941bc308d65cc270d5c4de5806f37ce405606John McCall          assert(lifetime && "no lifetime for property?");
7756dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian          if (lifetime == Qualifiers::OCL_Weak) {
7766dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian            bool err = false;
7776dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian            if (const ObjCObjectPointerType *ObjT =
7786dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian                PropertyIvarType->getAs<ObjCObjectPointerType>())
7796dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian              if (ObjT->getInterfaceDecl()->isArcWeakrefUnavailable()) {
780e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman                Diag(PropertyDiagLoc, diag::err_arc_weak_unavailable_property);
7816dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian                Diag(property->getLocation(), diag::note_property_declare);
7826dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian                err = true;
7836dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian              }
7844e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie            if (!err && !getLangOpts().ObjCRuntimeHasWeak) {
785e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman              Diag(PropertyDiagLoc, diag::err_arc_weak_no_runtime);
7866dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian              Diag(property->getLocation(), diag::note_property_declare);
7876dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian            }
788f85e193739c953358c865005855253af4f68a497John McCall          }
7896dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian
790f85e193739c953358c865005855253af4f68a497John McCall          Qualifiers qs;
791265941bc308d65cc270d5c4de5806f37ce405606John McCall          qs.addObjCLifetime(lifetime);
792f85e193739c953358c865005855253af4f68a497John McCall          PropertyIvarType = Context.getQualifiedType(PropertyIvarType, qs);
793f85e193739c953358c865005855253af4f68a497John McCall        }
794f85e193739c953358c865005855253af4f68a497John McCall      }
795f85e193739c953358c865005855253af4f68a497John McCall
796f85e193739c953358c865005855253af4f68a497John McCall      if (kind & ObjCPropertyDecl::OBJC_PR_weak &&
7974e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          !getLangOpts().ObjCAutoRefCount &&
7984e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          getLangOpts().getGC() == LangOptions::NonGC) {
799e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_synthesize_weak_non_arc_or_gc);
800f85e193739c953358c865005855253af4f68a497John McCall        Diag(property->getLocation(), diag::note_property_declare);
801f85e193739c953358c865005855253af4f68a497John McCall      }
802f85e193739c953358c865005855253af4f68a497John McCall
803ff676cb48fe8bf7be2feaa251dc7c5fb15af4730Abramo Bagnara      Ivar = ObjCIvarDecl::Create(Context, ClassImpDecl,
804f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis                                  PropertyIvarLoc,PropertyIvarLoc, PropertyIvar,
80514086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian                                  PropertyIvarType, /*Dinfo=*/0,
8067504966cc0d07eac26baabec80a7406695efb307Fariborz Jahanian                                  ObjCIvarDecl::Private,
807ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian                                  (Expr *)0, true);
808e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      if (CompleteTypeErr)
809e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Ivar->setInvalidDecl();
81029fa69addf8b19871e7866c32c92b5d142c4bfbcDaniel Dunbar      ClassImpDecl->addDecl(Ivar);
8111b7f9cbed1b96b58a6e5f7808ebc9345a76a0936Richard Smith      IDecl->makeDeclVisibleInContext(Ivar);
81228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      property->setPropertyIvarDecl(Ivar);
81328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
8144e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      if (!getLangOpts().ObjCNonFragileABI)
815e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_missing_property_ivar_decl)
816e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman            << PropertyId;
81728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // Note! I deliberately want it to fall thru so, we have a
81828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // a property implementation and to avoid future warnings.
8194e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    } else if (getLangOpts().ObjCNonFragileABI &&
82060ef308e51c71b760d7f598c1b763ceb7b768148Douglas Gregor               !declaresSameEntity(ClassDeclared, IDecl)) {
821e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      Diag(PropertyDiagLoc, diag::error_ivar_in_superclass_use)
82228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << property->getDeclName() << Ivar->getDeclName()
82328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << ClassDeclared->getDeclName();
82428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Ivar->getLocation(), diag::note_previous_access_declaration)
8254087f27e5416c799bcb6be072f905be752acb61cDaniel Dunbar      << Ivar << Ivar->getName();
82628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // Note! I deliberately want it to fall thru so more errors are caught.
82728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
82828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    QualType IvarType = Context.getCanonicalType(Ivar->getType());
82928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
83028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // Check that type of property and its ivar are type compatible.
831744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian    if (!Context.hasSameType(PropertyIvarType, IvarType)) {
832744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian      compat = false;
83314086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian      if (isa<ObjCObjectPointerType>(PropertyIvarType)
834f85e193739c953358c865005855253af4f68a497John McCall          && isa<ObjCObjectPointerType>(IvarType))
83562ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian        compat =
83662ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian          Context.canAssignObjCInterfaces(
83714086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian                                  PropertyIvarType->getAs<ObjCObjectPointerType>(),
83862ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian                                  IvarType->getAs<ObjCObjectPointerType>());
839b608b987718c6d841115464f79ab2d1820a63e17Douglas Gregor      else {
840f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis        compat = (CheckAssignmentConstraints(PropertyIvarLoc, PropertyIvarType,
841f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis                                             IvarType)
842daa8e4e888758d55a7a759dd4a91b83921cef222John McCall                    == Compatible);
843b608b987718c6d841115464f79ab2d1820a63e17Douglas Gregor      }
84462ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian      if (!compat) {
845e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_property_ivar_type)
846f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek          << property->getDeclName() << PropType
847f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek          << Ivar->getDeclName() << IvarType;
848f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek        Diag(Ivar->getLocation(), diag::note_ivar_decl);
84928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // Note! I deliberately want it to fall thru so, we have a
85028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // a property implementation and to avoid future warnings.
85128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
852744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian      else {
853744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        // FIXME! Rules for properties are somewhat different that those
854744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        // for assignments. Use a new routine to consolidate all cases;
855744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        // specifically for property redeclarations as well as for ivars.
856744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        QualType lhsType =Context.getCanonicalType(PropertyIvarType).getUnqualifiedType();
857744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        QualType rhsType =Context.getCanonicalType(IvarType).getUnqualifiedType();
858744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        if (lhsType != rhsType &&
859744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian            lhsType->isArithmeticType()) {
860744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian          Diag(PropertyDiagLoc, diag::error_property_ivar_type)
861744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian            << property->getDeclName() << PropType
862744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian            << Ivar->getDeclName() << IvarType;
863744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian          Diag(Ivar->getLocation(), diag::note_ivar_decl);
864744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian          // Fall thru - see previous comment
865744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        }
86628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
86728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // __weak is explicit. So it works on Canonical type.
868f85e193739c953358c865005855253af4f68a497John McCall      if ((PropType.isObjCGCWeak() && !IvarType.isObjCGCWeak() &&
8694e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie           getLangOpts().getGC() != LangOptions::NonGC)) {
870e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_weak_property)
87128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << property->getDeclName() << Ivar->getDeclName();
872edc08821d3f73514291b339db29aca1a17485e39Fariborz Jahanian        Diag(Ivar->getLocation(), diag::note_ivar_decl);
87328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // Fall thru - see previous comment
87428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
875f85e193739c953358c865005855253af4f68a497John McCall      // Fall thru - see previous comment
87628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if ((property->getType()->isObjCObjectPointerType() ||
87728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek           PropType.isObjCGCStrong()) && IvarType.isObjCGCWeak() &&
8784e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          getLangOpts().getGC() != LangOptions::NonGC) {
879e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_strong_property)
88028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << property->getDeclName() << Ivar->getDeclName();
88128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // Fall thru - see previous comment
88228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
88328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
8844e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount)
885f85e193739c953358c865005855253af4f68a497John McCall      checkARCPropertyImpl(*this, PropertyLoc, property, Ivar);
88628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else if (PropertyIvar)
88728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // @dynamic
888e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    Diag(PropertyDiagLoc, diag::error_dynamic_property_ivar_decl);
889f85e193739c953358c865005855253af4f68a497John McCall
89028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  assert (property && "ActOnPropertyImplDecl - property declaration missing");
89128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyImplDecl *PIDecl =
89228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyImplDecl::Create(Context, CurContext, AtLoc, PropertyLoc,
89328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                               property,
89428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                               (Synthesize ?
89528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                                ObjCPropertyImplDecl::Synthesize
89628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                                : ObjCPropertyImplDecl::Dynamic),
897a4ffd85a6684e42f900aad5459e58ad91bb88755Douglas Gregor                               Ivar, PropertyIvarLoc);
898e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
899744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian  if (CompleteTypeErr || !compat)
900e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    PIDecl->setInvalidDecl();
901e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
90217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  if (ObjCMethodDecl *getterMethod = property->getGetterMethodDecl()) {
90317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    getterMethod->createImplicitParams(Context, IDecl);
904e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    if (getLangOpts().CPlusPlus && Synthesize && !CompleteTypeErr &&
9050313f441b72ef6b69a93e5003c684b01cb72fd46Fariborz Jahanian        Ivar->getType()->isRecordType()) {
90617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // For Objective-C++, need to synthesize the AST for the IVAR object to be
90717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // returned by the getter as it must conform to C++'s copy-return rules.
90817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // FIXME. Eventually we want to do this for Objective-C as well.
90917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ImplicitParamDecl *SelfDecl = getterMethod->getSelfDecl();
91017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      DeclRefExpr *SelfExpr =
911f4b88a45902af1802a1cb42ba48b1c474474f228John McCall        new (Context) DeclRefExpr(SelfDecl, false, SelfDecl->getType(),
912f89e55ab1bfb3ea997f8b02997c611a02254eb2dJohn McCall                                  VK_RValue, SourceLocation());
91317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      Expr *IvarRefExpr =
91417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc,
91517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                      SelfExpr, true, true);
91660d7b3a319d84d688752be3870615ac0f111fb16John McCall      ExprResult Res =
91717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        PerformCopyInitialization(InitializedEntity::InitializeResult(
9183c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor                                    SourceLocation(),
9193c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor                                    getterMethod->getResultType(),
9203c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor                                    /*NRVO=*/false),
92117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                  SourceLocation(),
92217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                  Owned(IvarRefExpr));
92317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      if (!Res.isInvalid()) {
92417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        Expr *ResExpr = Res.takeAs<Expr>();
92517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        if (ResExpr)
9264765fa05b5652fcc4356371c2f481d0ea9a1b007John McCall          ResExpr = MaybeCreateExprWithCleanups(ResExpr);
92717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        PIDecl->setGetterCXXConstructor(ResExpr);
92817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      }
92917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    }
930831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    if (property->hasAttr<NSReturnsNotRetainedAttr>() &&
931831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian        !getterMethod->hasAttr<NSReturnsNotRetainedAttr>()) {
932831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian      Diag(getterMethod->getLocation(),
933831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian           diag::warn_property_getter_owning_mismatch);
934831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian      Diag(property->getLocation(), diag::note_property_declare);
935831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    }
93617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  }
93717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  if (ObjCMethodDecl *setterMethod = property->getSetterMethodDecl()) {
93817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    setterMethod->createImplicitParams(Context, IDecl);
939e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    if (getLangOpts().CPlusPlus && Synthesize && !CompleteTypeErr &&
940e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Ivar->getType()->isRecordType()) {
94117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // FIXME. Eventually we want to do this for Objective-C as well.
94217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ImplicitParamDecl *SelfDecl = setterMethod->getSelfDecl();
94317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      DeclRefExpr *SelfExpr =
944f4b88a45902af1802a1cb42ba48b1c474474f228John McCall        new (Context) DeclRefExpr(SelfDecl, false, SelfDecl->getType(),
945f89e55ab1bfb3ea997f8b02997c611a02254eb2dJohn McCall                                  VK_RValue, SourceLocation());
94617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      Expr *lhs =
94717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc,
94817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                      SelfExpr, true, true);
94917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ObjCMethodDecl::param_iterator P = setterMethod->param_begin();
95017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ParmVarDecl *Param = (*P);
9513c3b7f90a863af43fa63043d396553ecf205351cJohn McCall      QualType T = Param->getType().getNonReferenceType();
952f4b88a45902af1802a1cb42ba48b1c474474f228John McCall      Expr *rhs = new (Context) DeclRefExpr(Param, false, T,
953f89e55ab1bfb3ea997f8b02997c611a02254eb2dJohn McCall                                            VK_LValue, SourceLocation());
954fa432391a708448518df1e6ced4e34f4c349d169Fariborz Jahanian      ExprResult Res = BuildBinOp(S, lhs->getLocEnd(),
9552de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall                                  BO_Assign, lhs, rhs);
95657e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian      if (property->getPropertyAttributes() &
95757e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian          ObjCPropertyDecl::OBJC_PR_atomic) {
95857e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian        Expr *callExpr = Res.takeAs<Expr>();
95957e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian        if (const CXXOperatorCallExpr *CXXCE =
96013bf6336ab395eb6af487ab96d32229460907769Fariborz Jahanian              dyn_cast_or_null<CXXOperatorCallExpr>(callExpr))
96113bf6336ab395eb6af487ab96d32229460907769Fariborz Jahanian          if (const FunctionDecl *FuncDecl = CXXCE->getDirectCallee())
96257e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian            if (!FuncDecl->isTrivial())
96320abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian              if (property->getType()->isReferenceType()) {
96420abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                Diag(PropertyLoc,
96520abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                     diag::err_atomic_property_nontrivial_assign_op)
96657e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian                    << property->getType();
96720abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                Diag(FuncDecl->getLocStart(),
96820abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                     diag::note_callee_decl) << FuncDecl;
96920abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian              }
97057e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian      }
97117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      PIDecl->setSetterCXXAssignment(Res.takeAs<Expr>());
97217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    }
97317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  }
97417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian
97528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (IC) {
97628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Synthesize)
97728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if (ObjCPropertyImplDecl *PPIDecl =
97828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek          IC->FindPropertyImplIvarDecl(PropertyIvar)) {
97928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PropertyLoc, diag::error_duplicate_ivar_use)
98028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier()
98128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyIvar;
98228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PPIDecl->getLocation(), diag::note_previous_use);
98328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
98428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
98528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (ObjCPropertyImplDecl *PPIDecl
98628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        = IC->FindPropertyImplDecl(PropertyId)) {
98728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PropertyLoc, diag::error_property_implemented) << PropertyId;
98828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PPIDecl->getLocation(), diag::note_previous_declaration);
989d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
99028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
99128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IC->addPropertyImplementation(PIDecl);
9924e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCDefaultSynthProperties &&
9934e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        getLangOpts().ObjCNonFragileABI2 &&
99471207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek        !IDecl->isObjCRequiresPropertyDefs()) {
995ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      // Diagnose if an ivar was lazily synthesdized due to a previous
996ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      // use and if 1) property is @dynamic or 2) property is synthesized
997cdaa6a8fed16d8bd3987fb4f3304dfb4e52876c3Fariborz Jahanian      // but it requires an ivar of different name.
998411c25c4512f00e07442e8818d011c7e0902d739Fariborz Jahanian      ObjCInterfaceDecl *ClassDeclared=0;
999ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      ObjCIvarDecl *Ivar = 0;
1000ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      if (!Synthesize)
1001ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        Ivar = IDecl->lookupInstanceVariable(PropertyId, ClassDeclared);
1002ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      else {
1003ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        if (PropertyIvar && PropertyIvar != PropertyId)
1004ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian          Ivar = IDecl->lookupInstanceVariable(PropertyId, ClassDeclared);
1005ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      }
1006cdaa6a8fed16d8bd3987fb4f3304dfb4e52876c3Fariborz Jahanian      // Issue diagnostics only if Ivar belongs to current class.
1007cdaa6a8fed16d8bd3987fb4f3304dfb4e52876c3Fariborz Jahanian      if (Ivar && Ivar->getSynthesize() &&
100860ef308e51c71b760d7f598c1b763ceb7b768148Douglas Gregor          declaresSameEntity(IC->getClassInterface(), ClassDeclared)) {
1009ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        Diag(Ivar->getLocation(), diag::err_undeclared_var_use)
1010ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        << PropertyId;
1011ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        Ivar->setInvalidDecl();
1012ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      }
1013ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian    }
101428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else {
101528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Synthesize)
101628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if (ObjCPropertyImplDecl *PPIDecl =
101728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek          CatImplClass->FindPropertyImplIvarDecl(PropertyIvar)) {
1018e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_duplicate_ivar_use)
101928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier()
102028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyIvar;
102128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PPIDecl->getLocation(), diag::note_previous_use);
102228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
102328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
102428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (ObjCPropertyImplDecl *PPIDecl =
102528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        CatImplClass->FindPropertyImplDecl(PropertyId)) {
1026e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      Diag(PropertyDiagLoc, diag::error_property_implemented) << PropertyId;
102728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PPIDecl->getLocation(), diag::note_previous_declaration);
1028d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
102928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
103028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    CatImplClass->addPropertyImplementation(PIDecl);
103128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
103228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
1033d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall  return PIDecl;
103428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek}
103528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
103628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===//
103728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek// Helper methods.
103828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===//
103928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
10409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// DiagnosePropertyMismatch - Compares two properties for their
10419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// attributes and types and warns on a variety of inconsistencies.
10429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
10439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid
10449d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::DiagnosePropertyMismatch(ObjCPropertyDecl *Property,
10459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                               ObjCPropertyDecl *SuperProperty,
10469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                               const IdentifierInfo *inheritedName) {
10479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCPropertyDecl::PropertyAttributeKind CAttr =
10489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  Property->getPropertyAttributes();
10499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCPropertyDecl::PropertyAttributeKind SAttr =
10509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  SuperProperty->getPropertyAttributes();
10519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((CAttr & ObjCPropertyDecl::OBJC_PR_readonly)
10529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      && (SAttr & ObjCPropertyDecl::OBJC_PR_readwrite))
10539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_readonly_property)
10549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << inheritedName;
10559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((CAttr & ObjCPropertyDecl::OBJC_PR_copy)
10569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      != (SAttr & ObjCPropertyDecl::OBJC_PR_copy))
10579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
10589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "copy" << inheritedName;
10591b46d8d78be713c546fb9680eaeea8f793bee5bfFariborz Jahanian  else if (!(SAttr & ObjCPropertyDecl::OBJC_PR_readonly)){
1060f85e193739c953358c865005855253af4f68a497John McCall    unsigned CAttrRetain =
1061f85e193739c953358c865005855253af4f68a497John McCall      (CAttr &
1062f85e193739c953358c865005855253af4f68a497John McCall       (ObjCPropertyDecl::OBJC_PR_retain | ObjCPropertyDecl::OBJC_PR_strong));
1063f85e193739c953358c865005855253af4f68a497John McCall    unsigned SAttrRetain =
1064f85e193739c953358c865005855253af4f68a497John McCall      (SAttr &
1065f85e193739c953358c865005855253af4f68a497John McCall       (ObjCPropertyDecl::OBJC_PR_retain | ObjCPropertyDecl::OBJC_PR_strong));
1066f85e193739c953358c865005855253af4f68a497John McCall    bool CStrong = (CAttrRetain != 0);
1067f85e193739c953358c865005855253af4f68a497John McCall    bool SStrong = (SAttrRetain != 0);
1068f85e193739c953358c865005855253af4f68a497John McCall    if (CStrong != SStrong)
1069f85e193739c953358c865005855253af4f68a497John McCall      Diag(Property->getLocation(), diag::warn_property_attribute)
1070f85e193739c953358c865005855253af4f68a497John McCall        << Property->getDeclName() << "retain (or strong)" << inheritedName;
1071f85e193739c953358c865005855253af4f68a497John McCall  }
10729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
10739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((CAttr & ObjCPropertyDecl::OBJC_PR_nonatomic)
10749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      != (SAttr & ObjCPropertyDecl::OBJC_PR_nonatomic))
10759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
10769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "atomic" << inheritedName;
10779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (Property->getSetterName() != SuperProperty->getSetterName())
10789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
10799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "setter" << inheritedName;
10809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (Property->getGetterName() != SuperProperty->getGetterName())
10819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
10829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "getter" << inheritedName;
10839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
10849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  QualType LHSType =
10859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Context.getCanonicalType(SuperProperty->getType());
10869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  QualType RHSType =
10879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Context.getCanonicalType(Property->getType());
10889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1089c286f3835eb6001c61664cef5d610dfaf80a6e9bFariborz Jahanian  if (!Context.propertyTypesAreCompatible(LHSType, RHSType)) {
10908beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    // Do cases not handled in above.
10918beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    // FIXME. For future support of covariant property types, revisit this.
10928beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    bool IncompatibleObjC = false;
10938beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    QualType ConvertedType;
10948beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    if (!isObjCPointerConversion(RHSType, LHSType,
10958beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian                                 ConvertedType, IncompatibleObjC) ||
109613546a871db44c707001f507a9edfd0d5bcc81b4Fariborz Jahanian        IncompatibleObjC) {
10978beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian        Diag(Property->getLocation(), diag::warn_property_types_are_incompatible)
10988beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian        << Property->getType() << SuperProperty->getType() << inheritedName;
109913546a871db44c707001f507a9edfd0d5bcc81b4Fariborz Jahanian      Diag(SuperProperty->getLocation(), diag::note_property_declare);
110013546a871db44c707001f507a9edfd0d5bcc81b4Fariborz Jahanian    }
11019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
11029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
11039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
11049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekbool Sema::DiagnosePropertyAccessorMismatch(ObjCPropertyDecl *property,
11059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                            ObjCMethodDecl *GetterMethod,
11069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                            SourceLocation Loc) {
11079abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  if (!GetterMethod)
11089abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    return false;
11099abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  QualType GetterType = GetterMethod->getResultType().getNonReferenceType();
11109abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  QualType PropertyIvarType = property->getType().getNonReferenceType();
11119abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  bool compat = Context.hasSameType(PropertyIvarType, GetterType);
11129abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  if (!compat) {
11139abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    if (isa<ObjCObjectPointerType>(PropertyIvarType) &&
11149abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian        isa<ObjCObjectPointerType>(GetterType))
11159abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      compat =
11169abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian        Context.canAssignObjCInterfaces(
11179abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian                                      PropertyIvarType->getAs<ObjCObjectPointerType>(),
11189abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian                                      GetterType->getAs<ObjCObjectPointerType>());
11199abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    else if (CheckAssignmentConstraints(Loc, PropertyIvarType, GetterType)
11209abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian              != Compatible) {
11219abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian          Diag(Loc, diag::error_property_accessor_type)
11229abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian            << property->getDeclName() << PropertyIvarType
11239abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian            << GetterMethod->getSelector() << GetterType;
11249abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian          Diag(GetterMethod->getLocation(), diag::note_declared_at);
11259abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian          return true;
11269abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    } else {
11279abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      compat = true;
11289abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      QualType lhsType =Context.getCanonicalType(PropertyIvarType).getUnqualifiedType();
11299abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      QualType rhsType =Context.getCanonicalType(GetterType).getUnqualifiedType();
11309abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      if (lhsType != rhsType && lhsType->isArithmeticType())
11319abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian        compat = false;
11329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
11339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
11349abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian
11359abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  if (!compat) {
11369abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    Diag(Loc, diag::warn_accessor_property_type_mismatch)
11379abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    << property->getDeclName()
11389abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    << GetterMethod->getSelector();
11399abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    Diag(GetterMethod->getLocation(), diag::note_declared_at);
11409abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    return true;
11419abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  }
11429abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian
11439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  return false;
11449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
11459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
11469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// ComparePropertiesInBaseAndSuper - This routine compares property
11479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// declarations in base and its super class, if any, and issues
1148fc8f0e14ad142ed811e90fbd9a30e419e301c717Chris Lattner/// diagnostics in a variety of inconsistent situations.
11499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
11509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::ComparePropertiesInBaseAndSuper(ObjCInterfaceDecl *IDecl) {
11519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCInterfaceDecl *SDecl = IDecl->getSuperClass();
11529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!SDecl)
11539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
11549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // FIXME: O(N^2)
11559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCInterfaceDecl::prop_iterator S = SDecl->prop_begin(),
11569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       E = SDecl->prop_end(); S != E; ++S) {
1157262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie    ObjCPropertyDecl *SuperPDecl = &*S;
11589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Does property in super class has declaration in current class?
11599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCInterfaceDecl::prop_iterator I = IDecl->prop_begin(),
11609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = IDecl->prop_end(); I != E; ++I) {
1161262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *PDecl = &*I;
11629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (SuperPDecl->getIdentifier() == PDecl->getIdentifier())
11639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          DiagnosePropertyMismatch(PDecl, SuperPDecl,
11649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                   SDecl->getIdentifier());
11659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
11669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
11679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
11689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
11699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// MatchOneProtocolPropertiesInClass - This routine goes thru the list
11709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// of properties declared in a protocol and compares their attribute against
11719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// the same property declared in the class or category.
11729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid
11739d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::MatchOneProtocolPropertiesInClass(Decl *CDecl,
11749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                          ObjCProtocolDecl *PDecl) {
11759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl);
11769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!IDecl) {
11779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Category
11789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl);
11799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    assert (CatDecl && "MatchOneProtocolPropertiesInClass");
11809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!CatDecl->IsClassExtension())
11819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
11829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = PDecl->prop_end(); P != E; ++P) {
1183262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie        ObjCPropertyDecl *Pr = &*P;
11849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        ObjCCategoryDecl::prop_iterator CP, CE;
11859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        // Is this property already in  category's list of properties?
11862d2f9368d35b3628c7e3b4563f74849a0f901a00Ted Kremenek        for (CP = CatDecl->prop_begin(), CE = CatDecl->prop_end(); CP!=CE; ++CP)
1187262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie          if (CP->getIdentifier() == Pr->getIdentifier())
11889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            break;
11899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        if (CP != CE)
11909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          // Property protocol already exist in class. Diagnose any mismatch.
1191262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie          DiagnosePropertyMismatch(&*CP, Pr, PDecl->getIdentifier());
11929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      }
11939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
11949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
11959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
11969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       E = PDecl->prop_end(); P != E; ++P) {
1197262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie    ObjCPropertyDecl *Pr = &*P;
11989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCInterfaceDecl::prop_iterator CP, CE;
11999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Is this property already in  class's list of properties?
12009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (CP = IDecl->prop_begin(), CE = IDecl->prop_end(); CP != CE; ++CP)
1201262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      if (CP->getIdentifier() == Pr->getIdentifier())
12029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        break;
12039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (CP != CE)
12049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Property protocol already exist in class. Diagnose any mismatch.
1205262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      DiagnosePropertyMismatch(&*CP, Pr, PDecl->getIdentifier());
12069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
12079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
12089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// CompareProperties - This routine compares properties
12109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// declared in 'ClassOrProtocol' objects (which can be a class or an
12119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// inherited protocol with the list of properties for class/category 'CDecl'
12129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
1213d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallvoid Sema::CompareProperties(Decl *CDecl, Decl *ClassOrProtocol) {
1214d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall  Decl *ClassDecl = ClassOrProtocol;
12159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl);
12169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!IDecl) {
12189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Category
12199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl);
12209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    assert (CatDecl && "CompareProperties");
12219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (ObjCCategoryDecl *MDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl)) {
12229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCCategoryDecl::protocol_iterator P = MDecl->protocol_begin(),
12239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = MDecl->protocol_end(); P != E; ++P)
12249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Match properties of category with those of protocol (*P)
12259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      MatchOneProtocolPropertiesInClass(CatDecl, *P);
12269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Go thru the list of protocols for this category and recursively match
12289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // their properties with those in the category.
12299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCCategoryDecl::protocol_iterator P = CatDecl->protocol_begin(),
12309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = CatDecl->protocol_end(); P != E; ++P)
1231d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall        CompareProperties(CatDecl, *P);
12329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    } else {
12339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl);
12349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(),
12359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = MD->protocol_end(); P != E; ++P)
12369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        MatchOneProtocolPropertiesInClass(CatDecl, *P);
12379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
12389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
12399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
12409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCInterfaceDecl *MDecl = dyn_cast<ObjCInterfaceDecl>(ClassDecl)) {
124253b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
124353b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          P = MDecl->all_referenced_protocol_begin(),
124453b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          E = MDecl->all_referenced_protocol_end(); P != E; ++P)
12459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Match properties of class IDecl with those of protocol (*P).
12469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      MatchOneProtocolPropertiesInClass(IDecl, *P);
12479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Go thru the list of protocols for this class and recursively match
12499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // their properties with those declared in the class.
125053b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
125153b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          P = IDecl->all_referenced_protocol_begin(),
125253b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          E = IDecl->all_referenced_protocol_end(); P != E; ++P)
1253d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      CompareProperties(IDecl, *P);
12549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  } else {
12559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl);
12569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(),
12579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = MD->protocol_end(); P != E; ++P)
12589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      MatchOneProtocolPropertiesInClass(IDecl, *P);
12599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
12609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
12619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// isPropertyReadonly - Return true if property is readonly, by searching
12639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// for the property in the class and in its categories and implementations
12649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
12659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekbool Sema::isPropertyReadonly(ObjCPropertyDecl *PDecl,
12669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                              ObjCInterfaceDecl *IDecl) {
12679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // by far the most common case.
12689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!PDecl->isReadOnly())
12699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return false;
12709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Even if property is ready only, if interface has a user defined setter,
12719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // it is not considered read only.
12729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (IDecl->getInstanceMethod(PDecl->getSetterName()))
12739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return false;
12749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Main class has the property as 'readonly'. Must search
12769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // through the category list to see if the property's
12779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // attribute has been over-ridden to 'readwrite'.
12789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCCategoryDecl *Category = IDecl->getCategoryList();
12799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       Category; Category = Category->getNextClassCategory()) {
12809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Even if property is ready only, if a category has a user defined setter,
12819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // it is not considered read only.
12829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (Category->getInstanceMethod(PDecl->getSetterName()))
12839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      return false;
12849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCPropertyDecl *P =
12859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Category->FindPropertyDeclaration(PDecl->getIdentifier());
12869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (P && !P->isReadOnly())
12879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      return false;
12889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
12899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Also, check for definition of a setter method in the implementation if
12919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // all else failed.
12929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(CurContext)) {
12939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (ObjCImplementationDecl *IMD =
12949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        dyn_cast<ObjCImplementationDecl>(OMD->getDeclContext())) {
12959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (IMD->getInstanceMethod(PDecl->getSetterName()))
12969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return false;
12979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    } else if (ObjCCategoryImplDecl *CIMD =
12989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek               dyn_cast<ObjCCategoryImplDecl>(OMD->getDeclContext())) {
12999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (CIMD->getInstanceMethod(PDecl->getSetterName()))
13009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return false;
13019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
13029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Lastly, look through the implementation (if one is in scope).
13049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCImplementationDecl *ImpDecl = IDecl->getImplementation())
13059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (ImpDecl->getInstanceMethod(PDecl->getSetterName()))
13069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      return false;
13079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // If all fails, look at the super class.
13089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCInterfaceDecl *SIDecl = IDecl->getSuperClass())
13099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return isPropertyReadonly(PDecl, SIDecl);
13109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  return true;
13119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
13129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// CollectImmediateProperties - This routine collects all properties in
13149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// the class and its conforming protocols; but not those it its super class.
13159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::CollectImmediateProperties(ObjCContainerDecl *CDecl,
1316cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian            llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap,
1317cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian            llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& SuperPropMap) {
13189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) {
13199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
13209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = IDecl->prop_end(); P != E; ++P) {
1321262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *Prop = &*P;
13229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      PropMap[Prop->getIdentifier()] = Prop;
13239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
13249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through class's protocols.
132553b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
132653b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         PI = IDecl->all_referenced_protocol_begin(),
132753b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         E = IDecl->all_referenced_protocol_end(); PI != E; ++PI)
1328cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian        CollectImmediateProperties((*PI), PropMap, SuperPropMap);
13299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCCategoryDecl *CATDecl = dyn_cast<ObjCCategoryDecl>(CDecl)) {
13319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!CATDecl->IsClassExtension())
13329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCContainerDecl::prop_iterator P = CATDecl->prop_begin(),
13339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = CATDecl->prop_end(); P != E; ++P) {
1334262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie        ObjCPropertyDecl *Prop = &*P;
13359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        PropMap[Prop->getIdentifier()] = Prop;
13369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      }
13379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through class's protocols.
133853b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCCategoryDecl::protocol_iterator PI = CATDecl->protocol_begin(),
13399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = CATDecl->protocol_end(); PI != E; ++PI)
1340cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      CollectImmediateProperties((*PI), PropMap, SuperPropMap);
13419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) {
13439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
13449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->prop_end(); P != E; ++P) {
1345262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *Prop = &*P;
1346cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      ObjCPropertyDecl *PropertyFromSuper = SuperPropMap[Prop->getIdentifier()];
1347cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      // Exclude property for protocols which conform to class's super-class,
1348cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      // as super-class has to implement the property.
1349a929ec7b673b244bec024ba7eb95140404a7af5dFariborz Jahanian      if (!PropertyFromSuper ||
1350a929ec7b673b244bec024ba7eb95140404a7af5dFariborz Jahanian          PropertyFromSuper->getIdentifier() != Prop->getIdentifier()) {
1351cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian        ObjCPropertyDecl *&PropEntry = PropMap[Prop->getIdentifier()];
1352cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian        if (!PropEntry)
1353cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian          PropEntry = Prop;
1354cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      }
13559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
13569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through protocol's protocols.
13579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
13589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->protocol_end(); PI != E; ++PI)
1359cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      CollectImmediateProperties((*PI), PropMap, SuperPropMap);
13609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
13629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1363509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// CollectClassPropertyImplementations - This routine collects list of
1364509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties to be implemented in the class. This includes, class's
1365509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// and its conforming protocols' properties.
1366509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianstatic void CollectClassPropertyImplementations(ObjCContainerDecl *CDecl,
1367509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian                llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) {
1368509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) {
1369509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
1370509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian         E = IDecl->prop_end(); P != E; ++P) {
1371262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *Prop = &*P;
1372509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      PropMap[Prop->getIdentifier()] = Prop;
1373509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    }
137453b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
137553b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         PI = IDecl->all_referenced_protocol_begin(),
137653b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         E = IDecl->all_referenced_protocol_end(); PI != E; ++PI)
1377509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      CollectClassPropertyImplementations((*PI), PropMap);
1378509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  }
1379509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) {
1380509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1381509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian         E = PDecl->prop_end(); P != E; ++P) {
1382262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *Prop = &*P;
1383ac37150c13c4e6f54551c52a1118f105be5f136bFariborz Jahanian      if (!PropMap.count(Prop->getIdentifier()))
1384ac37150c13c4e6f54551c52a1118f105be5f136bFariborz Jahanian        PropMap[Prop->getIdentifier()] = Prop;
1385509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    }
1386509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    // scan through protocol's protocols.
1387509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
1388509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian         E = PDecl->protocol_end(); PI != E; ++PI)
1389509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      CollectClassPropertyImplementations((*PI), PropMap);
1390509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  }
1391509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian}
1392509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
1393509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// CollectSuperClassPropertyImplementations - This routine collects list of
1394509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties to be implemented in super class(s) and also coming from their
1395509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// conforming protocols.
1396509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianstatic void CollectSuperClassPropertyImplementations(ObjCInterfaceDecl *CDecl,
1397509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian                llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) {
1398509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  if (ObjCInterfaceDecl *SDecl = CDecl->getSuperClass()) {
1399509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    while (SDecl) {
1400509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      CollectClassPropertyImplementations(SDecl, PropMap);
1401509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      SDecl = SDecl->getSuperClass();
1402509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    }
1403509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  }
1404509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian}
1405509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
14069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// LookupPropertyDecl - Looks up a property in the current class and all
14079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// its protocols.
14089d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekObjCPropertyDecl *Sema::LookupPropertyDecl(const ObjCContainerDecl *CDecl,
14099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                     IdentifierInfo *II) {
14109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (const ObjCInterfaceDecl *IDecl =
14119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        dyn_cast<ObjCInterfaceDecl>(CDecl)) {
14129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
14139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = IDecl->prop_end(); P != E; ++P) {
1414262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *Prop = &*P;
14159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop->getIdentifier() == II)
14169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
14179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through class's protocols.
141953b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
142053b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         PI = IDecl->all_referenced_protocol_begin(),
142153b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         E = IDecl->all_referenced_protocol_end(); PI != E; ++PI) {
14229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II);
14239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop)
14249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
14259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  else if (const ObjCProtocolDecl *PDecl =
14289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            dyn_cast<ObjCProtocolDecl>(CDecl)) {
14299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
14309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->prop_end(); P != E; ++P) {
1431262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      ObjCPropertyDecl *Prop = &*P;
14329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop->getIdentifier() == II)
14339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
14349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through protocol's protocols.
14369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
14379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->protocol_end(); PI != E; ++PI) {
14389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II);
14399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop)
14409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
14419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  return 0;
14449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
14459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1446d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenekstatic IdentifierInfo * getDefaultSynthIvarName(ObjCPropertyDecl *Prop,
1447d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek                                                ASTContext &Ctx) {
1448f7ccbad5d9949e7ddd1cbef43d482553b811e026Dylan Noblesmith  SmallString<128> ivarName;
1449d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek  {
1450d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek    llvm::raw_svector_ostream os(ivarName);
1451d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek    os << '_' << Prop->getIdentifier()->getName();
1452d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek  }
1453d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek  return &Ctx.Idents.get(ivarName.str());
1454d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek}
1455d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek
1456509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// DefaultSynthesizeProperties - This routine default synthesizes all
1457509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties which must be synthesized in class's @implementation.
1458d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenekvoid Sema::DefaultSynthesizeProperties(Scope *S, ObjCImplDecl* IMPDecl,
1459d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek                                       ObjCInterfaceDecl *IDecl) {
1460509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
1461509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap;
1462509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  CollectClassPropertyImplementations(IDecl, PropMap);
1463509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  if (PropMap.empty())
1464509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    return;
1465509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap;
1466509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  CollectSuperClassPropertyImplementations(IDecl, SuperPropMap);
1467509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
1468509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator
1469509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian       P = PropMap.begin(), E = PropMap.end(); P != E; ++P) {
1470509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    ObjCPropertyDecl *Prop = P->second;
1471509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    // If property to be implemented in the super class, ignore.
1472509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    if (SuperPropMap[Prop->getIdentifier()])
1473509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      continue;
1474509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    // Is there a matching propery synthesize/dynamic?
1475509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    if (Prop->isInvalidDecl() ||
1476509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian        Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional ||
1477509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian        IMPDecl->FindPropertyImplIvarDecl(Prop->getIdentifier()))
1478509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      continue;
1479d3635b9681daaef74974932f77080910b032b9fdFariborz Jahanian    // Property may have been synthesized by user.
1480d3635b9681daaef74974932f77080910b032b9fdFariborz Jahanian    if (IMPDecl->FindPropertyImplDecl(Prop->getIdentifier()))
1481d3635b9681daaef74974932f77080910b032b9fdFariborz Jahanian      continue;
148295f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian    if (IMPDecl->getInstanceMethod(Prop->getGetterName())) {
148395f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian      if (Prop->getPropertyAttributes() & ObjCPropertyDecl::OBJC_PR_readonly)
148495f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian        continue;
148595f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian      if (IMPDecl->getInstanceMethod(Prop->getSetterName()))
148695f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian        continue;
148795f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian    }
1488f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian    if (isa<ObjCProtocolDecl>(Prop->getDeclContext())) {
1489f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      // We won't auto-synthesize properties declared in protocols.
1490f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      Diag(IMPDecl->getLocation(),
1491f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian           diag::warn_auto_synthesizing_protocol_property);
1492f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      Diag(Prop->getLocation(), diag::note_property_declare);
1493f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      continue;
1494f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian    }
14952a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek
14962a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // We use invalid SourceLocations for the synthesized ivars since they
14972a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // aren't really synthesized at a particular location; they just exist.
14982a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // Saying that they are located at the @implementation isn't really going
14992a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // to help users.
1500975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian    ObjCPropertyImplDecl *PIDecl = dyn_cast_or_null<ObjCPropertyImplDecl>(
1501975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian      ActOnPropertyImplDecl(S, SourceLocation(), SourceLocation(),
1502975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            true,
1503975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            /* property = */ Prop->getIdentifier(),
1504975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            /* ivar = */ getDefaultSynthIvarName(Prop, Context),
1505975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            SourceLocation()));
1506975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian    if (PIDecl) {
1507975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian      Diag(Prop->getLocation(), diag::warn_missing_explicit_synthesis);
15085ea66619a3108e2985e952b9a60a84316d464b25Fariborz Jahanian      Diag(IMPDecl->getLocation(), diag::note_while_in_implementation);
1509975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian    }
15102a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek  }
1511509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian}
15129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
15138697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanianvoid Sema::DefaultSynthesizeProperties(Scope *S, Decl *D) {
15148697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  if (!LangOpts.ObjCDefaultSynthProperties || !LangOpts.ObjCNonFragileABI2)
15158697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian    return;
15168697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  ObjCImplementationDecl *IC=dyn_cast_or_null<ObjCImplementationDecl>(D);
15178697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  if (!IC)
15188697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian    return;
15198697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  if (ObjCInterfaceDecl* IDecl = IC->getClassInterface())
152071207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek    if (!IDecl->isObjCRequiresPropertyDefs())
1521eb4f2c56c298071d58b441ccf801b039be93788aFariborz Jahanian      DefaultSynthesizeProperties(S, IC, IDecl);
15228697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian}
15238697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian
152417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanianvoid Sema::DiagnoseUnimplementedProperties(Scope *S, ObjCImplDecl* IMPDecl,
15259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                      ObjCContainerDecl *CDecl,
15269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                      const llvm::DenseSet<Selector>& InsMap) {
1527cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap;
1528cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian  if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl))
1529cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian    CollectSuperClassPropertyImplementations(IDecl, SuperPropMap);
1530cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian
15319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap;
1532cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian  CollectImmediateProperties(CDecl, PropMap, SuperPropMap);
15339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (PropMap.empty())
15349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
15359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
15369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  llvm::DenseSet<ObjCPropertyDecl *> PropImplMap;
15379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCImplDecl::propimpl_iterator
15389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       I = IMPDecl->propimpl_begin(),
15399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       EI = IMPDecl->propimpl_end(); I != EI; ++I)
1540262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie    PropImplMap.insert(I->getPropertyDecl());
15419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
15429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator
15439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       P = PropMap.begin(), E = PropMap.end(); P != E; ++P) {
15449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCPropertyDecl *Prop = P->second;
15459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Is there a matching propery synthesize/dynamic?
15469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (Prop->isInvalidDecl() ||
15479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional ||
1548327126ee3d2faad9314b2633974eefc672f73b79Fariborz Jahanian        PropImplMap.count(Prop) || Prop->hasAttr<UnavailableAttr>())
15499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      continue;
15509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!InsMap.count(Prop->getGetterName())) {
1551b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(IMPDecl->getLocation(),
15529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           isa<ObjCCategoryDecl>(CDecl) ?
15539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            diag::warn_setter_getter_impl_required_in_category :
15549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            diag::warn_setter_getter_impl_required)
15559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Prop->getDeclName() << Prop->getGetterName();
1556b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(Prop->getLocation(),
1557b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian           diag::note_property_declare);
1558da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian      if (LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCNonFragileABI2)
1559da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian        if (ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(CDecl))
156071207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek          if (const ObjCInterfaceDecl *RID = ID->isObjCRequiresPropertyDefs())
1561da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian            Diag(RID->getLocation(), diag::note_suppressed_class_declare);
1562da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian
15639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
15649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
15659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!Prop->isReadOnly() && !InsMap.count(Prop->getSetterName())) {
1566b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(IMPDecl->getLocation(),
15679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           isa<ObjCCategoryDecl>(CDecl) ?
15689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           diag::warn_setter_getter_impl_required_in_category :
15699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           diag::warn_setter_getter_impl_required)
15709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Prop->getDeclName() << Prop->getSetterName();
1571b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(Prop->getLocation(),
1572b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian           diag::note_property_declare);
1573da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian      if (LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCNonFragileABI2)
1574da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian        if (ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(CDecl))
157571207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek          if (const ObjCInterfaceDecl *RID = ID->isObjCRequiresPropertyDefs())
1576da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian            Diag(RID->getLocation(), diag::note_suppressed_class_declare);
15779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
15789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
15799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
15809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
15819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid
15829d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::AtomicPropertySetterGetterRules (ObjCImplDecl* IMPDecl,
15839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                       ObjCContainerDecl* IDecl) {
15849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Rules apply in non-GC mode only
15854e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().getGC() != LangOptions::NonGC)
15869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
15879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCContainerDecl::prop_iterator I = IDecl->prop_begin(),
15889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       E = IDecl->prop_end();
15899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       I != E; ++I) {
1590262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie    ObjCPropertyDecl *Property = &*I;
159194659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    ObjCMethodDecl *GetterMethod = 0;
159294659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    ObjCMethodDecl *SetterMethod = 0;
159394659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    bool LookedUpGetterSetter = false;
159494659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis
15959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    unsigned Attributes = Property->getPropertyAttributes();
1596265941bc308d65cc270d5c4de5806f37ce405606John McCall    unsigned AttributesAsWritten = Property->getPropertyAttributesAsWritten();
159794659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis
1598265941bc308d65cc270d5c4de5806f37ce405606John McCall    if (!(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_atomic) &&
1599265941bc308d65cc270d5c4de5806f37ce405606John McCall        !(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_nonatomic)) {
160094659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      GetterMethod = IMPDecl->getInstanceMethod(Property->getGetterName());
160194659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      SetterMethod = IMPDecl->getInstanceMethod(Property->getSetterName());
160294659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      LookedUpGetterSetter = true;
160394659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      if (GetterMethod) {
160494659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(GetterMethod->getLocation(),
160594659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis             diag::warn_default_atomic_custom_getter_setter)
1606293a45e724a15fb58b8805a5791f9f3aee769cf6Argyrios Kyrtzidis          << Property->getIdentifier() << 0;
160794659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(Property->getLocation(), diag::note_property_declare);
160894659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      }
160994659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      if (SetterMethod) {
161094659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(SetterMethod->getLocation(),
161194659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis             diag::warn_default_atomic_custom_getter_setter)
1612293a45e724a15fb58b8805a5791f9f3aee769cf6Argyrios Kyrtzidis          << Property->getIdentifier() << 1;
161394659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(Property->getLocation(), diag::note_property_declare);
161494659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      }
161594659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    }
161694659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis
16179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // We only care about readwrite atomic property.
16189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if ((Attributes & ObjCPropertyDecl::OBJC_PR_nonatomic) ||
16199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        !(Attributes & ObjCPropertyDecl::OBJC_PR_readwrite))
16209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      continue;
16219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (const ObjCPropertyImplDecl *PIDecl
16229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         = IMPDecl->FindPropertyImplDecl(Property->getIdentifier())) {
16239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (PIDecl->getPropertyImplementation() == ObjCPropertyImplDecl::Dynamic)
16249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        continue;
162594659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      if (!LookedUpGetterSetter) {
162694659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        GetterMethod = IMPDecl->getInstanceMethod(Property->getGetterName());
162794659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        SetterMethod = IMPDecl->getInstanceMethod(Property->getSetterName());
162894659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        LookedUpGetterSetter = true;
162994659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      }
16309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if ((GetterMethod && !SetterMethod) || (!GetterMethod && SetterMethod)) {
16319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        SourceLocation MethodLoc =
16329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          (GetterMethod ? GetterMethod->getLocation()
16339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                        : SetterMethod->getLocation());
16349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Diag(MethodLoc, diag::warn_atomic_property_rule)
16357d65f6965d215f4cb2eb0738ee1b9024e5ab5bbaFariborz Jahanian          << Property->getIdentifier() << (GetterMethod != 0)
16367d65f6965d215f4cb2eb0738ee1b9024e5ab5bbaFariborz Jahanian          << (SetterMethod != 0);
163777bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        // fixit stuff.
163877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        if (!AttributesAsWritten) {
163977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          if (Property->getLParenLoc().isValid()) {
164077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            // @property () ... case.
164177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            SourceRange PropSourceRange(Property->getAtLoc(),
164277bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                        Property->getLParenLoc());
164377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            Diag(Property->getLocation(), diag::note_atomic_property_fixup_suggest) <<
164477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian              FixItHint::CreateReplacement(PropSourceRange, "@property (nonatomic");
164577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          }
164677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          else {
164777bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            //@property id etc.
164877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            SourceLocation endLoc =
164977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian              Property->getTypeSourceInfo()->getTypeLoc().getBeginLoc();
165077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            endLoc = endLoc.getLocWithOffset(-1);
165177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            SourceRange PropSourceRange(Property->getAtLoc(), endLoc);
165277bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            Diag(Property->getLocation(), diag::note_atomic_property_fixup_suggest) <<
165377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian              FixItHint::CreateReplacement(PropSourceRange, "@property (nonatomic) ");
165477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          }
165577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        }
165677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        else if (!(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_atomic)) {
165777bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          // @property () ... case.
165877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          SourceLocation endLoc = Property->getLParenLoc();
165977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          SourceRange PropSourceRange(Property->getAtLoc(), endLoc);
166077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          Diag(Property->getLocation(), diag::note_atomic_property_fixup_suggest) <<
166177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian           FixItHint::CreateReplacement(PropSourceRange, "@property (nonatomic, ");
166277bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        }
166377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        else
166477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          Diag(MethodLoc, diag::note_atomic_property_fixup_suggest);
16659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Diag(Property->getLocation(), diag::note_property_declare);
16669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      }
16679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
16689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
16699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
16709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1671f85e193739c953358c865005855253af4f68a497John McCallvoid Sema::DiagnoseOwningPropertyGetterSynthesis(const ObjCImplementationDecl *D) {
16724e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().getGC() == LangOptions::GCOnly)
1673f85e193739c953358c865005855253af4f68a497John McCall    return;
1674f85e193739c953358c865005855253af4f68a497John McCall
1675f85e193739c953358c865005855253af4f68a497John McCall  for (ObjCImplementationDecl::propimpl_iterator
1676f85e193739c953358c865005855253af4f68a497John McCall         i = D->propimpl_begin(), e = D->propimpl_end(); i != e; ++i) {
1677262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie    ObjCPropertyImplDecl *PID = &*i;
1678f85e193739c953358c865005855253af4f68a497John McCall    if (PID->getPropertyImplementation() != ObjCPropertyImplDecl::Synthesize)
1679f85e193739c953358c865005855253af4f68a497John McCall      continue;
1680f85e193739c953358c865005855253af4f68a497John McCall
1681f85e193739c953358c865005855253af4f68a497John McCall    const ObjCPropertyDecl *PD = PID->getPropertyDecl();
1682831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    if (PD && !PD->hasAttr<NSReturnsNotRetainedAttr>() &&
1683831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian        !D->getInstanceMethod(PD->getGetterName())) {
1684f85e193739c953358c865005855253af4f68a497John McCall      ObjCMethodDecl *method = PD->getGetterMethodDecl();
1685f85e193739c953358c865005855253af4f68a497John McCall      if (!method)
1686f85e193739c953358c865005855253af4f68a497John McCall        continue;
1687f85e193739c953358c865005855253af4f68a497John McCall      ObjCMethodFamily family = method->getMethodFamily();
1688f85e193739c953358c865005855253af4f68a497John McCall      if (family == OMF_alloc || family == OMF_copy ||
1689f85e193739c953358c865005855253af4f68a497John McCall          family == OMF_mutableCopy || family == OMF_new) {
16904e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        if (getLangOpts().ObjCAutoRefCount)
1691f85e193739c953358c865005855253af4f68a497John McCall          Diag(PID->getLocation(), diag::err_ownin_getter_rule);
1692f85e193739c953358c865005855253af4f68a497John McCall        else
1693920c9c1155c118ac40e5209d4b757b9f54186339Ted Kremenek          Diag(PID->getLocation(), diag::warn_owning_getter_rule);
1694f85e193739c953358c865005855253af4f68a497John McCall        Diag(PD->getLocation(), diag::note_property_declare);
1695f85e193739c953358c865005855253af4f68a497John McCall      }
1696f85e193739c953358c865005855253af4f68a497John McCall    }
1697f85e193739c953358c865005855253af4f68a497John McCall  }
1698f85e193739c953358c865005855253af4f68a497John McCall}
1699f85e193739c953358c865005855253af4f68a497John McCall
17005de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall/// AddPropertyAttrs - Propagates attributes from a property to the
17015de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall/// implicitly-declared getter or setter for that property.
17025de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCallstatic void AddPropertyAttrs(Sema &S, ObjCMethodDecl *PropertyMethod,
17035de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall                             ObjCPropertyDecl *Property) {
17045de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall  // Should we just clone all attributes over?
17050a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor  for (Decl::attr_iterator A = Property->attr_begin(),
17060a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor                        AEnd = Property->attr_end();
17070a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor       A != AEnd; ++A) {
17080a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor    if (isa<DeprecatedAttr>(*A) ||
17090a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor        isa<UnavailableAttr>(*A) ||
17100a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor        isa<AvailabilityAttr>(*A))
17110a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor      PropertyMethod->addAttr((*A)->clone(S.Context));
17120a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor  }
17135de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall}
17145de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
17159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// ProcessPropertyDecl - Make sure that any user-defined setter/getter methods
17169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// have the property type and issue diagnostics if they don't.
17179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// Also synthesize a getter/setter method if none exist (and update the
17189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// appropriate lookup tables. FIXME: Should reconsider if adding synthesized
17199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// methods is the "right" thing to do.
17209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::ProcessPropertyDecl(ObjCPropertyDecl *property,
17218254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               ObjCContainerDecl *CD,
17228254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               ObjCPropertyDecl *redeclaredProperty,
17238254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               ObjCContainerDecl *lexicalDC) {
17248254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek
17259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCMethodDecl *GetterMethod, *SetterMethod;
17269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
17279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  GetterMethod = CD->getInstanceMethod(property->getGetterName());
17289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  SetterMethod = CD->getInstanceMethod(property->getSetterName());
17299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  DiagnosePropertyAccessorMismatch(property, GetterMethod,
17309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                   property->getLocation());
17319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
17329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (SetterMethod) {
17339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCPropertyDecl::PropertyAttributeKind CAttr =
17349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      property->getPropertyAttributes();
17359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if ((!(CAttr & ObjCPropertyDecl::OBJC_PR_readonly)) &&
17369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Context.getCanonicalType(SetterMethod->getResultType()) !=
17379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          Context.VoidTy)
17389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Diag(SetterMethod->getLocation(), diag::err_setter_type_void);
17399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (SetterMethod->param_size() != 1 ||
17402aac0c9f8bc4dca3f883f429af8c1f57132d62d0Fariborz Jahanian        !Context.hasSameUnqualifiedType(
1741bb13c320fff4bc4b3536e62626c97d7b055c6113Fariborz Jahanian          (*SetterMethod->param_begin())->getType().getNonReferenceType(),
1742bb13c320fff4bc4b3536e62626c97d7b055c6113Fariborz Jahanian          property->getType().getNonReferenceType())) {
17439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Diag(property->getLocation(),
17449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           diag::warn_accessor_property_type_mismatch)
17459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        << property->getDeclName()
17469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        << SetterMethod->getSelector();
17479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Diag(SetterMethod->getLocation(), diag::note_declared_at);
17489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
17499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
17509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
17519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Synthesize getter/setter methods if none exist.
17529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Find the default getter and if one not found, add one.
17539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // FIXME: The synthesized property we set here is misleading. We almost always
17549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // synthesize these methods unless the user explicitly provided prototypes
17559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // (which is odd, but allowed). Sema should be typechecking that the
17569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // declarations jive in that situation (which it is not currently).
17579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!GetterMethod) {
17589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // No instance method of same name as property getter name was found.
17599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Declare a getter method and add it to the list of methods
17609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // for this class.
1761a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek    SourceLocation Loc = redeclaredProperty ?
1762a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek      redeclaredProperty->getLocation() :
1763a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek      property->getLocation();
1764a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek
1765a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek    GetterMethod = ObjCMethodDecl::Create(Context, Loc, Loc,
1766a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek                             property->getGetterName(),
176775cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                             property->getType(), 0, CD, /*isInstance=*/true,
176875cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                             /*isVariadic=*/false, /*isSynthesized=*/true,
176975cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                             /*isImplicitlyDeclared=*/true, /*isDefined=*/false,
17709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                             (property->getPropertyImplementation() ==
17719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                              ObjCPropertyDecl::Optional) ?
17729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                             ObjCMethodDecl::Optional :
17739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                             ObjCMethodDecl::Required);
17749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    CD->addDecl(GetterMethod);
17755de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
17765de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall    AddPropertyAttrs(*this, GetterMethod, property);
17775de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
177823173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek    // FIXME: Eventually this shouldn't be needed, as the lexical context
177923173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek    // and the real context should be the same.
1780a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek    if (lexicalDC)
178123173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      GetterMethod->setLexicalDeclContext(lexicalDC);
1782831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    if (property->hasAttr<NSReturnsNotRetainedAttr>())
1783831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian      GetterMethod->addAttr(
1784831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian        ::new (Context) NSReturnsNotRetainedAttr(Loc, Context));
17859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  } else
17869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // A user declared getter will be synthesize when @synthesize of
17879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // the property with the same name is seen in the @implementation
17889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    GetterMethod->setSynthesized(true);
17899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  property->setGetterMethodDecl(GetterMethod);
17909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
17919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Skip setter if property is read-only.
17929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!property->isReadOnly()) {
17939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Find the default setter and if one not found, add one.
17949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!SetterMethod) {
17959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // No instance method of same name as property setter name was found.
17969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Declare a setter method and add it to the list of methods
17979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // for this class.
17988254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek      SourceLocation Loc = redeclaredProperty ?
17998254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek        redeclaredProperty->getLocation() :
18008254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek        property->getLocation();
18018254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek
18028254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek      SetterMethod =
1803491306a83c4f0f49f95a3bcbca8580cb98a91c7aArgyrios Kyrtzidis        ObjCMethodDecl::Create(Context, Loc, Loc,
18048254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               property->getSetterName(), Context.VoidTy, 0,
180575cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               CD, /*isInstance=*/true, /*isVariadic=*/false,
180675cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               /*isSynthesized=*/true,
180775cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               /*isImplicitlyDeclared=*/true,
180875cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               /*isDefined=*/false,
18099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                               (property->getPropertyImplementation() ==
18109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                ObjCPropertyDecl::Optional) ?
18118254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                                ObjCMethodDecl::Optional :
18128254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                                ObjCMethodDecl::Required);
18138254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek
18149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Invent the arguments for the setter. We don't bother making a
18159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // nice name for the argument.
1816ff676cb48fe8bf7be2feaa251dc7c5fb15af4730Abramo Bagnara      ParmVarDecl *Argument = ParmVarDecl::Create(Context, SetterMethod,
1817ff676cb48fe8bf7be2feaa251dc7c5fb15af4730Abramo Bagnara                                                  Loc, Loc,
18189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                                  property->getIdentifier(),
1819f85e193739c953358c865005855253af4f68a497John McCall                                    property->getType().getUnqualifiedType(),
18209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                                  /*TInfo=*/0,
1821d931b086984257de68868a64a235c2b4b34003fbJohn McCall                                                  SC_None,
1822d931b086984257de68868a64a235c2b4b34003fbJohn McCall                                                  SC_None,
18239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                                  0);
1824491306a83c4f0f49f95a3bcbca8580cb98a91c7aArgyrios Kyrtzidis      SetterMethod->setMethodParams(Context, Argument,
1825491306a83c4f0f49f95a3bcbca8580cb98a91c7aArgyrios Kyrtzidis                                    ArrayRef<SourceLocation>());
18265de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
18275de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall      AddPropertyAttrs(*this, SetterMethod, property);
18285de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
18299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      CD->addDecl(SetterMethod);
183023173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      // FIXME: Eventually this shouldn't be needed, as the lexical context
183123173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      // and the real context should be the same.
18328254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek      if (lexicalDC)
183323173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek        SetterMethod->setLexicalDeclContext(lexicalDC);
18349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    } else
18359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // A user declared setter will be synthesize when @synthesize of
18369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // the property with the same name is seen in the @implementation
18379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      SetterMethod->setSynthesized(true);
18389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    property->setSetterMethodDecl(SetterMethod);
18399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
18409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Add any synthesized methods to the global pool. This allows us to
18419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // handle the following, which is supported by GCC (and part of the design).
18429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //
18439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // @interface Foo
18449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // @property double bar;
18459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // @end
18469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //
18479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // void thisIsUnfortunate() {
18489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //   id foo;
18499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //   double bar = [foo bar];
18509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // }
18519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //
18529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (GetterMethod)
18539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    AddInstanceMethodToGlobalPool(GetterMethod);
18549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (SetterMethod)
18559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    AddInstanceMethodToGlobalPool(SetterMethod);
1856e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis
1857e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  ObjCInterfaceDecl *CurrentClass = dyn_cast<ObjCInterfaceDecl>(CD);
1858e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  if (!CurrentClass) {
1859e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    if (ObjCCategoryDecl *Cat = dyn_cast<ObjCCategoryDecl>(CD))
1860e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis      CurrentClass = Cat->getClassInterface();
1861e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    else if (ObjCImplDecl *Impl = dyn_cast<ObjCImplDecl>(CD))
1862e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis      CurrentClass = Impl->getClassInterface();
1863e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  }
1864e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  if (GetterMethod)
1865e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    CheckObjCMethodOverrides(GetterMethod, CurrentClass, Sema::RTC_Unknown);
1866e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  if (SetterMethod)
1867e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    CheckObjCMethodOverrides(SetterMethod, CurrentClass, Sema::RTC_Unknown);
18689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
18699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1870d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallvoid Sema::CheckObjCPropertyAttributes(Decl *PDecl,
18719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                       SourceLocation Loc,
18729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                       unsigned &Attributes) {
18739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // FIXME: Improve the reported location.
1874f85e193739c953358c865005855253af4f68a497John McCall  if (!PDecl || PDecl->isInvalidDecl())
18755fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek    return;
18765fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek
18775fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek  ObjCPropertyDecl *PropertyDecl = cast<ObjCPropertyDecl>(PDecl);
1878842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  QualType PropertyTy = PropertyDecl->getType();
18799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
18804e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().ObjCAutoRefCount &&
1881015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
1882015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      PropertyTy->isObjCRetainableType()) {
1883015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // 'readonly' property with no obvious lifetime.
1884015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // its life time will be determined by its backing ivar.
1885015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    unsigned rel = (ObjCDeclSpec::DQ_PR_unsafe_unretained |
1886015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_copy |
1887015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_retain |
1888015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_strong |
1889015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_weak |
1890015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_assign);
1891015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    if ((Attributes & rel) == 0)
1892015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      return;
1893015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  }
1894015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian
18959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // readonly and readwrite/assign/retain/copy conflict.
18969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
189728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      (Attributes & (ObjCDeclSpec::DQ_PR_readwrite |
189828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                     ObjCDeclSpec::DQ_PR_assign |
1899f85e193739c953358c865005855253af4f68a497John McCall                     ObjCDeclSpec::DQ_PR_unsafe_unretained |
190028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                     ObjCDeclSpec::DQ_PR_copy |
1901f85e193739c953358c865005855253af4f68a497John McCall                     ObjCDeclSpec::DQ_PR_retain |
1902f85e193739c953358c865005855253af4f68a497John McCall                     ObjCDeclSpec::DQ_PR_strong))) {
190328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    const char * which = (Attributes & ObjCDeclSpec::DQ_PR_readwrite) ?
190428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                          "readwrite" :
190528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                         (Attributes & ObjCDeclSpec::DQ_PR_assign) ?
190628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                          "assign" :
1907f85e193739c953358c865005855253af4f68a497John McCall                         (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) ?
1908f85e193739c953358c865005855253af4f68a497John McCall                          "unsafe_unretained" :
190928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                         (Attributes & ObjCDeclSpec::DQ_PR_copy) ?
191028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                          "copy" : "retain";
19119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
191228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(Loc, (Attributes & (ObjCDeclSpec::DQ_PR_readwrite)) ?
191328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                 diag::err_objc_property_attr_mutually_exclusive :
191428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                 diag::warn_objc_property_attr_mutually_exclusive)
191528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << "readonly" << which;
191628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
19179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
191828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Check for copy or retain on non-object types.
1919f85e193739c953358c865005855253af4f68a497John McCall  if ((Attributes & (ObjCDeclSpec::DQ_PR_weak | ObjCDeclSpec::DQ_PR_copy |
1920f85e193739c953358c865005855253af4f68a497John McCall                    ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong)) &&
1921f85e193739c953358c865005855253af4f68a497John McCall      !PropertyTy->isObjCRetainableType() &&
1922842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian      !PropertyDecl->getAttr<ObjCNSObjectAttr>()) {
192328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(Loc, diag::err_objc_property_requires_object)
1924f85e193739c953358c865005855253af4f68a497John McCall      << (Attributes & ObjCDeclSpec::DQ_PR_weak ? "weak" :
1925f85e193739c953358c865005855253af4f68a497John McCall          Attributes & ObjCDeclSpec::DQ_PR_copy ? "copy" : "retain (or strong)");
1926f85e193739c953358c865005855253af4f68a497John McCall    Attributes &= ~(ObjCDeclSpec::DQ_PR_weak   | ObjCDeclSpec::DQ_PR_copy |
1927f85e193739c953358c865005855253af4f68a497John McCall                    ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong);
1928977ea7864a50eee39f6bef8a5a79154ac9ba4b3eJohn McCall    PropertyDecl->setInvalidDecl();
192928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
19309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
193128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Check for more than one of { assign, copy, retain }.
193228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_assign) {
193328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
193428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
193528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << "assign" << "copy";
193628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Attributes &= ~ObjCDeclSpec::DQ_PR_copy;
193728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
193828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
193928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
194028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << "assign" << "retain";
194128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
194228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
1943f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
1944f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1945f85e193739c953358c865005855253af4f68a497John McCall        << "assign" << "strong";
1946f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
1947f85e193739c953358c865005855253af4f68a497John McCall    }
19484e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount  &&
1949f85e193739c953358c865005855253af4f68a497John McCall        (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1950f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1951f85e193739c953358c865005855253af4f68a497John McCall        << "assign" << "weak";
1952f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
1953f85e193739c953358c865005855253af4f68a497John McCall    }
1954f85e193739c953358c865005855253af4f68a497John McCall  } else if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) {
1955f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
1956f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1957f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "copy";
1958f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_copy;
1959f85e193739c953358c865005855253af4f68a497John McCall    }
1960f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
1961f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1962f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "retain";
1963f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
1964f85e193739c953358c865005855253af4f68a497John McCall    }
1965f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
1966f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1967f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "strong";
1968f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
1969f85e193739c953358c865005855253af4f68a497John McCall    }
19704e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount  &&
1971f85e193739c953358c865005855253af4f68a497John McCall        (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1972f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1973f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "weak";
1974f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
1975f85e193739c953358c865005855253af4f68a497John McCall    }
197628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
197728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
197828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
197928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << "copy" << "retain";
198028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
198128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
1982f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
1983f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1984f85e193739c953358c865005855253af4f68a497John McCall        << "copy" << "strong";
1985f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
1986f85e193739c953358c865005855253af4f68a497John McCall    }
1987f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_weak) {
1988f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1989f85e193739c953358c865005855253af4f68a497John McCall        << "copy" << "weak";
1990f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
1991f85e193739c953358c865005855253af4f68a497John McCall    }
1992f85e193739c953358c865005855253af4f68a497John McCall  }
1993f85e193739c953358c865005855253af4f68a497John McCall  else if ((Attributes & ObjCDeclSpec::DQ_PR_retain) &&
1994f85e193739c953358c865005855253af4f68a497John McCall           (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1995f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1996f85e193739c953358c865005855253af4f68a497John McCall        << "retain" << "weak";
1997528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
1998f85e193739c953358c865005855253af4f68a497John McCall  }
1999f85e193739c953358c865005855253af4f68a497John McCall  else if ((Attributes & ObjCDeclSpec::DQ_PR_strong) &&
2000f85e193739c953358c865005855253af4f68a497John McCall           (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
2001f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2002f85e193739c953358c865005855253af4f68a497John McCall        << "strong" << "weak";
2003f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
200428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
20059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
20069d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian  if ((Attributes & ObjCDeclSpec::DQ_PR_atomic) &&
20079d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)) {
20089d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
20099d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian        << "atomic" << "nonatomic";
20109d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian      Attributes &= ~ObjCDeclSpec::DQ_PR_atomic;
20119d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian  }
20129d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian
201328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Warn if user supplied no assignment attribute, property is
201428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // readwrite, and this is an object type.
201528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (!(Attributes & (ObjCDeclSpec::DQ_PR_assign | ObjCDeclSpec::DQ_PR_copy |
2016f85e193739c953358c865005855253af4f68a497John McCall                      ObjCDeclSpec::DQ_PR_unsafe_unretained |
2017f85e193739c953358c865005855253af4f68a497John McCall                      ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong |
2018f85e193739c953358c865005855253af4f68a497John McCall                      ObjCDeclSpec::DQ_PR_weak)) &&
201928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      PropertyTy->isObjCObjectPointerType()) {
20204e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      if (getLangOpts().ObjCAutoRefCount)
2021bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian        // With arc,  @property definitions should default to (strong) when
2022f21a92d35953adb038ffbe37cd1a4083e0df0ec0Fariborz Jahanian        // not specified; including when property is 'readonly'.
2023bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian        PropertyDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
2024f21a92d35953adb038ffbe37cd1a4083e0df0ec0Fariborz Jahanian      else if (!(Attributes & ObjCDeclSpec::DQ_PR_readonly)) {
20259f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        bool isAnyClassTy =
20269f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian          (PropertyTy->isObjCClassType() ||
20279f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian           PropertyTy->isObjCQualifiedClassType());
20289f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        // In non-gc, non-arc mode, 'Class' is treated as a 'void *' no need to
20299f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        // issue any warning.
20304e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        if (isAnyClassTy && getLangOpts().getGC() == LangOptions::NonGC)
20319f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian          ;
20329f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        else {
2033bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian          // Skip this warning in gc-only mode.
20344e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          if (getLangOpts().getGC() != LangOptions::GCOnly)
2035bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian            Diag(Loc, diag::warn_objc_property_no_assignment_attribute);
20360a68dc7f04be1542ce249ff4adb169453698ad91Argyrios Kyrtzidis
2037bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian          // If non-gc code warn that this is likely inappropriate.
20384e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          if (getLangOpts().getGC() == LangOptions::NonGC)
2039bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian            Diag(Loc, diag::warn_objc_property_default_assign_on_object);
20409f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        }
2041bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian      }
20429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
204328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // FIXME: Implement warning dependent on NSCopying being
204428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // implemented. See also:
204528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // <rdar://5168496&4855821&5607453&5096644&4947311&5698469&4947014&5168496>
204628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // (please trim this list while you are at it).
204728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
20489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
204928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (!(Attributes & ObjCDeclSpec::DQ_PR_copy)
20502b77cb803c76d01d2d10a58cec9334e2090cd7c3Fariborz Jahanian      &&!(Attributes & ObjCDeclSpec::DQ_PR_readonly)
20514e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      && getLangOpts().getGC() == LangOptions::GCOnly
205228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      && PropertyTy->isBlockPointerType())
205328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(Loc, diag::warn_objc_property_copy_missing_on_block);
20544e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  else if (getLangOpts().ObjCAutoRefCount &&
2055528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           (Attributes & ObjCDeclSpec::DQ_PR_retain) &&
2056528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           !(Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
2057528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           !(Attributes & ObjCDeclSpec::DQ_PR_strong) &&
2058528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           PropertyTy->isBlockPointerType())
2059528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian      Diag(Loc, diag::warn_objc_property_retain_of_block);
206048a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian
206148a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian  if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
206248a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_setter))
206348a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian    Diag(Loc, diag::warn_objc_readonly_property_has_setter);
206448a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian
20659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
2066