SemaObjCProperty.cpp revision d9f95b35dc4fc168a167a9b9b61a5f9e1d0462ba
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)) {
46d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall    return Qualifiers::OCL_Strong;
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
105dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanianstatic unsigned deduceWeakPropertyFromType(Sema &S, QualType T) {
106dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian  if ((S.getLangOpts().getGC() != LangOptions::NonGC &&
107dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian       T.isObjCGCWeak()) ||
108dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian      (S.getLangOpts().ObjCAutoRefCount &&
109dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian       T.getObjCLifetime() == Qualifiers::OCL_Weak))
110dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian    return ObjCDeclSpec::DQ_PR_weak;
111dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian  return 0;
112dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian}
113dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian
114d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallDecl *Sema::ActOnProperty(Scope *S, SourceLocation AtLoc,
11577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                          SourceLocation LParenLoc,
116d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          FieldDeclarator &FD,
117d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          ObjCDeclSpec &ODS,
118d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          Selector GetterSel,
119d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          Selector SetterSel,
120d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                          bool *isOverridingProperty,
1214a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                          tok::ObjCKeywordKind MethodImplKind,
1224a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                          DeclContext *lexicalDC) {
12328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  unsigned Attributes = ODS.getPropertyAttributes();
124f85e193739c953358c865005855253af4f68a497John McCall  TypeSourceInfo *TSI = GetTypeForDeclarator(FD.D, S);
125f85e193739c953358c865005855253af4f68a497John McCall  QualType T = TSI->getType();
126dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian  Attributes |= deduceWeakPropertyFromType(*this, T);
127dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian
12828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  bool isReadWrite = ((Attributes & ObjCDeclSpec::DQ_PR_readwrite) ||
12928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                      // default is readwrite!
13028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                      !(Attributes & ObjCDeclSpec::DQ_PR_readonly));
13128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // property is defaulted to 'assign' if it is readwrite and is
13228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // not retain or copy
13328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  bool isAssign = ((Attributes & ObjCDeclSpec::DQ_PR_assign) ||
13428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                   (isReadWrite &&
13528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                    !(Attributes & ObjCDeclSpec::DQ_PR_retain) &&
136f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_strong) &&
137f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_copy) &&
138f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) &&
139f85e193739c953358c865005855253af4f68a497John McCall                    !(Attributes & ObjCDeclSpec::DQ_PR_weak)));
14014086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian
141e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Proceed with constructing the ObjCPropertDecls.
142a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian  ObjCContainerDecl *ClassDecl = cast<ObjCContainerDecl>(CurContext);
14328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (ObjCCategoryDecl *CDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl))
144ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian    if (CDecl->IsClassExtension()) {
14577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian      Decl *Res = HandlePropertyInClassExtension(S, AtLoc, LParenLoc,
146ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           FD, GetterSel, SetterSel,
147ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           isAssign, isReadWrite,
148ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           Attributes,
149dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                           ODS.getPropertyAttributes(),
150ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           isOverridingProperty, TSI,
151ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian                                           MethodImplKind);
152f85e193739c953358c865005855253af4f68a497John McCall      if (Res) {
153cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        CheckObjCPropertyAttributes(Res, AtLoc, Attributes, false);
1544e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        if (getLangOpts().ObjCAutoRefCount)
155f85e193739c953358c865005855253af4f68a497John McCall          checkARCPropertyDecl(*this, cast<ObjCPropertyDecl>(Res));
156f85e193739c953358c865005855253af4f68a497John McCall      }
157abd56c816e9164b17bb3e7154a511b0c9896ffdbDmitri Gribenko      ActOnDocumentableDecl(Res);
158ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian      return Res;
159ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian    }
160ae415dc21ee6402cee1675ec8bb965a24f9e5b6bFariborz Jahanian
16177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian  ObjCPropertyDecl *Res = CreatePropertyDecl(S, ClassDecl, AtLoc, LParenLoc, FD,
162f85e193739c953358c865005855253af4f68a497John McCall                                             GetterSel, SetterSel,
163f85e193739c953358c865005855253af4f68a497John McCall                                             isAssign, isReadWrite,
164dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                             Attributes,
165dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                             ODS.getPropertyAttributes(),
166dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                             TSI, MethodImplKind);
1674a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek  if (lexicalDC)
1684a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek    Res->setLexicalDeclContext(lexicalDC);
1694a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek
170842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  // Validate the attributes on the @property.
171cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  CheckObjCPropertyAttributes(Res, AtLoc, Attributes,
172cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                              (isa<ObjCInterfaceDecl>(ClassDecl) ||
173cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                               isa<ObjCProtocolDecl>(ClassDecl)));
174f85e193739c953358c865005855253af4f68a497John McCall
1754e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().ObjCAutoRefCount)
176f85e193739c953358c865005855253af4f68a497John McCall    checkARCPropertyDecl(*this, Res);
177f85e193739c953358c865005855253af4f68a497John McCall
178abd56c816e9164b17bb3e7154a511b0c9896ffdbDmitri Gribenko  ActOnDocumentableDecl(Res);
179842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  return Res;
180e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek}
181e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
182b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidisstatic ObjCPropertyDecl::PropertyAttributeKind
183b98ffded10453d80369951f33f3892f35d747c95Argyrios KyrtzidismakePropertyAttributesAsWritten(unsigned Attributes) {
184b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  unsigned attributesAsWritten = 0;
185b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
186b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_readonly;
187b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_readwrite)
188b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_readwrite;
189b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_getter)
190b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_getter;
191b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_setter)
192b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_setter;
193b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_assign)
194b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_assign;
195b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_retain)
196b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_retain;
197b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_strong)
198b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_strong;
199b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_weak)
200b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_weak;
201b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_copy)
202b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_copy;
203b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
204b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_unsafe_unretained;
205b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)
206b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_nonatomic;
207b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  if (Attributes & ObjCDeclSpec::DQ_PR_atomic)
208b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis    attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_atomic;
209b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis
210b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  return (ObjCPropertyDecl::PropertyAttributeKind)attributesAsWritten;
211b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis}
212b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis
2135bf0e3522fac400cf7bf49a775554f0fc942b336Fariborz Jahanianstatic bool LocPropertyAttribute( ASTContext &Context, const char *attrName,
214edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian                                 SourceLocation LParenLoc, SourceLocation &Loc) {
215edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  if (LParenLoc.isMacroID())
216edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    return false;
217edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian
218edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  SourceManager &SM = Context.getSourceManager();
219edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  std::pair<FileID, unsigned> locInfo = SM.getDecomposedLoc(LParenLoc);
220edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  // Try to load the file buffer.
221edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  bool invalidTemp = false;
222edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  StringRef file = SM.getBufferData(locInfo.first, &invalidTemp);
223edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  if (invalidTemp)
224edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    return false;
225edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  const char *tokenBegin = file.data() + locInfo.second;
226edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian
227edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  // Lex from the start of the given location.
228edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  Lexer lexer(SM.getLocForStartOfFile(locInfo.first),
229edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian              Context.getLangOpts(),
230edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian              file.begin(), tokenBegin, file.end());
231edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  Token Tok;
232edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  do {
233edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    lexer.LexFromRawLexer(Tok);
234edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    if (Tok.is(tok::raw_identifier) &&
235edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        StringRef(Tok.getRawIdentifierData(), Tok.getLength()) == attrName) {
236edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      Loc = Tok.getLocation();
237edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      return true;
238edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian    }
239edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  } while (Tok.isNot(tok::r_paren));
240edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian  return false;
241edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian
2422b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian}
2432b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian
244d9f95b35dc4fc168a167a9b9b61a5f9e1d0462baFariborz Jahanianstatic unsigned getOwnershipRule(unsigned attr) {
245dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian  return attr & (ObjCPropertyDecl::OBJC_PR_assign |
246dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian                 ObjCPropertyDecl::OBJC_PR_retain |
247dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian                 ObjCPropertyDecl::OBJC_PR_copy   |
248dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian                 ObjCPropertyDecl::OBJC_PR_weak   |
249dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian                 ObjCPropertyDecl::OBJC_PR_strong |
250dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian                 ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
251dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian}
252dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian
253d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallDecl *
254a28948f34817476d02412fa204cae038e228c827Fariborz JahanianSema::HandlePropertyInClassExtension(Scope *S,
25577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                     SourceLocation AtLoc,
25677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                     SourceLocation LParenLoc,
25777bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                     FieldDeclarator &FD,
258e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     Selector GetterSel, Selector SetterSel,
259e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     const bool isAssign,
260e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     const bool isReadWrite,
261e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     const unsigned Attributes,
262dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                     const unsigned AttributesAsWritten,
263e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     bool *isOverridingProperty,
26483a230c83a54190366138c1a4f4310ef838b88fcJohn McCall                                     TypeSourceInfo *T,
265e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                     tok::ObjCKeywordKind MethodImplKind) {
26658a764940df0cd41767367eb2f4fced6f374176bFariborz Jahanian  ObjCCategoryDecl *CDecl = cast<ObjCCategoryDecl>(CurContext);
267e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Diagnose if this property is already in continuation class.
268a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian  DeclContext *DC = CurContext;
269e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  IdentifierInfo *PropertyId = FD.D.getIdentifier();
2702a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian  ObjCInterfaceDecl *CCPrimary = CDecl->getClassInterface();
2712a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian
2722a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian  if (CCPrimary)
2732a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    // Check for duplicate declaration of this property in current and
2742a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    // other class extensions.
2752a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    for (const ObjCCategoryDecl *ClsExtDecl =
2762a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian         CCPrimary->getFirstClassExtension();
2772a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian         ClsExtDecl; ClsExtDecl = ClsExtDecl->getNextClassExtension()) {
2782a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian      if (ObjCPropertyDecl *prevDecl =
2792a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian          ObjCPropertyDecl::findPropertyDecl(ClsExtDecl, PropertyId)) {
2802a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian        Diag(AtLoc, diag::err_duplicate_property);
2812a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian        Diag(prevDecl->getLocation(), diag::note_property_declare);
2822a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian        return 0;
2832a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian      }
2842a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian    }
2852a2891410b02d146b779cdb29eeb9b6d1a9fa030Fariborz Jahanian
286e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Create a new ObjCPropertyDecl with the DeclContext being
287e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // the class extension.
28888f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  // FIXME. We should really be using CreatePropertyDecl for this.
289e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  ObjCPropertyDecl *PDecl =
290e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    ObjCPropertyDecl::Create(Context, DC, FD.D.getIdentifierLoc(),
29177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                             PropertyId, AtLoc, LParenLoc, T);
292b98ffded10453d80369951f33f3892f35d747c95Argyrios Kyrtzidis  PDecl->setPropertyAttributesAsWritten(
293dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                          makePropertyAttributesAsWritten(AttributesAsWritten));
29422f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian  if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
29522f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly);
29622f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian  if (Attributes & ObjCDeclSpec::DQ_PR_readwrite)
29722f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite);
29888f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  // Set setter/getter selector name. Needed later.
29988f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  PDecl->setGetterName(GetterSel);
30088f5e9be350f4b107f8665183a6d441874e0fcc7Fariborz Jahanian  PDecl->setSetterName(SetterSel);
30191ae6b43b3a294243698cdb544dd7a18efa4c7a8Douglas Gregor  ProcessDeclAttributes(S, PDecl, FD.D);
302e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  DC->addDecl(PDecl);
303e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
304e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // We need to look in the @interface to see if the @property was
305e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // already declared.
306e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  if (!CCPrimary) {
307e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    Diag(CDecl->getLocation(), diag::err_continuation_class);
308e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    *isOverridingProperty = true;
309d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall    return 0;
310e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  }
311e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
312e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Find the property in continuation class's primary class only.
313e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  ObjCPropertyDecl *PIDecl =
314e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    CCPrimary->FindPropertyVisibleInPrimaryClass(PropertyId);
315e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
316e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  if (!PIDecl) {
317e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // No matching property found in the primary class. Just fall thru
318e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // and add property to continuation class's primary class.
319d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    ObjCPropertyDecl *PrimaryPDecl =
32077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian      CreatePropertyDecl(S, CCPrimary, AtLoc, LParenLoc,
321e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                         FD, GetterSel, SetterSel, isAssign, isReadWrite,
322dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                         Attributes,AttributesAsWritten, T, MethodImplKind, DC);
323e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
324e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // A case of continuation class adding a new property in the class. This
325e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // is not what it was meant for. However, gcc supports it and so should we.
326e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    // Make sure setter/getters are declared here.
327d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    ProcessPropertyDecl(PrimaryPDecl, CCPrimary, /* redeclaredProperty = */ 0,
328a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek                        /* lexicalDC = */ CDecl);
329d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    PDecl->setGetterMethodDecl(PrimaryPDecl->getGetterMethodDecl());
330d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    PDecl->setSetterMethodDecl(PrimaryPDecl->getSetterMethodDecl());
331c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis    if (ASTMutationListener *L = Context.getASTMutationListener())
332d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis      L->AddedObjCPropertyInClassExtension(PrimaryPDecl, /*OrigProp=*/0, CDecl);
333d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis    return PrimaryPDecl;
334e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  }
335e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian  if (!Context.hasSameType(PIDecl->getType(), PDecl->getType())) {
336e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    bool IncompatibleObjC = false;
337e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    QualType ConvertedType;
338ff2a0ecae1d7692dbb63aa540dba41a8c21d7ee4Fariborz Jahanian    // Relax the strict type matching for property type in continuation class.
339ff2a0ecae1d7692dbb63aa540dba41a8c21d7ee4Fariborz Jahanian    // Allow property object type of continuation class to be different as long
340ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // as it narrows the object type in its primary class property. Note that
341ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // this conversion is safe only because the wider type is for a 'readonly'
342ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // property in primary class and 'narrowed' type for a 'readwrite' property
343ad7eff2faf517779689327dc268817c2c2c8ebc4Fariborz Jahanian    // in continuation class.
344e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    if (!isa<ObjCObjectPointerType>(PIDecl->getType()) ||
345e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian        !isa<ObjCObjectPointerType>(PDecl->getType()) ||
346e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian        (!isObjCPointerConversion(PDecl->getType(), PIDecl->getType(),
347e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian                                  ConvertedType, IncompatibleObjC))
348e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian        || IncompatibleObjC) {
349e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian      Diag(AtLoc,
350e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian          diag::err_type_mismatch_continuation_class) << PDecl->getType();
351e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian      Diag(PIDecl->getLocation(), diag::note_property_declare);
352e23518342e326e3d6e9139f05384f030e2a94d4aFariborz Jahanian    }
353a4b984d8396724a8e1137c22186c558c0cb0bf3fFariborz Jahanian  }
354a4b984d8396724a8e1137c22186c558c0cb0bf3fFariborz Jahanian
355e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // The property 'PIDecl's readonly attribute will be over-ridden
356e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // with continuation class's readwrite property attribute!
35780aa1cd7973561889e51c1c152c8990a8de9c953Fariborz Jahanian  unsigned PIkind = PIDecl->getPropertyAttributesAsWritten();
358e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  if (isReadWrite && (PIkind & ObjCPropertyDecl::OBJC_PR_readonly)) {
359dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian    PIkind |= deduceWeakPropertyFromType(*this, PIDecl->getType());
360d9f95b35dc4fc168a167a9b9b61a5f9e1d0462baFariborz Jahanian    unsigned ClassExtensionMemoryModel = getOwnershipRule(Attributes);
361d9f95b35dc4fc168a167a9b9b61a5f9e1d0462baFariborz Jahanian    unsigned PrimaryClassMemoryModel = getOwnershipRule(PIkind);
362dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian    if (PrimaryClassMemoryModel && ClassExtensionMemoryModel &&
363dad633b0a0955ceb4579e9508224562ad41b6a8dFariborz Jahanian        (PrimaryClassMemoryModel != ClassExtensionMemoryModel)) {
364e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      Diag(AtLoc, diag::warn_property_attr_mismatch);
365e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      Diag(PIDecl->getLocation(), diag::note_property_declare);
366e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    }
3679944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek    DeclContext *DC = cast<DeclContext>(CCPrimary);
3689944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek    if (!ObjCPropertyDecl::findPropertyDecl(DC,
3699944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek                                 PIDecl->getDeclName().getAsIdentifierInfo())) {
370e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // Protocol is not in the primary class. Must build one for it.
371e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      ObjCDeclSpec ProtocolPropertyODS;
372e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // FIXME. Assuming that ObjCDeclSpec::ObjCPropertyAttributeKind
373e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // and ObjCPropertyDecl::PropertyAttributeKind have identical
374e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      // values.  Should consolidate both into one enum type.
375e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      ProtocolPropertyODS.
376e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      setPropertyAttributes((ObjCDeclSpec::ObjCPropertyAttributeKind)
377e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                            PIkind);
378a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian      // Must re-establish the context from class extension to primary
379a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian      // class context.
3807939418cbdf1b610a69e6adf29cd7dc8d0e9eaf8Fariborz Jahanian      ContextRAII SavedContext(*this, CCPrimary);
3817939418cbdf1b610a69e6adf29cd7dc8d0e9eaf8Fariborz Jahanian
382d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      Decl *ProtocolPtrTy =
38377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        ActOnProperty(S, AtLoc, LParenLoc, FD, ProtocolPropertyODS,
384e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                      PIDecl->getGetterName(),
385e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                      PIDecl->getSetterName(),
386a28948f34817476d02412fa204cae038e228c827Fariborz Jahanian                      isOverridingProperty,
3874a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                      MethodImplKind,
3884a2e9ea0c191176ebe996516a9cf4f93b7e932ccTed Kremenek                      /* lexicalDC = */ CDecl);
389d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      PIDecl = cast<ObjCPropertyDecl>(ProtocolPtrTy);
39028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
391e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    PIDecl->makeitReadWriteAttribute();
392e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_retain)
393e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain);
394f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong)
395f85e193739c953358c865005855253af4f68a497John McCall      PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
396e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_copy)
397e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy);
398e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    PIDecl->setSetterName(SetterSel);
399e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  } else {
400788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // Tailor the diagnostics for the common case where a readwrite
401788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // property is declared both in the @interface and the continuation.
402788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // This is a common error where the user often intended the original
403788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    // declaration to be readonly.
404788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    unsigned diag =
405788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      (Attributes & ObjCDeclSpec::DQ_PR_readwrite) &&
406788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      (PIkind & ObjCPropertyDecl::OBJC_PR_readwrite)
407788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      ? diag::err_use_continuation_class_redeclaration_readwrite
408788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek      : diag::err_use_continuation_class;
409788f489a37bf1150deb9378cec6a6eb902d0d1b2Ted Kremenek    Diag(AtLoc, diag)
410e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek      << CCPrimary->getDeclName();
411e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek    Diag(PIDecl->getLocation(), diag::note_property_declare);
412e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  }
413e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  *isOverridingProperty = true;
414e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  // Make sure setter decl is synthesized, and added to primary class's list.
4158254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek  ProcessPropertyDecl(PIDecl, CCPrimary, PDecl, CDecl);
416d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis  PDecl->setGetterMethodDecl(PIDecl->getGetterMethodDecl());
417d7c15a64ba8ebdca0dd48dd1d2f233107d34494eArgyrios Kyrtzidis  PDecl->setSetterMethodDecl(PIDecl->getSetterMethodDecl());
418c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis  if (ASTMutationListener *L = Context.getASTMutationListener())
419c80553efdc269dabd4ed0c3210555f45729ab5ddArgyrios Kyrtzidis    L->AddedObjCPropertyInClassExtension(PDecl, PIDecl, CDecl);
420d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall  return 0;
421e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek}
422e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
423e3d67bcc7b89b335dbcd097a299658057ea31097Ted KremenekObjCPropertyDecl *Sema::CreatePropertyDecl(Scope *S,
424e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           ObjCContainerDecl *CDecl,
425e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           SourceLocation AtLoc,
42677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                           SourceLocation LParenLoc,
427e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           FieldDeclarator &FD,
428e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           Selector GetterSel,
429e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           Selector SetterSel,
430e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           const bool isAssign,
431e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           const bool isReadWrite,
432e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek                                           const unsigned Attributes,
433dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                                           const unsigned AttributesAsWritten,
43483a230c83a54190366138c1a4f4310ef838b88fcJohn McCall                                           TypeSourceInfo *TInfo,
43523173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek                                           tok::ObjCKeywordKind MethodImplKind,
43623173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek                                           DeclContext *lexicalDC){
437e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  IdentifierInfo *PropertyId = FD.D.getIdentifier();
43883a230c83a54190366138c1a4f4310ef838b88fcJohn McCall  QualType T = TInfo->getType();
43928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
44028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Issue a warning if property is 'assign' as default and its object, which is
44128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // gc'able conforms to NSCopying protocol
4424e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().getGC() != LangOptions::NonGC &&
44328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      isAssign && !(Attributes & ObjCDeclSpec::DQ_PR_assign))
444c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall    if (const ObjCObjectPointerType *ObjPtrTy =
445c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall          T->getAs<ObjCObjectPointerType>()) {
446c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall      ObjCInterfaceDecl *IDecl = ObjPtrTy->getObjectType()->getInterface();
447c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall      if (IDecl)
448c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall        if (ObjCProtocolDecl* PNSCopying =
449c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall            LookupProtocol(&Context.Idents.get("NSCopying"), AtLoc))
450c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall          if (IDecl->ClassImplementsProtocol(PNSCopying, true))
451c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall            Diag(AtLoc, diag::warn_implements_nscopying) << PropertyId;
45228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
453c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall  if (T->isObjCObjectType())
45428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(FD.D.getIdentifierLoc(), diag::err_statically_allocated_object);
45528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
456e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  DeclContext *DC = cast<DeclContext>(CDecl);
45728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyDecl *PDecl = ObjCPropertyDecl::Create(Context, DC,
45828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                                                     FD.D.getIdentifierLoc(),
45977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                                     PropertyId, AtLoc, LParenLoc, TInfo);
460e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek
4619f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek  if (ObjCPropertyDecl *prevDecl =
4629f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek        ObjCPropertyDecl::findPropertyDecl(DC, PropertyId)) {
46328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(PDecl->getLocation(), diag::err_duplicate_property);
464894ae6ad081465a3b168db7e60ab3f94e3730b57Ted Kremenek    Diag(prevDecl->getLocation(), diag::note_property_declare);
46528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setInvalidDecl();
46628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
46723173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek  else {
46828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    DC->addDecl(PDecl);
46923173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek    if (lexicalDC)
47023173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      PDecl->setLexicalDeclContext(lexicalDC);
47123173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek  }
47228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
47328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (T->isArrayType() || T->isFunctionType()) {
47428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(AtLoc, diag::err_property_type) << T;
47528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setInvalidDecl();
47628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
47728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
47828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ProcessDeclAttributes(S, PDecl, FD.D);
47928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
48028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Regardless of setter/getter attribute, we save the default getter/setter
48128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // selector names in anticipation of declaration of setter/getter methods.
48228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  PDecl->setGetterName(GetterSel);
48328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  PDecl->setSetterName(SetterSel);
4840a68dc7f04be1542ce249ff4adb169453698ad91Argyrios Kyrtzidis  PDecl->setPropertyAttributesAsWritten(
485dbbdec994f5a96b5c25aaa679cd86ecabf545f7bArgyrios Kyrtzidis                          makePropertyAttributesAsWritten(AttributesAsWritten));
4860a68dc7f04be1542ce249ff4adb169453698ad91Argyrios Kyrtzidis
48728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
48828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly);
48928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
49028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_getter)
49128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_getter);
49228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
49328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_setter)
49428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_setter);
49528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
49628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (isReadWrite)
49728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite);
49828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
49928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_retain)
50028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain);
50128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
502f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_strong)
503f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
504f85e193739c953358c865005855253af4f68a497John McCall
505f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_weak)
506f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_weak);
507f85e193739c953358c865005855253af4f68a497John McCall
50828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_copy)
50928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy);
51028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
511f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
512f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
513f85e193739c953358c865005855253af4f68a497John McCall
51428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (isAssign)
51528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign);
51628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
517265941bc308d65cc270d5c4de5806f37ce405606John McCall  // In the semantic attributes, one of nonatomic or atomic is always set.
51828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)
51928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_nonatomic);
520265941bc308d65cc270d5c4de5806f37ce405606John McCall  else
52145937ae10a0f70f74508165aab4f2b63e18ea747Fariborz Jahanian    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_atomic);
52228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
523f85e193739c953358c865005855253af4f68a497John McCall  // 'unsafe_unretained' is alias for 'assign'.
524f85e193739c953358c865005855253af4f68a497John McCall  if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
525f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign);
526f85e193739c953358c865005855253af4f68a497John McCall  if (isAssign)
527f85e193739c953358c865005855253af4f68a497John McCall    PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
528f85e193739c953358c865005855253af4f68a497John McCall
52928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (MethodImplKind == tok::objc_required)
53028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyImplementation(ObjCPropertyDecl::Required);
53128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  else if (MethodImplKind == tok::objc_optional)
53228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    PDecl->setPropertyImplementation(ObjCPropertyDecl::Optional);
53328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
534e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek  return PDecl;
53528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek}
53628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
537f85e193739c953358c865005855253af4f68a497John McCallstatic void checkARCPropertyImpl(Sema &S, SourceLocation propertyImplLoc,
538f85e193739c953358c865005855253af4f68a497John McCall                                 ObjCPropertyDecl *property,
539f85e193739c953358c865005855253af4f68a497John McCall                                 ObjCIvarDecl *ivar) {
540f85e193739c953358c865005855253af4f68a497John McCall  if (property->isInvalidDecl() || ivar->isInvalidDecl()) return;
541f85e193739c953358c865005855253af4f68a497John McCall
542f85e193739c953358c865005855253af4f68a497John McCall  QualType ivarType = ivar->getType();
543f85e193739c953358c865005855253af4f68a497John McCall  Qualifiers::ObjCLifetime ivarLifetime = ivarType.getObjCLifetime();
544f85e193739c953358c865005855253af4f68a497John McCall
545265941bc308d65cc270d5c4de5806f37ce405606John McCall  // The lifetime implied by the property's attributes.
546265941bc308d65cc270d5c4de5806f37ce405606John McCall  Qualifiers::ObjCLifetime propertyLifetime =
547265941bc308d65cc270d5c4de5806f37ce405606John McCall    getImpliedARCOwnership(property->getPropertyAttributes(),
548265941bc308d65cc270d5c4de5806f37ce405606John McCall                           property->getType());
549f85e193739c953358c865005855253af4f68a497John McCall
550265941bc308d65cc270d5c4de5806f37ce405606John McCall  // We're fine if they match.
551265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (propertyLifetime == ivarLifetime) return;
552f85e193739c953358c865005855253af4f68a497John McCall
553265941bc308d65cc270d5c4de5806f37ce405606John McCall  // These aren't valid lifetimes for object ivars;  don't diagnose twice.
554265941bc308d65cc270d5c4de5806f37ce405606John McCall  if (ivarLifetime == Qualifiers::OCL_None ||
555265941bc308d65cc270d5c4de5806f37ce405606John McCall      ivarLifetime == Qualifiers::OCL_Autoreleasing)
556265941bc308d65cc270d5c4de5806f37ce405606John McCall    return;
557265941bc308d65cc270d5c4de5806f37ce405606John McCall
558d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall  // If the ivar is private, and it's implicitly __unsafe_unretained
559d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall  // becaues of its type, then pretend it was actually implicitly
560d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall  // __strong.  This is only sound because we're processing the
561d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall  // property implementation before parsing any method bodies.
562d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall  if (ivarLifetime == Qualifiers::OCL_ExplicitNone &&
563d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      propertyLifetime == Qualifiers::OCL_Strong &&
564d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      ivar->getAccessControl() == ObjCIvarDecl::Private) {
565d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall    SplitQualType split = ivarType.split();
566d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall    if (split.Quals.hasObjCLifetime()) {
567d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      assert(ivarType->isObjCARCImplicitlyUnretainedType());
568d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      split.Quals.setObjCLifetime(Qualifiers::OCL_Strong);
569d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      ivarType = S.Context.getQualifiedType(split);
570d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      ivar->setType(ivarType);
571d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall      return;
572d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall    }
573d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall  }
574d64c2eb83d7ec86faa4f2554935a977a19573f59John McCall
575265941bc308d65cc270d5c4de5806f37ce405606John McCall  switch (propertyLifetime) {
576265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_Strong:
577265941bc308d65cc270d5c4de5806f37ce405606John McCall    S.Diag(propertyImplLoc, diag::err_arc_strong_property_ownership)
578265941bc308d65cc270d5c4de5806f37ce405606John McCall      << property->getDeclName()
579265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivar->getDeclName()
580265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivarLifetime;
581265941bc308d65cc270d5c4de5806f37ce405606John McCall    break;
582265941bc308d65cc270d5c4de5806f37ce405606John McCall
583265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_Weak:
584265941bc308d65cc270d5c4de5806f37ce405606John McCall    S.Diag(propertyImplLoc, diag::error_weak_property)
585265941bc308d65cc270d5c4de5806f37ce405606John McCall      << property->getDeclName()
586265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivar->getDeclName();
587265941bc308d65cc270d5c4de5806f37ce405606John McCall    break;
588265941bc308d65cc270d5c4de5806f37ce405606John McCall
589265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_ExplicitNone:
590265941bc308d65cc270d5c4de5806f37ce405606John McCall    S.Diag(propertyImplLoc, diag::err_arc_assign_property_ownership)
591265941bc308d65cc270d5c4de5806f37ce405606John McCall      << property->getDeclName()
592265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ivar->getDeclName()
593265941bc308d65cc270d5c4de5806f37ce405606John McCall      << ((property->getPropertyAttributesAsWritten()
594265941bc308d65cc270d5c4de5806f37ce405606John McCall           & ObjCPropertyDecl::OBJC_PR_assign) != 0);
595265941bc308d65cc270d5c4de5806f37ce405606John McCall    break;
596265941bc308d65cc270d5c4de5806f37ce405606John McCall
597265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_Autoreleasing:
598265941bc308d65cc270d5c4de5806f37ce405606John McCall    llvm_unreachable("properties cannot be autoreleasing");
599265941bc308d65cc270d5c4de5806f37ce405606John McCall
600265941bc308d65cc270d5c4de5806f37ce405606John McCall  case Qualifiers::OCL_None:
601265941bc308d65cc270d5c4de5806f37ce405606John McCall    // Any other property should be ignored.
602f85e193739c953358c865005855253af4f68a497John McCall    return;
603f85e193739c953358c865005855253af4f68a497John McCall  }
604f85e193739c953358c865005855253af4f68a497John McCall
605f85e193739c953358c865005855253af4f68a497John McCall  S.Diag(property->getLocation(), diag::note_property_declare);
606f85e193739c953358c865005855253af4f68a497John McCall}
607f85e193739c953358c865005855253af4f68a497John McCall
608015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// setImpliedPropertyAttributeForReadOnlyProperty -
609015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// This routine evaludates life-time attributes for a 'readonly'
610015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// property with no known lifetime of its own, using backing
611015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// 'ivar's attribute, if any. If no backing 'ivar', property's
612015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian/// life-time is assumed 'strong'.
613015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanianstatic void setImpliedPropertyAttributeForReadOnlyProperty(
614015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian              ObjCPropertyDecl *property, ObjCIvarDecl *ivar) {
615015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  Qualifiers::ObjCLifetime propertyLifetime =
616015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    getImpliedARCOwnership(property->getPropertyAttributes(),
617015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                           property->getType());
618015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  if (propertyLifetime != Qualifiers::OCL_None)
619015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    return;
620015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian
621015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  if (!ivar) {
622015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // if no backing ivar, make property 'strong'.
623015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    property->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
624015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    return;
625015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  }
626015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  // property assumes owenership of backing ivar.
627015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  QualType ivarType = ivar->getType();
628015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  Qualifiers::ObjCLifetime ivarLifetime = ivarType.getObjCLifetime();
629015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  if (ivarLifetime == Qualifiers::OCL_Strong)
630015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    property->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
631015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  else if (ivarLifetime == Qualifiers::OCL_Weak)
632015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    property->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_weak);
633015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  return;
634015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian}
63528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
636cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian/// DiagnoseClassAndClassExtPropertyMismatch - diagnose inconsistant property
637cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian/// attribute declared in primary class and attributes overridden in any of its
638cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian/// class extensions.
639cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanianstatic void
640cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz JahanianDiagnoseClassAndClassExtPropertyMismatch(Sema &S, ObjCInterfaceDecl *ClassDecl,
641cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                                         ObjCPropertyDecl *property) {
642cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  unsigned Attributes = property->getPropertyAttributesAsWritten();
643cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  bool warn = (Attributes & ObjCDeclSpec::DQ_PR_readonly);
644cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  for (const ObjCCategoryDecl *CDecl = ClassDecl->getFirstClassExtension();
645cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian       CDecl; CDecl = CDecl->getNextClassExtension()) {
646cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    ObjCPropertyDecl *ClassExtProperty = 0;
647cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    for (ObjCContainerDecl::prop_iterator P = CDecl->prop_begin(),
648cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian         E = CDecl->prop_end(); P != E; ++P) {
649cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      if ((*P)->getIdentifier() == property->getIdentifier()) {
650cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        ClassExtProperty = *P;
651cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        break;
652cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      }
653cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    }
654cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    if (ClassExtProperty) {
655c78ff27813288b30ee566477705353ae660fd577Fariborz Jahanian      warn = false;
656cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      unsigned classExtPropertyAttr =
657cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        ClassExtProperty->getPropertyAttributesAsWritten();
658cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      // We are issuing the warning that we postponed because class extensions
659cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      // can override readonly->readwrite and 'setter' attributes originally
660cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      // placed on class's property declaration now make sense in the overridden
661cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      // property.
662cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      if (Attributes & ObjCDeclSpec::DQ_PR_readonly) {
663cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        if (!classExtPropertyAttr ||
664cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian            (classExtPropertyAttr & ObjCDeclSpec::DQ_PR_readwrite))
665cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian          continue;
666cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        warn = true;
667cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        break;
668cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      }
669cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    }
670cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  }
671cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  if (warn) {
672cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    unsigned setterAttrs = (ObjCDeclSpec::DQ_PR_assign |
673cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                            ObjCDeclSpec::DQ_PR_unsafe_unretained |
674cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                            ObjCDeclSpec::DQ_PR_copy |
675cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                            ObjCDeclSpec::DQ_PR_retain |
676cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                            ObjCDeclSpec::DQ_PR_strong);
677cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    if (Attributes & setterAttrs) {
678cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      const char * which =
679cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_assign) ?
680cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      "assign" :
681cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) ?
682cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      "unsafe_unretained" :
683cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_copy) ?
684cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      "copy" :
685cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_retain) ?
686cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      "retain" : "strong";
687cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian
688cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      S.Diag(property->getLocation(),
689cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian             diag::warn_objc_property_attr_mutually_exclusive)
690cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      << "readonly" << which;
691cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    }
692cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  }
693cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian
694cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian
695cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian}
696cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian
69728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// ActOnPropertyImplDecl - This routine performs semantic checks and
69828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// builds the AST node for a property implementation declaration; declared
699699c9044c7d53a2774d0dd261a6901dd2c4a545fJames Dennett/// as \@synthesize or \@dynamic.
70028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek///
701d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallDecl *Sema::ActOnPropertyImplDecl(Scope *S,
702d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  SourceLocation AtLoc,
703d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  SourceLocation PropertyLoc,
704d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  bool Synthesize,
705d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall                                  IdentifierInfo *PropertyId,
706a4ffd85a6684e42f900aad5459e58ad91bb88755Douglas Gregor                                  IdentifierInfo *PropertyIvar,
707a4ffd85a6684e42f900aad5459e58ad91bb88755Douglas Gregor                                  SourceLocation PropertyIvarLoc) {
708e9686572f17968048a5e097254f85c202ea44aecTed Kremenek  ObjCContainerDecl *ClassImpDecl =
70984e0ccff7a6e51d7b81fd58686b493a4880dd44dFariborz Jahanian    dyn_cast<ObjCContainerDecl>(CurContext);
71028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Make sure we have a context for the property implementation declaration.
71128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (!ClassImpDecl) {
71228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(AtLoc, diag::error_missing_property_context);
713d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall    return 0;
71428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
715f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis  if (PropertyIvarLoc.isInvalid())
716f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis    PropertyIvarLoc = PropertyLoc;
717e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman  SourceLocation PropertyDiagLoc = PropertyLoc;
718e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman  if (PropertyDiagLoc.isInvalid())
719e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    PropertyDiagLoc = ClassImpDecl->getLocStart();
72028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyDecl *property = 0;
72128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCInterfaceDecl* IDecl = 0;
72228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Find the class or category class where this property must have
72328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // a declaration.
72428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCImplementationDecl *IC = 0;
72528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCCategoryImplDecl* CatImplClass = 0;
72628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if ((IC = dyn_cast<ObjCImplementationDecl>(ClassImpDecl))) {
72728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IDecl = IC->getClassInterface();
72828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // We always synthesize an interface for an implementation
72928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // without an interface decl. So, IDecl is always non-zero.
73028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    assert(IDecl &&
73128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek           "ActOnPropertyImplDecl - @implementation without @interface");
73228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
73328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // Look for this property declaration in the @implementation's @interface
73428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    property = IDecl->FindPropertyDeclaration(PropertyId);
73528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!property) {
73628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PropertyLoc, diag::error_bad_property_decl) << IDecl->getDeclName();
737d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
73828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
739dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian    unsigned PIkind = property->getPropertyAttributesAsWritten();
74045937ae10a0f70f74508165aab4f2b63e18ea747Fariborz Jahanian    if ((PIkind & (ObjCPropertyDecl::OBJC_PR_atomic |
74145937ae10a0f70f74508165aab4f2b63e18ea747Fariborz Jahanian                   ObjCPropertyDecl::OBJC_PR_nonatomic) ) == 0) {
742dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian      if (AtLoc.isValid())
743dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian        Diag(AtLoc, diag::warn_implicit_atomic_property);
744dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian      else
745dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian        Diag(IC->getLocation(), diag::warn_auto_implicit_atomic_property);
746dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian      Diag(property->getLocation(), diag::note_property_declare);
747dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian    }
748dd4430e596fac34e9ce44228a249f71e73effd4aFariborz Jahanian
74928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (const ObjCCategoryDecl *CD =
75028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        dyn_cast<ObjCCategoryDecl>(property->getDeclContext())) {
75128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if (!CD->IsClassExtension()) {
75228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PropertyLoc, diag::error_category_property) << CD->getDeclName();
75328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(property->getLocation(), diag::note_property_declare);
754d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall        return 0;
75528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
75628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
7572b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian
7582b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian    if (Synthesize&&
7592b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian        (PIkind & ObjCPropertyDecl::OBJC_PR_readonly) &&
7602b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian        property->hasAttr<IBOutletAttr>() &&
7612b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian        !AtLoc.isValid()) {
7622b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian      Diag(IC->getLocation(), diag::warn_auto_readonly_iboutlet_property);
7632b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian      Diag(property->getLocation(), diag::note_property_declare);
764edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      SourceLocation readonlyLoc;
7655bf0e3522fac400cf7bf49a775554f0fc942b336Fariborz Jahanian      if (LocPropertyAttribute(Context, "readonly",
766edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian                               property->getLParenLoc(), readonlyLoc)) {
767edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        SourceLocation endLoc =
768edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian          readonlyLoc.getLocWithOffset(strlen("readonly")-1);
769edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        SourceRange ReadonlySourceRange(readonlyLoc, endLoc);
770edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        Diag(property->getLocation(),
771edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian             diag::note_auto_readonly_iboutlet_fixup_suggest) <<
772edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian        FixItHint::CreateReplacement(ReadonlySourceRange, "readwrite");
773edcc27f6171fd8cc736d057d87b128252fcb7344Fariborz Jahanian      }
7742b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian    }
775cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian
776cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    DiagnoseClassAndClassExtPropertyMismatch(*this, IDecl, property);
7772b309fb125906b1fb10efb1890860cd042fbfab3Fariborz Jahanian
77828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else if ((CatImplClass = dyn_cast<ObjCCategoryImplDecl>(ClassImpDecl))) {
77928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Synthesize) {
78028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(AtLoc, diag::error_synthesize_category_decl);
781d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
78228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
78328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IDecl = CatImplClass->getClassInterface();
78428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!IDecl) {
78528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(AtLoc, diag::error_missing_property_interface);
786d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
78728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
78828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    ObjCCategoryDecl *Category =
78928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IDecl->FindCategoryDeclaration(CatImplClass->getIdentifier());
79028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
79128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // If category for this implementation not found, it is an error which
79228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // has already been reported eralier.
79328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!Category)
794d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
79528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // Look for this property declaration in @implementation's category
79628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    property = Category->FindPropertyDeclaration(PropertyId);
79728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!property) {
79828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PropertyLoc, diag::error_bad_category_property_decl)
79928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << Category->getDeclName();
800d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
80128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
80228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else {
80328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(AtLoc, diag::error_bad_property_context);
804d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall    return 0;
80528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
80628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCIvarDecl *Ivar = 0;
807e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman  bool CompleteTypeErr = false;
808744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian  bool compat = true;
80928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Check that we have a valid, previously declared ivar for @synthesize
81028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Synthesize) {
81128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // @synthesize
81228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (!PropertyIvar)
81328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      PropertyIvar = PropertyId;
814015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // Check that this is a previously declared 'ivar' in 'IDecl' interface
815015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    ObjCInterfaceDecl *ClassDeclared;
816015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    Ivar = IDecl->lookupInstanceVariable(PropertyIvar, ClassDeclared);
817265941bc308d65cc270d5c4de5806f37ce405606John McCall    QualType PropType = property->getType();
818265941bc308d65cc270d5c4de5806f37ce405606John McCall    QualType PropertyIvarType = PropType.getNonReferenceType();
819e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
820e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    if (RequireCompleteType(PropertyDiagLoc, PropertyIvarType,
821d10099e5c8238fa0327f03921cf2e3c8975c881eDouglas Gregor                            diag::err_incomplete_synthesized_property,
822d10099e5c8238fa0327f03921cf2e3c8975c881eDouglas Gregor                            property->getDeclName())) {
823e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      Diag(property->getLocation(), diag::note_property_declare);
824e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      CompleteTypeErr = true;
825e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    }
826e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
8274e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount &&
828015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian        (property->getPropertyAttributesAsWritten() &
8293efd348d3586b22bb0153f17181128a1b3c41140Fariborz Jahanian         ObjCPropertyDecl::OBJC_PR_readonly) &&
8303efd348d3586b22bb0153f17181128a1b3c41140Fariborz Jahanian        PropertyIvarType->isObjCRetainableType()) {
831015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      setImpliedPropertyAttributeForReadOnlyProperty(property, Ivar);
832015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    }
833015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian
834015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    ObjCPropertyDecl::PropertyAttributeKind kind
835015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      = property->getPropertyAttributes();
836265941bc308d65cc270d5c4de5806f37ce405606John McCall
837265941bc308d65cc270d5c4de5806f37ce405606John McCall    // Add GC __weak to the ivar type if the property is weak.
838265941bc308d65cc270d5c4de5806f37ce405606John McCall    if ((kind & ObjCPropertyDecl::OBJC_PR_weak) &&
8394e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        getLangOpts().getGC() != LangOptions::NonGC) {
8404e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      assert(!getLangOpts().ObjCAutoRefCount);
841265941bc308d65cc270d5c4de5806f37ce405606John McCall      if (PropertyIvarType.isObjCGCStrong()) {
842e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::err_gc_weak_property_strong_type);
843265941bc308d65cc270d5c4de5806f37ce405606John McCall        Diag(property->getLocation(), diag::note_property_declare);
844265941bc308d65cc270d5c4de5806f37ce405606John McCall      } else {
845265941bc308d65cc270d5c4de5806f37ce405606John McCall        PropertyIvarType =
846265941bc308d65cc270d5c4de5806f37ce405606John McCall          Context.getObjCGCQualType(PropertyIvarType, Qualifiers::Weak);
847edc08821d3f73514291b339db29aca1a17485e39Fariborz Jahanian      }
848edc08821d3f73514291b339db29aca1a17485e39Fariborz Jahanian    }
849e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian    if (AtLoc.isInvalid()) {
850e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      // Check when default synthesizing a property that there is
851e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      // an ivar matching property name and issue warning; since this
852e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      // is the most common case of not using an ivar used for backing
853e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      // property in non-default synthesis case.
854e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      ObjCInterfaceDecl *ClassDeclared=0;
855e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      ObjCIvarDecl *originalIvar =
856e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      IDecl->lookupInstanceVariable(property->getIdentifier(),
857e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian                                    ClassDeclared);
858e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian      if (originalIvar) {
859e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian        Diag(PropertyDiagLoc,
860e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian             diag::warn_autosynthesis_property_ivar_match)
8612578532c7323efdd68ec42a31b7bdf72e5218867Fariborz Jahanian        << PropertyId << (Ivar == 0) << PropertyIvar
86220e7d992140cc6bc94f2e65f3638346527150756Fariborz Jahanian        << originalIvar->getIdentifier();
863e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian        Diag(property->getLocation(), diag::note_property_declare);
864e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian        Diag(originalIvar->getLocation(), diag::note_ivar_decl);
865dd3284b17829adfd542ecf4a7b4423dce1d7502bFariborz Jahanian      }
866e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian    }
867e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian
868e95f8ef6563bdf26de5294873733a3425c09db42Fariborz Jahanian    if (!Ivar) {
869265941bc308d65cc270d5c4de5806f37ce405606John McCall      // In ARC, give the ivar a lifetime qualifier based on the
870f85e193739c953358c865005855253af4f68a497John McCall      // property attributes.
8714e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      if (getLangOpts().ObjCAutoRefCount &&
872265941bc308d65cc270d5c4de5806f37ce405606John McCall          !PropertyIvarType.getObjCLifetime() &&
873265941bc308d65cc270d5c4de5806f37ce405606John McCall          PropertyIvarType->isObjCRetainableType()) {
874f85e193739c953358c865005855253af4f68a497John McCall
875265941bc308d65cc270d5c4de5806f37ce405606John McCall        // It's an error if we have to do this and the user didn't
876265941bc308d65cc270d5c4de5806f37ce405606John McCall        // explicitly write an ownership attribute on the property.
877473506bd9dfd84944c2c5ca2c2a38814f46febc6Argyrios Kyrtzidis        if (!property->hasWrittenStorageAttribute() &&
878265941bc308d65cc270d5c4de5806f37ce405606John McCall            !(kind & ObjCPropertyDecl::OBJC_PR_strong)) {
879e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman          Diag(PropertyDiagLoc,
880473506bd9dfd84944c2c5ca2c2a38814f46febc6Argyrios Kyrtzidis               diag::err_arc_objc_property_default_assign_on_object);
881473506bd9dfd84944c2c5ca2c2a38814f46febc6Argyrios Kyrtzidis          Diag(property->getLocation(), diag::note_property_declare);
882265941bc308d65cc270d5c4de5806f37ce405606John McCall        } else {
883265941bc308d65cc270d5c4de5806f37ce405606John McCall          Qualifiers::ObjCLifetime lifetime =
884265941bc308d65cc270d5c4de5806f37ce405606John McCall            getImpliedARCOwnership(kind, PropertyIvarType);
885265941bc308d65cc270d5c4de5806f37ce405606John McCall          assert(lifetime && "no lifetime for property?");
8866dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian          if (lifetime == Qualifiers::OCL_Weak) {
8876dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian            bool err = false;
8886dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian            if (const ObjCObjectPointerType *ObjT =
8896dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian                PropertyIvarType->getAs<ObjCObjectPointerType>())
8906dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian              if (ObjT->getInterfaceDecl()->isArcWeakrefUnavailable()) {
891e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman                Diag(PropertyDiagLoc, diag::err_arc_weak_unavailable_property);
8926dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian                Diag(property->getLocation(), diag::note_property_declare);
8936dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian                err = true;
8946dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian              }
8950a7dd788dbef975f35f273c7ab913f480f7edd60John McCall            if (!err && !getLangOpts().ObjCARCWeak) {
896e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman              Diag(PropertyDiagLoc, diag::err_arc_weak_no_runtime);
8976dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian              Diag(property->getLocation(), diag::note_property_declare);
8986dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian            }
899f85e193739c953358c865005855253af4f68a497John McCall          }
9006dce88d9f71cd4be18554af94145a9147b363199Fariborz Jahanian
901f85e193739c953358c865005855253af4f68a497John McCall          Qualifiers qs;
902265941bc308d65cc270d5c4de5806f37ce405606John McCall          qs.addObjCLifetime(lifetime);
903f85e193739c953358c865005855253af4f68a497John McCall          PropertyIvarType = Context.getQualifiedType(PropertyIvarType, qs);
904f85e193739c953358c865005855253af4f68a497John McCall        }
905f85e193739c953358c865005855253af4f68a497John McCall      }
906f85e193739c953358c865005855253af4f68a497John McCall
907f85e193739c953358c865005855253af4f68a497John McCall      if (kind & ObjCPropertyDecl::OBJC_PR_weak &&
9084e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          !getLangOpts().ObjCAutoRefCount &&
9094e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          getLangOpts().getGC() == LangOptions::NonGC) {
910e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_synthesize_weak_non_arc_or_gc);
911f85e193739c953358c865005855253af4f68a497John McCall        Diag(property->getLocation(), diag::note_property_declare);
912f85e193739c953358c865005855253af4f68a497John McCall      }
913f85e193739c953358c865005855253af4f68a497John McCall
914ff676cb48fe8bf7be2feaa251dc7c5fb15af4730Abramo Bagnara      Ivar = ObjCIvarDecl::Create(Context, ClassImpDecl,
915f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis                                  PropertyIvarLoc,PropertyIvarLoc, PropertyIvar,
91614086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian                                  PropertyIvarType, /*Dinfo=*/0,
9177504966cc0d07eac26baabec80a7406695efb307Fariborz Jahanian                                  ObjCIvarDecl::Private,
918ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian                                  (Expr *)0, true);
919e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      if (CompleteTypeErr)
920e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Ivar->setInvalidDecl();
92129fa69addf8b19871e7866c32c92b5d142c4bfbcDaniel Dunbar      ClassImpDecl->addDecl(Ivar);
9221b7f9cbed1b96b58a6e5f7808ebc9345a76a0936Richard Smith      IDecl->makeDeclVisibleInContext(Ivar);
92328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      property->setPropertyIvarDecl(Ivar);
92428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
925260611a32535c851237926bfcf78869b13c07d5bJohn McCall      if (getLangOpts().ObjCRuntime.isFragile())
926e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_missing_property_ivar_decl)
927e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman            << PropertyId;
92828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // Note! I deliberately want it to fall thru so, we have a
92928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // a property implementation and to avoid future warnings.
930260611a32535c851237926bfcf78869b13c07d5bJohn McCall    } else if (getLangOpts().ObjCRuntime.isNonFragile() &&
93160ef308e51c71b760d7f598c1b763ceb7b768148Douglas Gregor               !declaresSameEntity(ClassDeclared, IDecl)) {
932e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      Diag(PropertyDiagLoc, diag::error_ivar_in_superclass_use)
93328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << property->getDeclName() << Ivar->getDeclName()
93428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << ClassDeclared->getDeclName();
93528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Ivar->getLocation(), diag::note_previous_access_declaration)
9364087f27e5416c799bcb6be072f905be752acb61cDaniel Dunbar      << Ivar << Ivar->getName();
93728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // Note! I deliberately want it to fall thru so more errors are caught.
93828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
93928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    QualType IvarType = Context.getCanonicalType(Ivar->getType());
94028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
94128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // Check that type of property and its ivar are type compatible.
942744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian    if (!Context.hasSameType(PropertyIvarType, IvarType)) {
943744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian      compat = false;
94414086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian      if (isa<ObjCObjectPointerType>(PropertyIvarType)
945f85e193739c953358c865005855253af4f68a497John McCall          && isa<ObjCObjectPointerType>(IvarType))
94662ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian        compat =
94762ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian          Context.canAssignObjCInterfaces(
94814086764e340267e17803d0f8243070ffae2c76eFariborz Jahanian                                  PropertyIvarType->getAs<ObjCObjectPointerType>(),
94962ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian                                  IvarType->getAs<ObjCObjectPointerType>());
950b608b987718c6d841115464f79ab2d1820a63e17Douglas Gregor      else {
951f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis        compat = (CheckAssignmentConstraints(PropertyIvarLoc, PropertyIvarType,
952f911242f43ae1b0a85c323631fe817df95c9cbe9Argyrios Kyrtzidis                                             IvarType)
953daa8e4e888758d55a7a759dd4a91b83921cef222John McCall                    == Compatible);
954b608b987718c6d841115464f79ab2d1820a63e17Douglas Gregor      }
95562ac5d01aade35790a6d8e814edb21062da5d3f7Fariborz Jahanian      if (!compat) {
956e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_property_ivar_type)
957f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek          << property->getDeclName() << PropType
958f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek          << Ivar->getDeclName() << IvarType;
959f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek        Diag(Ivar->getLocation(), diag::note_ivar_decl);
96028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // Note! I deliberately want it to fall thru so, we have a
96128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // a property implementation and to avoid future warnings.
96228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
963744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian      else {
964744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        // FIXME! Rules for properties are somewhat different that those
965744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        // for assignments. Use a new routine to consolidate all cases;
966744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        // specifically for property redeclarations as well as for ivars.
967744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        QualType lhsType =Context.getCanonicalType(PropertyIvarType).getUnqualifiedType();
968744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        QualType rhsType =Context.getCanonicalType(IvarType).getUnqualifiedType();
969744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        if (lhsType != rhsType &&
970744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian            lhsType->isArithmeticType()) {
971744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian          Diag(PropertyDiagLoc, diag::error_property_ivar_type)
972744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian            << property->getDeclName() << PropType
973744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian            << Ivar->getDeclName() << IvarType;
974744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian          Diag(Ivar->getLocation(), diag::note_ivar_decl);
975744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian          // Fall thru - see previous comment
976744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian        }
97728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
97828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      // __weak is explicit. So it works on Canonical type.
979f85e193739c953358c865005855253af4f68a497John McCall      if ((PropType.isObjCGCWeak() && !IvarType.isObjCGCWeak() &&
9804e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie           getLangOpts().getGC() != LangOptions::NonGC)) {
981e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_weak_property)
98228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << property->getDeclName() << Ivar->getDeclName();
983edc08821d3f73514291b339db29aca1a17485e39Fariborz Jahanian        Diag(Ivar->getLocation(), diag::note_ivar_decl);
98428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // Fall thru - see previous comment
98528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
986f85e193739c953358c865005855253af4f68a497John McCall      // Fall thru - see previous comment
98728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if ((property->getType()->isObjCObjectPointerType() ||
98828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek           PropType.isObjCGCStrong()) && IvarType.isObjCGCWeak() &&
9894e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          getLangOpts().getGC() != LangOptions::NonGC) {
990e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_strong_property)
99128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << property->getDeclName() << Ivar->getDeclName();
99228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        // Fall thru - see previous comment
99328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
99428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
9954e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount)
996f85e193739c953358c865005855253af4f68a497John McCall      checkARCPropertyImpl(*this, PropertyLoc, property, Ivar);
99728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else if (PropertyIvar)
99828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // @dynamic
999e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    Diag(PropertyDiagLoc, diag::error_dynamic_property_ivar_decl);
1000f85e193739c953358c865005855253af4f68a497John McCall
100128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  assert (property && "ActOnPropertyImplDecl - property declaration missing");
100228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyImplDecl *PIDecl =
100328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  ObjCPropertyImplDecl::Create(Context, CurContext, AtLoc, PropertyLoc,
100428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                               property,
100528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                               (Synthesize ?
100628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                                ObjCPropertyImplDecl::Synthesize
100728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                                : ObjCPropertyImplDecl::Dynamic),
1008a4ffd85a6684e42f900aad5459e58ad91bb88755Douglas Gregor                               Ivar, PropertyIvarLoc);
1009e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
1010744147138cd2a77320c431578da130cc3d81a1d5Fariborz Jahanian  if (CompleteTypeErr || !compat)
1011e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    PIDecl->setInvalidDecl();
1012e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman
101317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  if (ObjCMethodDecl *getterMethod = property->getGetterMethodDecl()) {
101417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    getterMethod->createImplicitParams(Context, IDecl);
1015e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    if (getLangOpts().CPlusPlus && Synthesize && !CompleteTypeErr &&
10160313f441b72ef6b69a93e5003c684b01cb72fd46Fariborz Jahanian        Ivar->getType()->isRecordType()) {
101717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // For Objective-C++, need to synthesize the AST for the IVAR object to be
101817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // returned by the getter as it must conform to C++'s copy-return rules.
101917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // FIXME. Eventually we want to do this for Objective-C as well.
102017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ImplicitParamDecl *SelfDecl = getterMethod->getSelfDecl();
102117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      DeclRefExpr *SelfExpr =
1022f4b88a45902af1802a1cb42ba48b1c474474f228John McCall        new (Context) DeclRefExpr(SelfDecl, false, SelfDecl->getType(),
1023f89e55ab1bfb3ea997f8b02997c611a02254eb2dJohn McCall                                  VK_RValue, SourceLocation());
102417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      Expr *IvarRefExpr =
102517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc,
102617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                      SelfExpr, true, true);
102760d7b3a319d84d688752be3870615ac0f111fb16John McCall      ExprResult Res =
102817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        PerformCopyInitialization(InitializedEntity::InitializeResult(
10293c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor                                    SourceLocation(),
10303c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor                                    getterMethod->getResultType(),
10313c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor                                    /*NRVO=*/false),
103217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                  SourceLocation(),
103317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                  Owned(IvarRefExpr));
103417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      if (!Res.isInvalid()) {
103517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        Expr *ResExpr = Res.takeAs<Expr>();
103617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        if (ResExpr)
10374765fa05b5652fcc4356371c2f481d0ea9a1b007John McCall          ResExpr = MaybeCreateExprWithCleanups(ResExpr);
103817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        PIDecl->setGetterCXXConstructor(ResExpr);
103917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      }
104017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    }
1041831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    if (property->hasAttr<NSReturnsNotRetainedAttr>() &&
1042831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian        !getterMethod->hasAttr<NSReturnsNotRetainedAttr>()) {
1043831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian      Diag(getterMethod->getLocation(),
1044831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian           diag::warn_property_getter_owning_mismatch);
1045831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian      Diag(property->getLocation(), diag::note_property_declare);
1046831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    }
104717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  }
104817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  if (ObjCMethodDecl *setterMethod = property->getSetterMethodDecl()) {
104917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    setterMethod->createImplicitParams(Context, IDecl);
1050e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman    if (getLangOpts().CPlusPlus && Synthesize && !CompleteTypeErr &&
1051e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Ivar->getType()->isRecordType()) {
105217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      // FIXME. Eventually we want to do this for Objective-C as well.
105317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ImplicitParamDecl *SelfDecl = setterMethod->getSelfDecl();
105417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      DeclRefExpr *SelfExpr =
1055f4b88a45902af1802a1cb42ba48b1c474474f228John McCall        new (Context) DeclRefExpr(SelfDecl, false, SelfDecl->getType(),
1056f89e55ab1bfb3ea997f8b02997c611a02254eb2dJohn McCall                                  VK_RValue, SourceLocation());
105717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      Expr *lhs =
105817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian        new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc,
105917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian                                      SelfExpr, true, true);
106017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ObjCMethodDecl::param_iterator P = setterMethod->param_begin();
106117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      ParmVarDecl *Param = (*P);
10623c3b7f90a863af43fa63043d396553ecf205351cJohn McCall      QualType T = Param->getType().getNonReferenceType();
1063f4b88a45902af1802a1cb42ba48b1c474474f228John McCall      Expr *rhs = new (Context) DeclRefExpr(Param, false, T,
1064f89e55ab1bfb3ea997f8b02997c611a02254eb2dJohn McCall                                            VK_LValue, SourceLocation());
1065fa432391a708448518df1e6ced4e34f4c349d169Fariborz Jahanian      ExprResult Res = BuildBinOp(S, lhs->getLocEnd(),
10662de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall                                  BO_Assign, lhs, rhs);
106757e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian      if (property->getPropertyAttributes() &
106857e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian          ObjCPropertyDecl::OBJC_PR_atomic) {
106957e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian        Expr *callExpr = Res.takeAs<Expr>();
107057e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian        if (const CXXOperatorCallExpr *CXXCE =
107113bf6336ab395eb6af487ab96d32229460907769Fariborz Jahanian              dyn_cast_or_null<CXXOperatorCallExpr>(callExpr))
107213bf6336ab395eb6af487ab96d32229460907769Fariborz Jahanian          if (const FunctionDecl *FuncDecl = CXXCE->getDirectCallee())
107357e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian            if (!FuncDecl->isTrivial())
107420abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian              if (property->getType()->isReferenceType()) {
107520abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                Diag(PropertyLoc,
107620abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                     diag::err_atomic_property_nontrivial_assign_op)
107757e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian                    << property->getType();
107820abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                Diag(FuncDecl->getLocStart(),
107920abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian                     diag::note_callee_decl) << FuncDecl;
108020abee6b95c4f5a61e471b4b616439280ca4a81bFariborz Jahanian              }
108157e264e9f4ff0a72f3585a960cdf63437b76fa93Fariborz Jahanian      }
108217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian      PIDecl->setSetterCXXAssignment(Res.takeAs<Expr>());
108317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian    }
108417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian  }
108517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian
108628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (IC) {
108728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Synthesize)
108828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if (ObjCPropertyImplDecl *PPIDecl =
108928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek          IC->FindPropertyImplIvarDecl(PropertyIvar)) {
109028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PropertyLoc, diag::error_duplicate_ivar_use)
109128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier()
109228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyIvar;
109328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PPIDecl->getLocation(), diag::note_previous_use);
109428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
109528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
109628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (ObjCPropertyImplDecl *PPIDecl
109728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        = IC->FindPropertyImplDecl(PropertyId)) {
109828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PropertyLoc, diag::error_property_implemented) << PropertyId;
109928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PPIDecl->getLocation(), diag::note_previous_declaration);
1100d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
110128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
110228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    IC->addPropertyImplementation(PIDecl);
11034e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCDefaultSynthProperties &&
1104260611a32535c851237926bfcf78869b13c07d5bJohn McCall        getLangOpts().ObjCRuntime.isNonFragile() &&
110571207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek        !IDecl->isObjCRequiresPropertyDefs()) {
1106ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      // Diagnose if an ivar was lazily synthesdized due to a previous
1107ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      // use and if 1) property is @dynamic or 2) property is synthesized
1108cdaa6a8fed16d8bd3987fb4f3304dfb4e52876c3Fariborz Jahanian      // but it requires an ivar of different name.
1109411c25c4512f00e07442e8818d011c7e0902d739Fariborz Jahanian      ObjCInterfaceDecl *ClassDeclared=0;
1110ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      ObjCIvarDecl *Ivar = 0;
1111ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      if (!Synthesize)
1112ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        Ivar = IDecl->lookupInstanceVariable(PropertyId, ClassDeclared);
1113ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      else {
1114ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        if (PropertyIvar && PropertyIvar != PropertyId)
1115ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian          Ivar = IDecl->lookupInstanceVariable(PropertyId, ClassDeclared);
1116ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      }
1117cdaa6a8fed16d8bd3987fb4f3304dfb4e52876c3Fariborz Jahanian      // Issue diagnostics only if Ivar belongs to current class.
1118cdaa6a8fed16d8bd3987fb4f3304dfb4e52876c3Fariborz Jahanian      if (Ivar && Ivar->getSynthesize() &&
111960ef308e51c71b760d7f598c1b763ceb7b768148Douglas Gregor          declaresSameEntity(IC->getClassInterface(), ClassDeclared)) {
1120ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        Diag(Ivar->getLocation(), diag::err_undeclared_var_use)
1121ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        << PropertyId;
1122ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian        Ivar->setInvalidDecl();
1123ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian      }
1124ad51e74030a59a8aa4ef0ebca1d7a701602ef53bFariborz Jahanian    }
112528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else {
112628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Synthesize)
112728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      if (ObjCPropertyImplDecl *PPIDecl =
112828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek          CatImplClass->FindPropertyImplIvarDecl(PropertyIvar)) {
1129e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman        Diag(PropertyDiagLoc, diag::error_duplicate_ivar_use)
113028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier()
113128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << PropertyIvar;
113228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        Diag(PPIDecl->getLocation(), diag::note_previous_use);
113328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      }
113428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
113528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (ObjCPropertyImplDecl *PPIDecl =
113628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        CatImplClass->FindPropertyImplDecl(PropertyId)) {
1137e4c043d3fdd0f98ab724aa4f1419cec9985ea2c4Eli Friedman      Diag(PropertyDiagLoc, diag::error_property_implemented) << PropertyId;
113828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(PPIDecl->getLocation(), diag::note_previous_declaration);
1139d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      return 0;
114028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
114128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    CatImplClass->addPropertyImplementation(PIDecl);
114228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
114328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
1144d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall  return PIDecl;
114528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek}
114628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
114728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===//
114828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek// Helper methods.
114928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===//
115028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek
11519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// DiagnosePropertyMismatch - Compares two properties for their
11529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// attributes and types and warns on a variety of inconsistencies.
11539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
11549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid
11559d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::DiagnosePropertyMismatch(ObjCPropertyDecl *Property,
11569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                               ObjCPropertyDecl *SuperProperty,
11579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                               const IdentifierInfo *inheritedName) {
11589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCPropertyDecl::PropertyAttributeKind CAttr =
11599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  Property->getPropertyAttributes();
11609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCPropertyDecl::PropertyAttributeKind SAttr =
11619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  SuperProperty->getPropertyAttributes();
11629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((CAttr & ObjCPropertyDecl::OBJC_PR_readonly)
11639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      && (SAttr & ObjCPropertyDecl::OBJC_PR_readwrite))
11649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_readonly_property)
11659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << inheritedName;
11669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((CAttr & ObjCPropertyDecl::OBJC_PR_copy)
11679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      != (SAttr & ObjCPropertyDecl::OBJC_PR_copy))
11689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
11699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "copy" << inheritedName;
11701b46d8d78be713c546fb9680eaeea8f793bee5bfFariborz Jahanian  else if (!(SAttr & ObjCPropertyDecl::OBJC_PR_readonly)){
1171f85e193739c953358c865005855253af4f68a497John McCall    unsigned CAttrRetain =
1172f85e193739c953358c865005855253af4f68a497John McCall      (CAttr &
1173f85e193739c953358c865005855253af4f68a497John McCall       (ObjCPropertyDecl::OBJC_PR_retain | ObjCPropertyDecl::OBJC_PR_strong));
1174f85e193739c953358c865005855253af4f68a497John McCall    unsigned SAttrRetain =
1175f85e193739c953358c865005855253af4f68a497John McCall      (SAttr &
1176f85e193739c953358c865005855253af4f68a497John McCall       (ObjCPropertyDecl::OBJC_PR_retain | ObjCPropertyDecl::OBJC_PR_strong));
1177f85e193739c953358c865005855253af4f68a497John McCall    bool CStrong = (CAttrRetain != 0);
1178f85e193739c953358c865005855253af4f68a497John McCall    bool SStrong = (SAttrRetain != 0);
1179f85e193739c953358c865005855253af4f68a497John McCall    if (CStrong != SStrong)
1180f85e193739c953358c865005855253af4f68a497John McCall      Diag(Property->getLocation(), diag::warn_property_attribute)
1181f85e193739c953358c865005855253af4f68a497John McCall        << Property->getDeclName() << "retain (or strong)" << inheritedName;
1182f85e193739c953358c865005855253af4f68a497John McCall  }
11839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
11849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if ((CAttr & ObjCPropertyDecl::OBJC_PR_nonatomic)
11859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      != (SAttr & ObjCPropertyDecl::OBJC_PR_nonatomic))
11869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
11879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "atomic" << inheritedName;
11889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (Property->getSetterName() != SuperProperty->getSetterName())
11899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
11909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "setter" << inheritedName;
11919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (Property->getGetterName() != SuperProperty->getGetterName())
11929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Diag(Property->getLocation(), diag::warn_property_attribute)
11939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Property->getDeclName() << "getter" << inheritedName;
11949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
11959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  QualType LHSType =
11969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Context.getCanonicalType(SuperProperty->getType());
11979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  QualType RHSType =
11989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    Context.getCanonicalType(Property->getType());
11999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1200c286f3835eb6001c61664cef5d610dfaf80a6e9bFariborz Jahanian  if (!Context.propertyTypesAreCompatible(LHSType, RHSType)) {
12018beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    // Do cases not handled in above.
12028beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    // FIXME. For future support of covariant property types, revisit this.
12038beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    bool IncompatibleObjC = false;
12048beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    QualType ConvertedType;
12058beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian    if (!isObjCPointerConversion(RHSType, LHSType,
12068beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian                                 ConvertedType, IncompatibleObjC) ||
120713546a871db44c707001f507a9edfd0d5bcc81b4Fariborz Jahanian        IncompatibleObjC) {
12088beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian        Diag(Property->getLocation(), diag::warn_property_types_are_incompatible)
12098beb6a2426b5a6b79ecf019316d9fbd30755e087Fariborz Jahanian        << Property->getType() << SuperProperty->getType() << inheritedName;
121013546a871db44c707001f507a9edfd0d5bcc81b4Fariborz Jahanian      Diag(SuperProperty->getLocation(), diag::note_property_declare);
121113546a871db44c707001f507a9edfd0d5bcc81b4Fariborz Jahanian    }
12129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
12139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
12149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekbool Sema::DiagnosePropertyAccessorMismatch(ObjCPropertyDecl *property,
12169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                            ObjCMethodDecl *GetterMethod,
12179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                            SourceLocation Loc) {
12189abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  if (!GetterMethod)
12199abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    return false;
12209abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  QualType GetterType = GetterMethod->getResultType().getNonReferenceType();
12219abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  QualType PropertyIvarType = property->getType().getNonReferenceType();
12229abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  bool compat = Context.hasSameType(PropertyIvarType, GetterType);
12239abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  if (!compat) {
12249abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    if (isa<ObjCObjectPointerType>(PropertyIvarType) &&
12259abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian        isa<ObjCObjectPointerType>(GetterType))
12269abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      compat =
12279abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian        Context.canAssignObjCInterfaces(
1228490a52b4947381879a47b4251db5fb81cdf5d02bFariborz Jahanian                                      GetterType->getAs<ObjCObjectPointerType>(),
1229490a52b4947381879a47b4251db5fb81cdf5d02bFariborz Jahanian                                      PropertyIvarType->getAs<ObjCObjectPointerType>());
1230490a52b4947381879a47b4251db5fb81cdf5d02bFariborz Jahanian    else if (CheckAssignmentConstraints(Loc, GetterType, PropertyIvarType)
12319abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian              != Compatible) {
12329abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian          Diag(Loc, diag::error_property_accessor_type)
12339abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian            << property->getDeclName() << PropertyIvarType
12349abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian            << GetterMethod->getSelector() << GetterType;
12359abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian          Diag(GetterMethod->getLocation(), diag::note_declared_at);
12369abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian          return true;
12379abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    } else {
12389abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      compat = true;
12399abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      QualType lhsType =Context.getCanonicalType(PropertyIvarType).getUnqualifiedType();
12409abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      QualType rhsType =Context.getCanonicalType(GetterType).getUnqualifiedType();
12419abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian      if (lhsType != rhsType && lhsType->isArithmeticType())
12429abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian        compat = false;
12439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
12449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
12459abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian
12469abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  if (!compat) {
12479abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    Diag(Loc, diag::warn_accessor_property_type_mismatch)
12489abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    << property->getDeclName()
12499abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    << GetterMethod->getSelector();
12509abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    Diag(GetterMethod->getLocation(), diag::note_declared_at);
12519abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian    return true;
12529abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian  }
12539abf88c9286d7465ce18285f87bb302f588b59fcFariborz Jahanian
12549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  return false;
12559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
12569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// ComparePropertiesInBaseAndSuper - This routine compares property
12589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// declarations in base and its super class, if any, and issues
1259fc8f0e14ad142ed811e90fbd9a30e419e301c717Chris Lattner/// diagnostics in a variety of inconsistent situations.
12609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
12619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::ComparePropertiesInBaseAndSuper(ObjCInterfaceDecl *IDecl) {
12629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCInterfaceDecl *SDecl = IDecl->getSuperClass();
12639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!SDecl)
12649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
12659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // FIXME: O(N^2)
12669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCInterfaceDecl::prop_iterator S = SDecl->prop_begin(),
12679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       E = SDecl->prop_end(); S != E; ++S) {
1268581deb3da481053c4993c7600f97acf7768caac5David Blaikie    ObjCPropertyDecl *SuperPDecl = *S;
12699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Does property in super class has declaration in current class?
12709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCInterfaceDecl::prop_iterator I = IDecl->prop_begin(),
12719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = IDecl->prop_end(); I != E; ++I) {
1272581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *PDecl = *I;
12739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (SuperPDecl->getIdentifier() == PDecl->getIdentifier())
12749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          DiagnosePropertyMismatch(PDecl, SuperPDecl,
12759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                   SDecl->getIdentifier());
12769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
12779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
12789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
12799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
12809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// MatchOneProtocolPropertiesInClass - This routine goes thru the list
12819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// of properties declared in a protocol and compares their attribute against
12829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// the same property declared in the class or category.
12839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid
12849d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::MatchOneProtocolPropertiesInClass(Decl *CDecl,
12859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                          ObjCProtocolDecl *PDecl) {
12869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl);
12879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!IDecl) {
12889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Category
12899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl);
12909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    assert (CatDecl && "MatchOneProtocolPropertiesInClass");
12919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!CatDecl->IsClassExtension())
12929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
12939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = PDecl->prop_end(); P != E; ++P) {
1294581deb3da481053c4993c7600f97acf7768caac5David Blaikie        ObjCPropertyDecl *Pr = *P;
12959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        ObjCCategoryDecl::prop_iterator CP, CE;
12969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        // Is this property already in  category's list of properties?
12972d2f9368d35b3628c7e3b4563f74849a0f901a00Ted Kremenek        for (CP = CatDecl->prop_begin(), CE = CatDecl->prop_end(); CP!=CE; ++CP)
1298262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie          if (CP->getIdentifier() == Pr->getIdentifier())
12999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            break;
13009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        if (CP != CE)
13019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          // Property protocol already exist in class. Diagnose any mismatch.
1302581deb3da481053c4993c7600f97acf7768caac5David Blaikie          DiagnosePropertyMismatch(*CP, Pr, PDecl->getIdentifier());
13039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      }
13049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
13059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
13079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       E = PDecl->prop_end(); P != E; ++P) {
1308581deb3da481053c4993c7600f97acf7768caac5David Blaikie    ObjCPropertyDecl *Pr = *P;
13099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCInterfaceDecl::prop_iterator CP, CE;
13109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Is this property already in  class's list of properties?
13119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (CP = IDecl->prop_begin(), CE = IDecl->prop_end(); CP != CE; ++CP)
1312262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie      if (CP->getIdentifier() == Pr->getIdentifier())
13139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        break;
13149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (CP != CE)
13159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Property protocol already exist in class. Diagnose any mismatch.
1316581deb3da481053c4993c7600f97acf7768caac5David Blaikie      DiagnosePropertyMismatch(*CP, Pr, PDecl->getIdentifier());
13179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
13189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
13199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// CompareProperties - This routine compares properties
13219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// declared in 'ClassOrProtocol' objects (which can be a class or an
13229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// inherited protocol with the list of properties for class/category 'CDecl'
13239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
1324d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallvoid Sema::CompareProperties(Decl *CDecl, Decl *ClassOrProtocol) {
1325d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall  Decl *ClassDecl = ClassOrProtocol;
13269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl);
13279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!IDecl) {
13299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Category
13309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl);
13319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    assert (CatDecl && "CompareProperties");
13329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (ObjCCategoryDecl *MDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl)) {
13339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCCategoryDecl::protocol_iterator P = MDecl->protocol_begin(),
13349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = MDecl->protocol_end(); P != E; ++P)
13359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Match properties of category with those of protocol (*P)
13369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      MatchOneProtocolPropertiesInClass(CatDecl, *P);
13379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Go thru the list of protocols for this category and recursively match
13399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // their properties with those in the category.
13409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCCategoryDecl::protocol_iterator P = CatDecl->protocol_begin(),
13419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = CatDecl->protocol_end(); P != E; ++P)
1342d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall        CompareProperties(CatDecl, *P);
13439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    } else {
13449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl);
13459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(),
13469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = MD->protocol_end(); P != E; ++P)
13479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        MatchOneProtocolPropertiesInClass(CatDecl, *P);
13489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
13499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
13509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCInterfaceDecl *MDecl = dyn_cast<ObjCInterfaceDecl>(ClassDecl)) {
135353b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
135453b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          P = MDecl->all_referenced_protocol_begin(),
135553b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          E = MDecl->all_referenced_protocol_end(); P != E; ++P)
13569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Match properties of class IDecl with those of protocol (*P).
13579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      MatchOneProtocolPropertiesInClass(IDecl, *P);
13589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Go thru the list of protocols for this class and recursively match
13609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // their properties with those declared in the class.
136153b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
136253b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          P = IDecl->all_referenced_protocol_begin(),
136353b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek          E = IDecl->all_referenced_protocol_end(); P != E; ++P)
1364d226f65006733ed7f709c3174f22ce33391cb58fJohn McCall      CompareProperties(IDecl, *P);
13659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  } else {
13669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl);
13679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(),
13689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = MD->protocol_end(); P != E; ++P)
13699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      MatchOneProtocolPropertiesInClass(IDecl, *P);
13709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
13719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
13729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// isPropertyReadonly - Return true if property is readonly, by searching
13749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// for the property in the class and in its categories and implementations
13759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek///
13769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekbool Sema::isPropertyReadonly(ObjCPropertyDecl *PDecl,
13779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                              ObjCInterfaceDecl *IDecl) {
13789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // by far the most common case.
13799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!PDecl->isReadOnly())
13809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return false;
13819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Even if property is ready only, if interface has a user defined setter,
13829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // it is not considered read only.
13839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (IDecl->getInstanceMethod(PDecl->getSetterName()))
13849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return false;
13859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
13869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Main class has the property as 'readonly'. Must search
13879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // through the category list to see if the property's
13889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // attribute has been over-ridden to 'readwrite'.
13899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCCategoryDecl *Category = IDecl->getCategoryList();
13909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       Category; Category = Category->getNextClassCategory()) {
13919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Even if property is ready only, if a category has a user defined setter,
13929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // it is not considered read only.
13939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (Category->getInstanceMethod(PDecl->getSetterName()))
13949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      return false;
13959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCPropertyDecl *P =
13969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Category->FindPropertyDeclaration(PDecl->getIdentifier());
13979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (P && !P->isReadOnly())
13989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      return false;
13999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
14019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Also, check for definition of a setter method in the implementation if
14029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // all else failed.
14039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(CurContext)) {
14049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (ObjCImplementationDecl *IMD =
14059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        dyn_cast<ObjCImplementationDecl>(OMD->getDeclContext())) {
14069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (IMD->getInstanceMethod(PDecl->getSetterName()))
14079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return false;
14089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    } else if (ObjCCategoryImplDecl *CIMD =
14099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek               dyn_cast<ObjCCategoryImplDecl>(OMD->getDeclContext())) {
14109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (CIMD->getInstanceMethod(PDecl->getSetterName()))
14119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return false;
14129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Lastly, look through the implementation (if one is in scope).
14159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCImplementationDecl *ImpDecl = IDecl->getImplementation())
14169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (ImpDecl->getInstanceMethod(PDecl->getSetterName()))
14179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      return false;
14189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // If all fails, look at the super class.
14199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCInterfaceDecl *SIDecl = IDecl->getSuperClass())
14209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return isPropertyReadonly(PDecl, SIDecl);
14219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  return true;
14229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
14239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
14249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// CollectImmediateProperties - This routine collects all properties in
14259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// the class and its conforming protocols; but not those it its super class.
14269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::CollectImmediateProperties(ObjCContainerDecl *CDecl,
1427cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian            llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap,
1428cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian            llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& SuperPropMap) {
14299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) {
14309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
14319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = IDecl->prop_end(); P != E; ++P) {
1432581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *Prop = *P;
14339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      PropMap[Prop->getIdentifier()] = Prop;
14349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through class's protocols.
143653b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
143753b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         PI = IDecl->all_referenced_protocol_begin(),
143853b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         E = IDecl->all_referenced_protocol_end(); PI != E; ++PI)
1439cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian        CollectImmediateProperties((*PI), PropMap, SuperPropMap);
14409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (ObjCCategoryDecl *CATDecl = dyn_cast<ObjCCategoryDecl>(CDecl)) {
14429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!CATDecl->IsClassExtension())
14439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      for (ObjCContainerDecl::prop_iterator P = CATDecl->prop_begin(),
14449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           E = CATDecl->prop_end(); P != E; ++P) {
1445581deb3da481053c4993c7600f97acf7768caac5David Blaikie        ObjCPropertyDecl *Prop = *P;
14469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        PropMap[Prop->getIdentifier()] = Prop;
14479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      }
14489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through class's protocols.
144953b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCCategoryDecl::protocol_iterator PI = CATDecl->protocol_begin(),
14509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = CATDecl->protocol_end(); PI != E; ++PI)
1451cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      CollectImmediateProperties((*PI), PropMap, SuperPropMap);
14529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) {
14549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
14559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->prop_end(); P != E; ++P) {
1456581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *Prop = *P;
1457cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      ObjCPropertyDecl *PropertyFromSuper = SuperPropMap[Prop->getIdentifier()];
1458cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      // Exclude property for protocols which conform to class's super-class,
1459cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      // as super-class has to implement the property.
1460a929ec7b673b244bec024ba7eb95140404a7af5dFariborz Jahanian      if (!PropertyFromSuper ||
1461a929ec7b673b244bec024ba7eb95140404a7af5dFariborz Jahanian          PropertyFromSuper->getIdentifier() != Prop->getIdentifier()) {
1462cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian        ObjCPropertyDecl *&PropEntry = PropMap[Prop->getIdentifier()];
1463cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian        if (!PropEntry)
1464cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian          PropEntry = Prop;
1465cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      }
14669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
14679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through protocol's protocols.
14689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
14699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->protocol_end(); PI != E; ++PI)
1470cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian      CollectImmediateProperties((*PI), PropMap, SuperPropMap);
14719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
14729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
14739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1474509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// CollectClassPropertyImplementations - This routine collects list of
1475509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties to be implemented in the class. This includes, class's
1476509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// and its conforming protocols' properties.
1477509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianstatic void CollectClassPropertyImplementations(ObjCContainerDecl *CDecl,
1478509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian                llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) {
1479509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) {
1480509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
1481509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian         E = IDecl->prop_end(); P != E; ++P) {
1482581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *Prop = *P;
1483509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      PropMap[Prop->getIdentifier()] = Prop;
1484509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    }
148553b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
148653b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         PI = IDecl->all_referenced_protocol_begin(),
148753b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         E = IDecl->all_referenced_protocol_end(); PI != E; ++PI)
1488509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      CollectClassPropertyImplementations((*PI), PropMap);
1489509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  }
1490509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) {
1491509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1492509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian         E = PDecl->prop_end(); P != E; ++P) {
1493581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *Prop = *P;
1494ac37150c13c4e6f54551c52a1118f105be5f136bFariborz Jahanian      if (!PropMap.count(Prop->getIdentifier()))
1495ac37150c13c4e6f54551c52a1118f105be5f136bFariborz Jahanian        PropMap[Prop->getIdentifier()] = Prop;
1496509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    }
1497509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    // scan through protocol's protocols.
1498509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
1499509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian         E = PDecl->protocol_end(); PI != E; ++PI)
1500509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      CollectClassPropertyImplementations((*PI), PropMap);
1501509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  }
1502509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian}
1503509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
1504509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// CollectSuperClassPropertyImplementations - This routine collects list of
1505509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties to be implemented in super class(s) and also coming from their
1506509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// conforming protocols.
1507509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianstatic void CollectSuperClassPropertyImplementations(ObjCInterfaceDecl *CDecl,
1508509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian                llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) {
1509509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  if (ObjCInterfaceDecl *SDecl = CDecl->getSuperClass()) {
1510509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    while (SDecl) {
1511509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      CollectClassPropertyImplementations(SDecl, PropMap);
1512509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      SDecl = SDecl->getSuperClass();
1513509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    }
1514509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  }
1515509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian}
1516509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
15179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// LookupPropertyDecl - Looks up a property in the current class and all
15189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// its protocols.
15199d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekObjCPropertyDecl *Sema::LookupPropertyDecl(const ObjCContainerDecl *CDecl,
15209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                     IdentifierInfo *II) {
15219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (const ObjCInterfaceDecl *IDecl =
15229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        dyn_cast<ObjCInterfaceDecl>(CDecl)) {
15239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
15249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = IDecl->prop_end(); P != E; ++P) {
1525581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *Prop = *P;
15269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop->getIdentifier() == II)
15279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
15289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
15299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through class's protocols.
153053b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek    for (ObjCInterfaceDecl::all_protocol_iterator
153153b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         PI = IDecl->all_referenced_protocol_begin(),
153253b9441b5a81a24fa1f66f3f6416f1e36baa9c2fTed Kremenek         E = IDecl->all_referenced_protocol_end(); PI != E; ++PI) {
15339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II);
15349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop)
15359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
15369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
15379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
15389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  else if (const ObjCProtocolDecl *PDecl =
15399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            dyn_cast<ObjCProtocolDecl>(CDecl)) {
15409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
15419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->prop_end(); P != E; ++P) {
1542581deb3da481053c4993c7600f97acf7768caac5David Blaikie      ObjCPropertyDecl *Prop = *P;
15439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop->getIdentifier() == II)
15449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
15459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
15469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // scan through protocol's protocols.
15479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
15489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         E = PDecl->protocol_end(); PI != E; ++PI) {
15499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II);
15509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (Prop)
15519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        return Prop;
15529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
15539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
15549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  return 0;
15559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
15569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1557d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenekstatic IdentifierInfo * getDefaultSynthIvarName(ObjCPropertyDecl *Prop,
1558d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek                                                ASTContext &Ctx) {
1559f7ccbad5d9949e7ddd1cbef43d482553b811e026Dylan Noblesmith  SmallString<128> ivarName;
1560d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek  {
1561d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek    llvm::raw_svector_ostream os(ivarName);
1562d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek    os << '_' << Prop->getIdentifier()->getName();
1563d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek  }
1564d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek  return &Ctx.Idents.get(ivarName.str());
1565d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek}
1566d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek
1567699c9044c7d53a2774d0dd261a6901dd2c4a545fJames Dennett/// \brief Default synthesizes all properties which must be synthesized
1568699c9044c7d53a2774d0dd261a6901dd2c4a545fJames Dennett/// in class's \@implementation.
1569d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenekvoid Sema::DefaultSynthesizeProperties(Scope *S, ObjCImplDecl* IMPDecl,
1570d2ee809f9e7c2c7a83b8918efa11e98863463f78Ted Kremenek                                       ObjCInterfaceDecl *IDecl) {
1571509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
1572509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap;
1573509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  CollectClassPropertyImplementations(IDecl, PropMap);
1574509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  if (PropMap.empty())
1575509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    return;
1576509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap;
1577509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  CollectSuperClassPropertyImplementations(IDecl, SuperPropMap);
1578509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian
1579509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian  for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator
1580509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian       P = PropMap.begin(), E = PropMap.end(); P != E; ++P) {
1581509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    ObjCPropertyDecl *Prop = P->second;
1582509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    // If property to be implemented in the super class, ignore.
1583509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    if (SuperPropMap[Prop->getIdentifier()])
1584509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      continue;
1585509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    // Is there a matching propery synthesize/dynamic?
1586509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian    if (Prop->isInvalidDecl() ||
1587509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian        Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional ||
1588509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian        IMPDecl->FindPropertyImplIvarDecl(Prop->getIdentifier()))
1589509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian      continue;
1590d3635b9681daaef74974932f77080910b032b9fdFariborz Jahanian    // Property may have been synthesized by user.
1591d3635b9681daaef74974932f77080910b032b9fdFariborz Jahanian    if (IMPDecl->FindPropertyImplDecl(Prop->getIdentifier()))
1592d3635b9681daaef74974932f77080910b032b9fdFariborz Jahanian      continue;
159395f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian    if (IMPDecl->getInstanceMethod(Prop->getGetterName())) {
159495f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian      if (Prop->getPropertyAttributes() & ObjCPropertyDecl::OBJC_PR_readonly)
159595f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian        continue;
159695f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian      if (IMPDecl->getInstanceMethod(Prop->getSetterName()))
159795f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian        continue;
159895f1b86b73a9a203311adc7646f2fa9ee5cdd2b7Fariborz Jahanian    }
1599f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian    if (isa<ObjCProtocolDecl>(Prop->getDeclContext())) {
1600f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      // We won't auto-synthesize properties declared in protocols.
1601f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      Diag(IMPDecl->getLocation(),
1602f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian           diag::warn_auto_synthesizing_protocol_property);
1603f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      Diag(Prop->getLocation(), diag::note_property_declare);
1604f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian      continue;
1605f8aba8c618920db8f6ef2db0c554b0c270503cefFariborz Jahanian    }
16062a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek
16072a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // We use invalid SourceLocations for the synthesized ivars since they
16082a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // aren't really synthesized at a particular location; they just exist.
16092a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // Saying that they are located at the @implementation isn't really going
16102a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek    // to help users.
1611975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian    ObjCPropertyImplDecl *PIDecl = dyn_cast_or_null<ObjCPropertyImplDecl>(
1612975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian      ActOnPropertyImplDecl(S, SourceLocation(), SourceLocation(),
1613975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            true,
1614975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            /* property = */ Prop->getIdentifier(),
1615975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian                            /* ivar = */ getDefaultSynthIvarName(Prop, Context),
1616390fff8e8b480fa146ffc14cbc63a1c2f9e2d206Argyrios Kyrtzidis                            Prop->getLocation()));
1617975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian    if (PIDecl) {
1618975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian      Diag(Prop->getLocation(), diag::warn_missing_explicit_synthesis);
16195ea66619a3108e2985e952b9a60a84316d464b25Fariborz Jahanian      Diag(IMPDecl->getLocation(), diag::note_while_in_implementation);
1620975eef64182a78d1afc0f3da8cba4cb961cda5e4Fariborz Jahanian    }
16212a6af6b7874e41f3e6213e8d18623c4bac150354Ted Kremenek  }
1622509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian}
16239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
16248697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanianvoid Sema::DefaultSynthesizeProperties(Scope *S, Decl *D) {
1625260611a32535c851237926bfcf78869b13c07d5bJohn McCall  if (!LangOpts.ObjCDefaultSynthProperties || LangOpts.ObjCRuntime.isFragile())
16268697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian    return;
16278697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  ObjCImplementationDecl *IC=dyn_cast_or_null<ObjCImplementationDecl>(D);
16288697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  if (!IC)
16298697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian    return;
16308697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian  if (ObjCInterfaceDecl* IDecl = IC->getClassInterface())
163171207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek    if (!IDecl->isObjCRequiresPropertyDefs())
1632eb4f2c56c298071d58b441ccf801b039be93788aFariborz Jahanian      DefaultSynthesizeProperties(S, IC, IDecl);
16338697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian}
16348697d308c1bdd50e5c45757ac11be701c26e9e97Fariborz Jahanian
163517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanianvoid Sema::DiagnoseUnimplementedProperties(Scope *S, ObjCImplDecl* IMPDecl,
16369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                      ObjCContainerDecl *CDecl,
1637811bfcd823d51db6a06b3fd5a5943ff910b79d68Benjamin Kramer                                      const SelectorSet &InsMap) {
1638cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap;
1639cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian  if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl))
1640cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian    CollectSuperClassPropertyImplementations(IDecl, SuperPropMap);
1641cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian
16429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap;
1643cfa6a27f3cc5add888c6ac84dbcc45854cfd8666Fariborz Jahanian  CollectImmediateProperties(CDecl, PropMap, SuperPropMap);
16449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (PropMap.empty())
16459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
16469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
16479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  llvm::DenseSet<ObjCPropertyDecl *> PropImplMap;
16489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCImplDecl::propimpl_iterator
16499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       I = IMPDecl->propimpl_begin(),
16509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       EI = IMPDecl->propimpl_end(); I != EI; ++I)
1651262bc18e32500558af7cb0afa205b34bd37bafedDavid Blaikie    PropImplMap.insert(I->getPropertyDecl());
16529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
16539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator
16549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       P = PropMap.begin(), E = PropMap.end(); P != E; ++P) {
16559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCPropertyDecl *Prop = P->second;
16569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Is there a matching propery synthesize/dynamic?
16579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (Prop->isInvalidDecl() ||
16589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional ||
1659327126ee3d2faad9314b2633974eefc672f73b79Fariborz Jahanian        PropImplMap.count(Prop) || Prop->hasAttr<UnavailableAttr>())
16609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      continue;
16619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!InsMap.count(Prop->getGetterName())) {
1662b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(IMPDecl->getLocation(),
16639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           isa<ObjCCategoryDecl>(CDecl) ?
16649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            diag::warn_setter_getter_impl_required_in_category :
16659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek            diag::warn_setter_getter_impl_required)
16669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Prop->getDeclName() << Prop->getGetterName();
1667b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(Prop->getLocation(),
1668b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian           diag::note_property_declare);
1669260611a32535c851237926bfcf78869b13c07d5bJohn McCall      if (LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCRuntime.isNonFragile())
1670da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian        if (ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(CDecl))
167171207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek          if (const ObjCInterfaceDecl *RID = ID->isObjCRequiresPropertyDefs())
1672da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian            Diag(RID->getLocation(), diag::note_suppressed_class_declare);
1673da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian
16749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
16759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
16769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!Prop->isReadOnly() && !InsMap.count(Prop->getSetterName())) {
1677b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(IMPDecl->getLocation(),
16789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           isa<ObjCCategoryDecl>(CDecl) ?
16799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           diag::warn_setter_getter_impl_required_in_category :
16809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           diag::warn_setter_getter_impl_required)
16819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      << Prop->getDeclName() << Prop->getSetterName();
1682b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian      Diag(Prop->getLocation(),
1683b8607398a5e1e552f82a86d1d8c3a4031ac4c946Fariborz Jahanian           diag::note_property_declare);
1684260611a32535c851237926bfcf78869b13c07d5bJohn McCall      if (LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCRuntime.isNonFragile())
1685da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian        if (ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(CDecl))
168671207fc0470e1eee40a2951cd5cc3ff47725b755Ted Kremenek          if (const ObjCInterfaceDecl *RID = ID->isObjCRequiresPropertyDefs())
1687da611a74c408af0f9526acc690b85214bf180852Fariborz Jahanian            Diag(RID->getLocation(), diag::note_suppressed_class_declare);
16889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
16899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
16909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
16919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
16929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid
16939d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::AtomicPropertySetterGetterRules (ObjCImplDecl* IMPDecl,
16949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                       ObjCContainerDecl* IDecl) {
16959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Rules apply in non-GC mode only
16964e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().getGC() != LangOptions::NonGC)
16979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    return;
16989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  for (ObjCContainerDecl::prop_iterator I = IDecl->prop_begin(),
16999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       E = IDecl->prop_end();
17009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek       I != E; ++I) {
1701581deb3da481053c4993c7600f97acf7768caac5David Blaikie    ObjCPropertyDecl *Property = *I;
170294659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    ObjCMethodDecl *GetterMethod = 0;
170394659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    ObjCMethodDecl *SetterMethod = 0;
170494659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    bool LookedUpGetterSetter = false;
170594659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis
17069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    unsigned Attributes = Property->getPropertyAttributes();
1707265941bc308d65cc270d5c4de5806f37ce405606John McCall    unsigned AttributesAsWritten = Property->getPropertyAttributesAsWritten();
170894659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis
1709265941bc308d65cc270d5c4de5806f37ce405606John McCall    if (!(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_atomic) &&
1710265941bc308d65cc270d5c4de5806f37ce405606John McCall        !(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_nonatomic)) {
171194659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      GetterMethod = IMPDecl->getInstanceMethod(Property->getGetterName());
171294659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      SetterMethod = IMPDecl->getInstanceMethod(Property->getSetterName());
171394659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      LookedUpGetterSetter = true;
171494659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      if (GetterMethod) {
171594659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(GetterMethod->getLocation(),
171694659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis             diag::warn_default_atomic_custom_getter_setter)
1717293a45e724a15fb58b8805a5791f9f3aee769cf6Argyrios Kyrtzidis          << Property->getIdentifier() << 0;
171894659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(Property->getLocation(), diag::note_property_declare);
171994659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      }
172094659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      if (SetterMethod) {
172194659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(SetterMethod->getLocation(),
172294659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis             diag::warn_default_atomic_custom_getter_setter)
1723293a45e724a15fb58b8805a5791f9f3aee769cf6Argyrios Kyrtzidis          << Property->getIdentifier() << 1;
172494659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        Diag(Property->getLocation(), diag::note_property_declare);
172594659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      }
172694659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis    }
172794659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis
17289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // We only care about readwrite atomic property.
17299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if ((Attributes & ObjCPropertyDecl::OBJC_PR_nonatomic) ||
17309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        !(Attributes & ObjCPropertyDecl::OBJC_PR_readwrite))
17319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      continue;
17329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (const ObjCPropertyImplDecl *PIDecl
17339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek         = IMPDecl->FindPropertyImplDecl(Property->getIdentifier())) {
17349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if (PIDecl->getPropertyImplementation() == ObjCPropertyImplDecl::Dynamic)
17359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        continue;
173694659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      if (!LookedUpGetterSetter) {
173794659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        GetterMethod = IMPDecl->getInstanceMethod(Property->getGetterName());
173894659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        SetterMethod = IMPDecl->getInstanceMethod(Property->getSetterName());
173994659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis        LookedUpGetterSetter = true;
174094659e4bdb87534f591ae185812548c42d6efacbArgyrios Kyrtzidis      }
17419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      if ((GetterMethod && !SetterMethod) || (!GetterMethod && SetterMethod)) {
17429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        SourceLocation MethodLoc =
17439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          (GetterMethod ? GetterMethod->getLocation()
17449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                        : SetterMethod->getLocation());
17459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Diag(MethodLoc, diag::warn_atomic_property_rule)
17467d65f6965d215f4cb2eb0738ee1b9024e5ab5bbaFariborz Jahanian          << Property->getIdentifier() << (GetterMethod != 0)
17477d65f6965d215f4cb2eb0738ee1b9024e5ab5bbaFariborz Jahanian          << (SetterMethod != 0);
174877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        // fixit stuff.
174977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        if (!AttributesAsWritten) {
175077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          if (Property->getLParenLoc().isValid()) {
175177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            // @property () ... case.
175277bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            SourceRange PropSourceRange(Property->getAtLoc(),
175377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian                                        Property->getLParenLoc());
175477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            Diag(Property->getLocation(), diag::note_atomic_property_fixup_suggest) <<
175577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian              FixItHint::CreateReplacement(PropSourceRange, "@property (nonatomic");
175677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          }
175777bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          else {
175877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            //@property id etc.
175977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            SourceLocation endLoc =
176077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian              Property->getTypeSourceInfo()->getTypeLoc().getBeginLoc();
176177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            endLoc = endLoc.getLocWithOffset(-1);
176277bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            SourceRange PropSourceRange(Property->getAtLoc(), endLoc);
176377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian            Diag(Property->getLocation(), diag::note_atomic_property_fixup_suggest) <<
176477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian              FixItHint::CreateReplacement(PropSourceRange, "@property (nonatomic) ");
176577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          }
176677bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        }
176777bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        else if (!(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_atomic)) {
176877bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          // @property () ... case.
176977bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          SourceLocation endLoc = Property->getLParenLoc();
177077bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          SourceRange PropSourceRange(Property->getAtLoc(), endLoc);
177177bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          Diag(Property->getLocation(), diag::note_atomic_property_fixup_suggest) <<
177277bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian           FixItHint::CreateReplacement(PropSourceRange, "@property (nonatomic, ");
177377bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        }
177477bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian        else
177577bfb8b43ec3f7dee3a71f6e854b03ad29dab84fFariborz Jahanian          Diag(MethodLoc, diag::note_atomic_property_fixup_suggest);
17769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Diag(Property->getLocation(), diag::note_property_declare);
17779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      }
17789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
17799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
17809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
17819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1782f85e193739c953358c865005855253af4f68a497John McCallvoid Sema::DiagnoseOwningPropertyGetterSynthesis(const ObjCImplementationDecl *D) {
17834e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().getGC() == LangOptions::GCOnly)
1784f85e193739c953358c865005855253af4f68a497John McCall    return;
1785f85e193739c953358c865005855253af4f68a497John McCall
1786f85e193739c953358c865005855253af4f68a497John McCall  for (ObjCImplementationDecl::propimpl_iterator
1787f85e193739c953358c865005855253af4f68a497John McCall         i = D->propimpl_begin(), e = D->propimpl_end(); i != e; ++i) {
1788581deb3da481053c4993c7600f97acf7768caac5David Blaikie    ObjCPropertyImplDecl *PID = *i;
1789f85e193739c953358c865005855253af4f68a497John McCall    if (PID->getPropertyImplementation() != ObjCPropertyImplDecl::Synthesize)
1790f85e193739c953358c865005855253af4f68a497John McCall      continue;
1791f85e193739c953358c865005855253af4f68a497John McCall
1792f85e193739c953358c865005855253af4f68a497John McCall    const ObjCPropertyDecl *PD = PID->getPropertyDecl();
1793831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    if (PD && !PD->hasAttr<NSReturnsNotRetainedAttr>() &&
1794831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian        !D->getInstanceMethod(PD->getGetterName())) {
1795f85e193739c953358c865005855253af4f68a497John McCall      ObjCMethodDecl *method = PD->getGetterMethodDecl();
1796f85e193739c953358c865005855253af4f68a497John McCall      if (!method)
1797f85e193739c953358c865005855253af4f68a497John McCall        continue;
1798f85e193739c953358c865005855253af4f68a497John McCall      ObjCMethodFamily family = method->getMethodFamily();
1799f85e193739c953358c865005855253af4f68a497John McCall      if (family == OMF_alloc || family == OMF_copy ||
1800f85e193739c953358c865005855253af4f68a497John McCall          family == OMF_mutableCopy || family == OMF_new) {
18014e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        if (getLangOpts().ObjCAutoRefCount)
1802f85e193739c953358c865005855253af4f68a497John McCall          Diag(PID->getLocation(), diag::err_ownin_getter_rule);
1803f85e193739c953358c865005855253af4f68a497John McCall        else
1804920c9c1155c118ac40e5209d4b757b9f54186339Ted Kremenek          Diag(PID->getLocation(), diag::warn_owning_getter_rule);
1805f85e193739c953358c865005855253af4f68a497John McCall        Diag(PD->getLocation(), diag::note_property_declare);
1806f85e193739c953358c865005855253af4f68a497John McCall      }
1807f85e193739c953358c865005855253af4f68a497John McCall    }
1808f85e193739c953358c865005855253af4f68a497John McCall  }
1809f85e193739c953358c865005855253af4f68a497John McCall}
1810f85e193739c953358c865005855253af4f68a497John McCall
18115de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall/// AddPropertyAttrs - Propagates attributes from a property to the
18125de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall/// implicitly-declared getter or setter for that property.
18135de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCallstatic void AddPropertyAttrs(Sema &S, ObjCMethodDecl *PropertyMethod,
18145de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall                             ObjCPropertyDecl *Property) {
18155de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall  // Should we just clone all attributes over?
18160a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor  for (Decl::attr_iterator A = Property->attr_begin(),
18170a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor                        AEnd = Property->attr_end();
18180a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor       A != AEnd; ++A) {
18190a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor    if (isa<DeprecatedAttr>(*A) ||
18200a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor        isa<UnavailableAttr>(*A) ||
18210a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor        isa<AvailabilityAttr>(*A))
18220a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor      PropertyMethod->addAttr((*A)->clone(S.Context));
18230a0d2b179085a52c10402feebeb6db8b4d96a140Douglas Gregor  }
18245de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall}
18255de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
18269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// ProcessPropertyDecl - Make sure that any user-defined setter/getter methods
18279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// have the property type and issue diagnostics if they don't.
18289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// Also synthesize a getter/setter method if none exist (and update the
18299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// appropriate lookup tables. FIXME: Should reconsider if adding synthesized
18309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// methods is the "right" thing to do.
18319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::ProcessPropertyDecl(ObjCPropertyDecl *property,
18328254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               ObjCContainerDecl *CD,
18338254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               ObjCPropertyDecl *redeclaredProperty,
18348254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               ObjCContainerDecl *lexicalDC) {
18358254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek
18369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  ObjCMethodDecl *GetterMethod, *SetterMethod;
18379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
18389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  GetterMethod = CD->getInstanceMethod(property->getGetterName());
18399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  SetterMethod = CD->getInstanceMethod(property->getSetterName());
18409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  DiagnosePropertyAccessorMismatch(property, GetterMethod,
18419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                   property->getLocation());
18429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
18439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (SetterMethod) {
18449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    ObjCPropertyDecl::PropertyAttributeKind CAttr =
18459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      property->getPropertyAttributes();
18469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if ((!(CAttr & ObjCPropertyDecl::OBJC_PR_readonly)) &&
18479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        Context.getCanonicalType(SetterMethod->getResultType()) !=
18489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek          Context.VoidTy)
18499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Diag(SetterMethod->getLocation(), diag::err_setter_type_void);
18509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (SetterMethod->param_size() != 1 ||
18512aac0c9f8bc4dca3f883f429af8c1f57132d62d0Fariborz Jahanian        !Context.hasSameUnqualifiedType(
1852bb13c320fff4bc4b3536e62626c97d7b055c6113Fariborz Jahanian          (*SetterMethod->param_begin())->getType().getNonReferenceType(),
1853bb13c320fff4bc4b3536e62626c97d7b055c6113Fariborz Jahanian          property->getType().getNonReferenceType())) {
18549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Diag(property->getLocation(),
18559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek           diag::warn_accessor_property_type_mismatch)
18569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        << property->getDeclName()
18579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek        << SetterMethod->getSelector();
18589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      Diag(SetterMethod->getLocation(), diag::note_declared_at);
18599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    }
18609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
18619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
18629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Synthesize getter/setter methods if none exist.
18639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Find the default getter and if one not found, add one.
18649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // FIXME: The synthesized property we set here is misleading. We almost always
18659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // synthesize these methods unless the user explicitly provided prototypes
18669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // (which is odd, but allowed). Sema should be typechecking that the
18679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // declarations jive in that situation (which it is not currently).
18689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!GetterMethod) {
18699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // No instance method of same name as property getter name was found.
18709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Declare a getter method and add it to the list of methods
18719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // for this class.
1872a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek    SourceLocation Loc = redeclaredProperty ?
1873a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek      redeclaredProperty->getLocation() :
1874a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek      property->getLocation();
1875a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek
1876a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek    GetterMethod = ObjCMethodDecl::Create(Context, Loc, Loc,
1877a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek                             property->getGetterName(),
187875cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                             property->getType(), 0, CD, /*isInstance=*/true,
187975cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                             /*isVariadic=*/false, /*isSynthesized=*/true,
188075cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                             /*isImplicitlyDeclared=*/true, /*isDefined=*/false,
18819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                             (property->getPropertyImplementation() ==
18829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                              ObjCPropertyDecl::Optional) ?
18839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                             ObjCMethodDecl::Optional :
18849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                             ObjCMethodDecl::Required);
18859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    CD->addDecl(GetterMethod);
18865de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
18875de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall    AddPropertyAttrs(*this, GetterMethod, property);
18885de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
188923173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek    // FIXME: Eventually this shouldn't be needed, as the lexical context
189023173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek    // and the real context should be the same.
1891a054fb46b1fb596d1719b89d2d9a5be3c32a4b0dTed Kremenek    if (lexicalDC)
189223173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      GetterMethod->setLexicalDeclContext(lexicalDC);
1893831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian    if (property->hasAttr<NSReturnsNotRetainedAttr>())
1894831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian      GetterMethod->addAttr(
1895831fb9622581fc3b777848e6b097a0cb23d124deFariborz Jahanian        ::new (Context) NSReturnsNotRetainedAttr(Loc, Context));
18969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  } else
18979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // A user declared getter will be synthesize when @synthesize of
18989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // the property with the same name is seen in the @implementation
18999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    GetterMethod->setSynthesized(true);
19009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  property->setGetterMethodDecl(GetterMethod);
19019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
19029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Skip setter if property is read-only.
19039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (!property->isReadOnly()) {
19049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    // Find the default setter and if one not found, add one.
19059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    if (!SetterMethod) {
19069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // No instance method of same name as property setter name was found.
19079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Declare a setter method and add it to the list of methods
19089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // for this class.
19098254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek      SourceLocation Loc = redeclaredProperty ?
19108254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek        redeclaredProperty->getLocation() :
19118254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek        property->getLocation();
19128254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek
19138254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek      SetterMethod =
1914491306a83c4f0f49f95a3bcbca8580cb98a91c7aArgyrios Kyrtzidis        ObjCMethodDecl::Create(Context, Loc, Loc,
19158254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                               property->getSetterName(), Context.VoidTy, 0,
191675cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               CD, /*isInstance=*/true, /*isVariadic=*/false,
191775cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               /*isSynthesized=*/true,
191875cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               /*isImplicitlyDeclared=*/true,
191975cf3e86d33ce810c12084126385371b335c30baArgyrios Kyrtzidis                               /*isDefined=*/false,
19209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                               (property->getPropertyImplementation() ==
19219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                ObjCPropertyDecl::Optional) ?
19228254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                                ObjCMethodDecl::Optional :
19238254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek                                ObjCMethodDecl::Required);
19248254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek
19259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // Invent the arguments for the setter. We don't bother making a
19269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // nice name for the argument.
1927ff676cb48fe8bf7be2feaa251dc7c5fb15af4730Abramo Bagnara      ParmVarDecl *Argument = ParmVarDecl::Create(Context, SetterMethod,
1928ff676cb48fe8bf7be2feaa251dc7c5fb15af4730Abramo Bagnara                                                  Loc, Loc,
19299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                                  property->getIdentifier(),
1930f85e193739c953358c865005855253af4f68a497John McCall                                    property->getType().getUnqualifiedType(),
19319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                                  /*TInfo=*/0,
1932d931b086984257de68868a64a235c2b4b34003fbJohn McCall                                                  SC_None,
1933d931b086984257de68868a64a235c2b4b34003fbJohn McCall                                                  SC_None,
19349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                                  0);
1935491306a83c4f0f49f95a3bcbca8580cb98a91c7aArgyrios Kyrtzidis      SetterMethod->setMethodParams(Context, Argument,
1936491306a83c4f0f49f95a3bcbca8580cb98a91c7aArgyrios Kyrtzidis                                    ArrayRef<SourceLocation>());
19375de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
19385de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall      AddPropertyAttrs(*this, SetterMethod, property);
19395de74d19ddcf84cfe9d7d973192ba93fd0c64d4fJohn McCall
19409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      CD->addDecl(SetterMethod);
194123173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      // FIXME: Eventually this shouldn't be needed, as the lexical context
194223173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek      // and the real context should be the same.
19438254aa62d9189395be1eed61194cd4b5ee6cb4a2Ted Kremenek      if (lexicalDC)
194423173d7f029f430611caceea72ae61ba6b80af1cTed Kremenek        SetterMethod->setLexicalDeclContext(lexicalDC);
19459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    } else
19469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // A user declared setter will be synthesize when @synthesize of
19479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      // the property with the same name is seen in the @implementation
19489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek      SetterMethod->setSynthesized(true);
19499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    property->setSetterMethodDecl(SetterMethod);
19509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  }
19519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // Add any synthesized methods to the global pool. This allows us to
19529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // handle the following, which is supported by GCC (and part of the design).
19539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //
19549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // @interface Foo
19559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // @property double bar;
19569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // @end
19579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //
19589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // void thisIsUnfortunate() {
19599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //   id foo;
19609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //   double bar = [foo bar];
19619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // }
19629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  //
19639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (GetterMethod)
19649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    AddInstanceMethodToGlobalPool(GetterMethod);
19659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  if (SetterMethod)
19669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek    AddInstanceMethodToGlobalPool(SetterMethod);
1967e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis
1968e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  ObjCInterfaceDecl *CurrentClass = dyn_cast<ObjCInterfaceDecl>(CD);
1969e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  if (!CurrentClass) {
1970e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    if (ObjCCategoryDecl *Cat = dyn_cast<ObjCCategoryDecl>(CD))
1971e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis      CurrentClass = Cat->getClassInterface();
1972e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    else if (ObjCImplDecl *Impl = dyn_cast<ObjCImplDecl>(CD))
1973e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis      CurrentClass = Impl->getClassInterface();
1974e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  }
1975e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  if (GetterMethod)
1976e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    CheckObjCMethodOverrides(GetterMethod, CurrentClass, Sema::RTC_Unknown);
1977e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis  if (SetterMethod)
1978e15db6f0d226a3bc88d244512d1004c7c1c07391Argyrios Kyrtzidis    CheckObjCMethodOverrides(SetterMethod, CurrentClass, Sema::RTC_Unknown);
19799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
19809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
1981d226f65006733ed7f709c3174f22ce33391cb58fJohn McCallvoid Sema::CheckObjCPropertyAttributes(Decl *PDecl,
19829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek                                       SourceLocation Loc,
1983cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                                       unsigned &Attributes,
1984cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian                                       bool propertyInPrimaryClass) {
19859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // FIXME: Improve the reported location.
1986f85e193739c953358c865005855253af4f68a497John McCall  if (!PDecl || PDecl->isInvalidDecl())
19875fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek    return;
19885fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek
19895fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek  ObjCPropertyDecl *PropertyDecl = cast<ObjCPropertyDecl>(PDecl);
1990842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian  QualType PropertyTy = PropertyDecl->getType();
19919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
19924e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie  if (getLangOpts().ObjCAutoRefCount &&
1993015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
1994015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      PropertyTy->isObjCRetainableType()) {
1995015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // 'readonly' property with no obvious lifetime.
1996015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    // its life time will be determined by its backing ivar.
1997015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    unsigned rel = (ObjCDeclSpec::DQ_PR_unsafe_unretained |
1998015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_copy |
1999015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_retain |
2000015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_strong |
2001015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_weak |
2002015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian                    ObjCDeclSpec::DQ_PR_assign);
2003015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian    if ((Attributes & rel) == 0)
2004015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian      return;
2005015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian  }
2006015f608042a1d247899866849e69a4e9a1aff6a8Fariborz Jahanian
2007cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  if (propertyInPrimaryClass) {
2008cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    // we postpone most property diagnosis until class's implementation
2009cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    // because, its readonly attribute may be overridden in its class
2010cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    // extensions making other attributes, which make no sense, to make sense.
2011cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian    if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
2012cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        (Attributes & ObjCDeclSpec::DQ_PR_readwrite))
2013cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2014cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian        << "readonly" << "readwrite";
2015cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  }
20169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek  // readonly and readwrite/assign/retain/copy conflict.
2017cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian  else if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
2018cea06d26b8f3a2599bba79f7e072b7550de949a7Fariborz Jahanian           (Attributes & (ObjCDeclSpec::DQ_PR_readwrite |
201928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                     ObjCDeclSpec::DQ_PR_assign |
2020f85e193739c953358c865005855253af4f68a497John McCall                     ObjCDeclSpec::DQ_PR_unsafe_unretained |
202128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                     ObjCDeclSpec::DQ_PR_copy |
2022f85e193739c953358c865005855253af4f68a497John McCall                     ObjCDeclSpec::DQ_PR_retain |
2023f85e193739c953358c865005855253af4f68a497John McCall                     ObjCDeclSpec::DQ_PR_strong))) {
202428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    const char * which = (Attributes & ObjCDeclSpec::DQ_PR_readwrite) ?
202528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                          "readwrite" :
202628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                         (Attributes & ObjCDeclSpec::DQ_PR_assign) ?
202728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                          "assign" :
2028f85e193739c953358c865005855253af4f68a497John McCall                         (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) ?
2029f85e193739c953358c865005855253af4f68a497John McCall                          "unsafe_unretained" :
203028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                         (Attributes & ObjCDeclSpec::DQ_PR_copy) ?
203128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                          "copy" : "retain";
20329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
203328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(Loc, (Attributes & (ObjCDeclSpec::DQ_PR_readwrite)) ?
203428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                 diag::err_objc_property_attr_mutually_exclusive :
203528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek                 diag::warn_objc_property_attr_mutually_exclusive)
203628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      << "readonly" << which;
203728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
20389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
203928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Check for copy or retain on non-object types.
2040f85e193739c953358c865005855253af4f68a497John McCall  if ((Attributes & (ObjCDeclSpec::DQ_PR_weak | ObjCDeclSpec::DQ_PR_copy |
2041f85e193739c953358c865005855253af4f68a497John McCall                    ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong)) &&
2042f85e193739c953358c865005855253af4f68a497John McCall      !PropertyTy->isObjCRetainableType() &&
2043842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian      !PropertyDecl->getAttr<ObjCNSObjectAttr>()) {
204428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(Loc, diag::err_objc_property_requires_object)
2045f85e193739c953358c865005855253af4f68a497John McCall      << (Attributes & ObjCDeclSpec::DQ_PR_weak ? "weak" :
2046f85e193739c953358c865005855253af4f68a497John McCall          Attributes & ObjCDeclSpec::DQ_PR_copy ? "copy" : "retain (or strong)");
2047f85e193739c953358c865005855253af4f68a497John McCall    Attributes &= ~(ObjCDeclSpec::DQ_PR_weak   | ObjCDeclSpec::DQ_PR_copy |
2048f85e193739c953358c865005855253af4f68a497John McCall                    ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong);
2049977ea7864a50eee39f6bef8a5a79154ac9ba4b3eJohn McCall    PropertyDecl->setInvalidDecl();
205028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
20519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
205228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Check for more than one of { assign, copy, retain }.
205328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (Attributes & ObjCDeclSpec::DQ_PR_assign) {
205428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
205528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
205628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << "assign" << "copy";
205728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Attributes &= ~ObjCDeclSpec::DQ_PR_copy;
205828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
205928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
206028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
206128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << "assign" << "retain";
206228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
206328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
2064f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
2065f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2066f85e193739c953358c865005855253af4f68a497John McCall        << "assign" << "strong";
2067f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
2068f85e193739c953358c865005855253af4f68a497John McCall    }
20694e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount  &&
2070f85e193739c953358c865005855253af4f68a497John McCall        (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
2071f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2072f85e193739c953358c865005855253af4f68a497John McCall        << "assign" << "weak";
2073f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
2074f85e193739c953358c865005855253af4f68a497John McCall    }
2075f85e193739c953358c865005855253af4f68a497John McCall  } else if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) {
2076f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
2077f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2078f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "copy";
2079f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_copy;
2080f85e193739c953358c865005855253af4f68a497John McCall    }
2081f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
2082f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2083f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "retain";
2084f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
2085f85e193739c953358c865005855253af4f68a497John McCall    }
2086f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
2087f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2088f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "strong";
2089f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
2090f85e193739c953358c865005855253af4f68a497John McCall    }
20914e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie    if (getLangOpts().ObjCAutoRefCount  &&
2092f85e193739c953358c865005855253af4f68a497John McCall        (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
2093f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2094f85e193739c953358c865005855253af4f68a497John McCall        << "unsafe_unretained" << "weak";
2095f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
2096f85e193739c953358c865005855253af4f68a497John McCall    }
209728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  } else if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
209828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
209928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
210028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek        << "copy" << "retain";
210128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
210228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    }
2103f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
2104f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2105f85e193739c953358c865005855253af4f68a497John McCall        << "copy" << "strong";
2106f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
2107f85e193739c953358c865005855253af4f68a497John McCall    }
2108f85e193739c953358c865005855253af4f68a497John McCall    if (Attributes & ObjCDeclSpec::DQ_PR_weak) {
2109f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2110f85e193739c953358c865005855253af4f68a497John McCall        << "copy" << "weak";
2111f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
2112f85e193739c953358c865005855253af4f68a497John McCall    }
2113f85e193739c953358c865005855253af4f68a497John McCall  }
2114f85e193739c953358c865005855253af4f68a497John McCall  else if ((Attributes & ObjCDeclSpec::DQ_PR_retain) &&
2115f85e193739c953358c865005855253af4f68a497John McCall           (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
2116f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2117f85e193739c953358c865005855253af4f68a497John McCall        << "retain" << "weak";
2118528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian      Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
2119f85e193739c953358c865005855253af4f68a497John McCall  }
2120f85e193739c953358c865005855253af4f68a497John McCall  else if ((Attributes & ObjCDeclSpec::DQ_PR_strong) &&
2121f85e193739c953358c865005855253af4f68a497John McCall           (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
2122f85e193739c953358c865005855253af4f68a497John McCall      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
2123f85e193739c953358c865005855253af4f68a497John McCall        << "strong" << "weak";
2124f85e193739c953358c865005855253af4f68a497John McCall      Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
212528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
21269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
21279d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian  if ((Attributes & ObjCDeclSpec::DQ_PR_atomic) &&
21289d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)) {
21299d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian      Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
21309d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian        << "atomic" << "nonatomic";
21319d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian      Attributes &= ~ObjCDeclSpec::DQ_PR_atomic;
21329d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian  }
21339d1bbeab2475fe45a3d4cb8de6810bc3275f1dd7Fariborz Jahanian
213428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // Warn if user supplied no assignment attribute, property is
213528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  // readwrite, and this is an object type.
213628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (!(Attributes & (ObjCDeclSpec::DQ_PR_assign | ObjCDeclSpec::DQ_PR_copy |
2137f85e193739c953358c865005855253af4f68a497John McCall                      ObjCDeclSpec::DQ_PR_unsafe_unretained |
2138f85e193739c953358c865005855253af4f68a497John McCall                      ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong |
2139f85e193739c953358c865005855253af4f68a497John McCall                      ObjCDeclSpec::DQ_PR_weak)) &&
214028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      PropertyTy->isObjCObjectPointerType()) {
21414e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      if (getLangOpts().ObjCAutoRefCount)
2142bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian        // With arc,  @property definitions should default to (strong) when
2143f21a92d35953adb038ffbe37cd1a4083e0df0ec0Fariborz Jahanian        // not specified; including when property is 'readonly'.
2144bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian        PropertyDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
2145f21a92d35953adb038ffbe37cd1a4083e0df0ec0Fariborz Jahanian      else if (!(Attributes & ObjCDeclSpec::DQ_PR_readonly)) {
21469f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        bool isAnyClassTy =
21479f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian          (PropertyTy->isObjCClassType() ||
21489f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian           PropertyTy->isObjCQualifiedClassType());
21499f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        // In non-gc, non-arc mode, 'Class' is treated as a 'void *' no need to
21509f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        // issue any warning.
21514e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie        if (isAnyClassTy && getLangOpts().getGC() == LangOptions::NonGC)
21529f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian          ;
21539f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        else {
2154bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian          // Skip this warning in gc-only mode.
21554e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          if (getLangOpts().getGC() != LangOptions::GCOnly)
2156bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian            Diag(Loc, diag::warn_objc_property_no_assignment_attribute);
21570a68dc7f04be1542ce249ff4adb169453698ad91Argyrios Kyrtzidis
2158bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian          // If non-gc code warn that this is likely inappropriate.
21594e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie          if (getLangOpts().getGC() == LangOptions::NonGC)
2160bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian            Diag(Loc, diag::warn_objc_property_default_assign_on_object);
21619f37cd1810646ffc9002c7a6477fe158ee15ede4Fariborz Jahanian        }
2162bc03aea4bec84d7abecabd26b8583adb687c4089Fariborz Jahanian      }
21639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
216428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // FIXME: Implement warning dependent on NSCopying being
216528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // implemented. See also:
216628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // <rdar://5168496&4855821&5607453&5096644&4947311&5698469&4947014&5168496>
216728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    // (please trim this list while you are at it).
216828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  }
21699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek
217028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek  if (!(Attributes & ObjCDeclSpec::DQ_PR_copy)
21712b77cb803c76d01d2d10a58cec9334e2090cd7c3Fariborz Jahanian      &&!(Attributes & ObjCDeclSpec::DQ_PR_readonly)
21724e4d08403ca5cfd4d558fa2936215d3a4e5a528dDavid Blaikie      && getLangOpts().getGC() == LangOptions::GCOnly
217328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek      && PropertyTy->isBlockPointerType())
217428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek    Diag(Loc, diag::warn_objc_property_copy_missing_on_block);
21757c16d58f8546365d1f56468e9ae1ecfa182afca0Fariborz Jahanian  else if ((Attributes & ObjCDeclSpec::DQ_PR_retain) &&
2176528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           !(Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
2177528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           !(Attributes & ObjCDeclSpec::DQ_PR_strong) &&
2178528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian           PropertyTy->isBlockPointerType())
2179528a499eb84d61667f65b16a13780c135b822f6bFariborz Jahanian      Diag(Loc, diag::warn_objc_property_retain_of_block);
218048a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian
218148a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian  if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
218248a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian      (Attributes & ObjCDeclSpec::DQ_PR_setter))
218348a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian    Diag(Loc, diag::warn_objc_readonly_property_has_setter);
218448a98c7a8467a9570d2fc7f2aab7f5273a3e218eFariborz Jahanian
21859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek}
2186