blob: bbc6d233854d4228dae3f513f4d6ee7d1ccc7dea [file] [log] [blame]
Ted Kremenek9d64c152010-03-12 00:38:38 +00001//===--- SemaObjCProperty.cpp - Semantic Analysis for ObjC @property ------===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This file implements semantic analysis for Objective C @property and
11// @synthesize declarations.
12//
13//===----------------------------------------------------------------------===//
14
John McCall2d887082010-08-25 22:03:47 +000015#include "clang/Sema/SemaInternal.h"
Douglas Gregore737f502010-08-12 20:07:10 +000016#include "clang/Sema/Initialization.h"
John McCall7cd088e2010-08-24 07:21:54 +000017#include "clang/AST/DeclObjC.h"
Fariborz Jahanian17cb3262010-05-05 21:52:17 +000018#include "clang/AST/ExprObjC.h"
Fariborz Jahanian57e264e2011-10-06 18:38:18 +000019#include "clang/AST/ExprCXX.h"
Argyrios Kyrtzidisc80553e2011-11-14 04:52:29 +000020#include "clang/AST/ASTMutationListener.h"
John McCall50df6ae2010-08-25 07:03:20 +000021#include "llvm/ADT/DenseSet.h"
Ted Kremenek9d64c152010-03-12 00:38:38 +000022
23using namespace clang;
24
Ted Kremenek28685ab2010-03-12 00:46:40 +000025//===----------------------------------------------------------------------===//
26// Grammar actions.
27//===----------------------------------------------------------------------===//
28
John McCall265941b2011-09-13 18:31:23 +000029/// getImpliedARCOwnership - Given a set of property attributes and a
30/// type, infer an expected lifetime. The type's ownership qualification
31/// is not considered.
32///
33/// Returns OCL_None if the attributes as stated do not imply an ownership.
34/// Never returns OCL_Autoreleasing.
35static Qualifiers::ObjCLifetime getImpliedARCOwnership(
36 ObjCPropertyDecl::PropertyAttributeKind attrs,
37 QualType type) {
38 // retain, strong, copy, weak, and unsafe_unretained are only legal
39 // on properties of retainable pointer type.
40 if (attrs & (ObjCPropertyDecl::OBJC_PR_retain |
41 ObjCPropertyDecl::OBJC_PR_strong |
42 ObjCPropertyDecl::OBJC_PR_copy)) {
Fariborz Jahanian5fa065b2011-10-13 23:45:45 +000043 return type->getObjCARCImplicitLifetime();
John McCall265941b2011-09-13 18:31:23 +000044 } else if (attrs & ObjCPropertyDecl::OBJC_PR_weak) {
45 return Qualifiers::OCL_Weak;
46 } else if (attrs & ObjCPropertyDecl::OBJC_PR_unsafe_unretained) {
47 return Qualifiers::OCL_ExplicitNone;
48 }
49
50 // assign can appear on other types, so we have to check the
51 // property type.
52 if (attrs & ObjCPropertyDecl::OBJC_PR_assign &&
53 type->isObjCRetainableType()) {
54 return Qualifiers::OCL_ExplicitNone;
55 }
56
57 return Qualifiers::OCL_None;
58}
59
John McCallf85e1932011-06-15 23:02:42 +000060/// Check the internal consistency of a property declaration.
61static void checkARCPropertyDecl(Sema &S, ObjCPropertyDecl *property) {
62 if (property->isInvalidDecl()) return;
63
64 ObjCPropertyDecl::PropertyAttributeKind propertyKind
65 = property->getPropertyAttributes();
66 Qualifiers::ObjCLifetime propertyLifetime
67 = property->getType().getObjCLifetime();
68
69 // Nothing to do if we don't have a lifetime.
70 if (propertyLifetime == Qualifiers::OCL_None) return;
71
John McCall265941b2011-09-13 18:31:23 +000072 Qualifiers::ObjCLifetime expectedLifetime
73 = getImpliedARCOwnership(propertyKind, property->getType());
74 if (!expectedLifetime) {
John McCallf85e1932011-06-15 23:02:42 +000075 // We have a lifetime qualifier but no dominating property
John McCall265941b2011-09-13 18:31:23 +000076 // attribute. That's okay, but restore reasonable invariants by
77 // setting the property attribute according to the lifetime
78 // qualifier.
79 ObjCPropertyDecl::PropertyAttributeKind attr;
80 if (propertyLifetime == Qualifiers::OCL_Strong) {
81 attr = ObjCPropertyDecl::OBJC_PR_strong;
82 } else if (propertyLifetime == Qualifiers::OCL_Weak) {
83 attr = ObjCPropertyDecl::OBJC_PR_weak;
84 } else {
85 assert(propertyLifetime == Qualifiers::OCL_ExplicitNone);
86 attr = ObjCPropertyDecl::OBJC_PR_unsafe_unretained;
87 }
88 property->setPropertyAttributes(attr);
John McCallf85e1932011-06-15 23:02:42 +000089 return;
90 }
91
92 if (propertyLifetime == expectedLifetime) return;
93
94 property->setInvalidDecl();
95 S.Diag(property->getLocation(),
Argyrios Kyrtzidisb8b03132011-06-24 00:08:59 +000096 diag::err_arc_inconsistent_property_ownership)
John McCallf85e1932011-06-15 23:02:42 +000097 << property->getDeclName()
John McCall265941b2011-09-13 18:31:23 +000098 << expectedLifetime
John McCallf85e1932011-06-15 23:02:42 +000099 << propertyLifetime;
100}
101
John McCalld226f652010-08-21 09:40:31 +0000102Decl *Sema::ActOnProperty(Scope *S, SourceLocation AtLoc,
103 FieldDeclarator &FD,
104 ObjCDeclSpec &ODS,
105 Selector GetterSel,
106 Selector SetterSel,
John McCalld226f652010-08-21 09:40:31 +0000107 bool *isOverridingProperty,
Ted Kremenek4a2e9ea2010-09-23 21:18:05 +0000108 tok::ObjCKeywordKind MethodImplKind,
109 DeclContext *lexicalDC) {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000110 unsigned Attributes = ODS.getPropertyAttributes();
John McCallf85e1932011-06-15 23:02:42 +0000111 TypeSourceInfo *TSI = GetTypeForDeclarator(FD.D, S);
112 QualType T = TSI->getType();
Douglas Gregore289d812011-09-13 17:21:33 +0000113 if ((getLangOptions().getGC() != LangOptions::NonGC &&
John McCallf85e1932011-06-15 23:02:42 +0000114 T.isObjCGCWeak()) ||
115 (getLangOptions().ObjCAutoRefCount &&
116 T.getObjCLifetime() == Qualifiers::OCL_Weak))
117 Attributes |= ObjCDeclSpec::DQ_PR_weak;
118
Ted Kremenek28685ab2010-03-12 00:46:40 +0000119 bool isReadWrite = ((Attributes & ObjCDeclSpec::DQ_PR_readwrite) ||
120 // default is readwrite!
121 !(Attributes & ObjCDeclSpec::DQ_PR_readonly));
122 // property is defaulted to 'assign' if it is readwrite and is
123 // not retain or copy
124 bool isAssign = ((Attributes & ObjCDeclSpec::DQ_PR_assign) ||
125 (isReadWrite &&
126 !(Attributes & ObjCDeclSpec::DQ_PR_retain) &&
John McCallf85e1932011-06-15 23:02:42 +0000127 !(Attributes & ObjCDeclSpec::DQ_PR_strong) &&
128 !(Attributes & ObjCDeclSpec::DQ_PR_copy) &&
129 !(Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) &&
130 !(Attributes & ObjCDeclSpec::DQ_PR_weak)));
Fariborz Jahanian14086762011-03-28 23:47:18 +0000131
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000132 // Proceed with constructing the ObjCPropertDecls.
Fariborz Jahaniana28948f2011-08-22 15:54:49 +0000133 ObjCContainerDecl *ClassDecl = cast<ObjCContainerDecl>(CurContext);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000134
Ted Kremenek28685ab2010-03-12 00:46:40 +0000135 if (ObjCCategoryDecl *CDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl))
Fariborz Jahanianae415dc2010-07-13 22:04:56 +0000136 if (CDecl->IsClassExtension()) {
Fariborz Jahaniana28948f2011-08-22 15:54:49 +0000137 Decl *Res = HandlePropertyInClassExtension(S, AtLoc,
Fariborz Jahanianae415dc2010-07-13 22:04:56 +0000138 FD, GetterSel, SetterSel,
139 isAssign, isReadWrite,
140 Attributes,
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000141 ODS.getPropertyAttributes(),
Fariborz Jahanianae415dc2010-07-13 22:04:56 +0000142 isOverridingProperty, TSI,
143 MethodImplKind);
John McCallf85e1932011-06-15 23:02:42 +0000144 if (Res) {
Fariborz Jahanianae415dc2010-07-13 22:04:56 +0000145 CheckObjCPropertyAttributes(Res, AtLoc, Attributes);
John McCallf85e1932011-06-15 23:02:42 +0000146 if (getLangOptions().ObjCAutoRefCount)
147 checkARCPropertyDecl(*this, cast<ObjCPropertyDecl>(Res));
148 }
Fariborz Jahanianae415dc2010-07-13 22:04:56 +0000149 return Res;
150 }
151
John McCallf85e1932011-06-15 23:02:42 +0000152 ObjCPropertyDecl *Res = CreatePropertyDecl(S, ClassDecl, AtLoc, FD,
153 GetterSel, SetterSel,
154 isAssign, isReadWrite,
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000155 Attributes,
156 ODS.getPropertyAttributes(),
157 TSI, MethodImplKind);
Ted Kremenek4a2e9ea2010-09-23 21:18:05 +0000158 if (lexicalDC)
159 Res->setLexicalDeclContext(lexicalDC);
160
Fariborz Jahanian842f07b2010-03-30 22:40:11 +0000161 // Validate the attributes on the @property.
162 CheckObjCPropertyAttributes(Res, AtLoc, Attributes);
John McCallf85e1932011-06-15 23:02:42 +0000163
164 if (getLangOptions().ObjCAutoRefCount)
165 checkARCPropertyDecl(*this, Res);
166
Fariborz Jahanian842f07b2010-03-30 22:40:11 +0000167 return Res;
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000168}
Ted Kremenek2d2f9362010-03-12 00:49:00 +0000169
Argyrios Kyrtzidisb98ffde2011-10-18 19:49:16 +0000170static ObjCPropertyDecl::PropertyAttributeKind
171makePropertyAttributesAsWritten(unsigned Attributes) {
172 unsigned attributesAsWritten = 0;
173 if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
174 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_readonly;
175 if (Attributes & ObjCDeclSpec::DQ_PR_readwrite)
176 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_readwrite;
177 if (Attributes & ObjCDeclSpec::DQ_PR_getter)
178 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_getter;
179 if (Attributes & ObjCDeclSpec::DQ_PR_setter)
180 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_setter;
181 if (Attributes & ObjCDeclSpec::DQ_PR_assign)
182 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_assign;
183 if (Attributes & ObjCDeclSpec::DQ_PR_retain)
184 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_retain;
185 if (Attributes & ObjCDeclSpec::DQ_PR_strong)
186 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_strong;
187 if (Attributes & ObjCDeclSpec::DQ_PR_weak)
188 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_weak;
189 if (Attributes & ObjCDeclSpec::DQ_PR_copy)
190 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_copy;
191 if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
192 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_unsafe_unretained;
193 if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)
194 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_nonatomic;
195 if (Attributes & ObjCDeclSpec::DQ_PR_atomic)
196 attributesAsWritten |= ObjCPropertyDecl::OBJC_PR_atomic;
197
198 return (ObjCPropertyDecl::PropertyAttributeKind)attributesAsWritten;
199}
200
John McCalld226f652010-08-21 09:40:31 +0000201Decl *
Fariborz Jahaniana28948f2011-08-22 15:54:49 +0000202Sema::HandlePropertyInClassExtension(Scope *S,
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000203 SourceLocation AtLoc, FieldDeclarator &FD,
204 Selector GetterSel, Selector SetterSel,
205 const bool isAssign,
206 const bool isReadWrite,
207 const unsigned Attributes,
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000208 const unsigned AttributesAsWritten,
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000209 bool *isOverridingProperty,
John McCall83a230c2010-06-04 20:50:08 +0000210 TypeSourceInfo *T,
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000211 tok::ObjCKeywordKind MethodImplKind) {
Fariborz Jahanian58a76492011-08-22 18:34:22 +0000212 ObjCCategoryDecl *CDecl = cast<ObjCCategoryDecl>(CurContext);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000213 // Diagnose if this property is already in continuation class.
Fariborz Jahaniana28948f2011-08-22 15:54:49 +0000214 DeclContext *DC = CurContext;
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000215 IdentifierInfo *PropertyId = FD.D.getIdentifier();
Fariborz Jahanian2a289142010-11-10 18:01:36 +0000216 ObjCInterfaceDecl *CCPrimary = CDecl->getClassInterface();
217
218 if (CCPrimary)
219 // Check for duplicate declaration of this property in current and
220 // other class extensions.
221 for (const ObjCCategoryDecl *ClsExtDecl =
222 CCPrimary->getFirstClassExtension();
223 ClsExtDecl; ClsExtDecl = ClsExtDecl->getNextClassExtension()) {
224 if (ObjCPropertyDecl *prevDecl =
225 ObjCPropertyDecl::findPropertyDecl(ClsExtDecl, PropertyId)) {
226 Diag(AtLoc, diag::err_duplicate_property);
227 Diag(prevDecl->getLocation(), diag::note_property_declare);
228 return 0;
229 }
230 }
231
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000232 // Create a new ObjCPropertyDecl with the DeclContext being
233 // the class extension.
Fariborz Jahanian88f5e9b2010-12-10 23:36:33 +0000234 // FIXME. We should really be using CreatePropertyDecl for this.
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000235 ObjCPropertyDecl *PDecl =
236 ObjCPropertyDecl::Create(Context, DC, FD.D.getIdentifierLoc(),
237 PropertyId, AtLoc, T);
Argyrios Kyrtzidisb98ffde2011-10-18 19:49:16 +0000238 PDecl->setPropertyAttributesAsWritten(
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000239 makePropertyAttributesAsWritten(AttributesAsWritten));
Fariborz Jahanian22f757b2010-03-22 23:25:52 +0000240 if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
241 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly);
242 if (Attributes & ObjCDeclSpec::DQ_PR_readwrite)
243 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite);
Fariborz Jahanian88f5e9b2010-12-10 23:36:33 +0000244 // Set setter/getter selector name. Needed later.
245 PDecl->setGetterName(GetterSel);
246 PDecl->setSetterName(SetterSel);
Douglas Gregor91ae6b42011-07-15 15:30:21 +0000247 ProcessDeclAttributes(S, PDecl, FD.D);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000248 DC->addDecl(PDecl);
249
250 // We need to look in the @interface to see if the @property was
251 // already declared.
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000252 if (!CCPrimary) {
253 Diag(CDecl->getLocation(), diag::err_continuation_class);
254 *isOverridingProperty = true;
John McCalld226f652010-08-21 09:40:31 +0000255 return 0;
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000256 }
257
258 // Find the property in continuation class's primary class only.
259 ObjCPropertyDecl *PIDecl =
260 CCPrimary->FindPropertyVisibleInPrimaryClass(PropertyId);
261
262 if (!PIDecl) {
263 // No matching property found in the primary class. Just fall thru
264 // and add property to continuation class's primary class.
265 ObjCPropertyDecl *PDecl =
266 CreatePropertyDecl(S, CCPrimary, AtLoc,
267 FD, GetterSel, SetterSel, isAssign, isReadWrite,
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000268 Attributes,AttributesAsWritten, T, MethodImplKind, DC);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000269
270 // A case of continuation class adding a new property in the class. This
271 // is not what it was meant for. However, gcc supports it and so should we.
272 // Make sure setter/getters are declared here.
Ted Kremeneka054fb42010-09-21 20:52:59 +0000273 ProcessPropertyDecl(PDecl, CCPrimary, /* redeclaredProperty = */ 0,
274 /* lexicalDC = */ CDecl);
Argyrios Kyrtzidisc80553e2011-11-14 04:52:29 +0000275 if (ASTMutationListener *L = Context.getASTMutationListener())
276 L->AddedObjCPropertyInClassExtension(PDecl, /*OrigProp=*/0, CDecl);
John McCalld226f652010-08-21 09:40:31 +0000277 return PDecl;
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000278 }
Fariborz Jahaniana4b984d2011-09-24 00:56:59 +0000279 if (PIDecl->getType().getCanonicalType()
280 != PDecl->getType().getCanonicalType()) {
281 Diag(AtLoc,
Fariborz Jahaniand3c147f2011-11-28 18:38:27 +0000282 diag::err_type_mismatch_continuation_class) << PDecl->getType();
Fariborz Jahaniana4b984d2011-09-24 00:56:59 +0000283 Diag(PIDecl->getLocation(), diag::note_property_declare);
284 }
285
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000286 // The property 'PIDecl's readonly attribute will be over-ridden
287 // with continuation class's readwrite property attribute!
Fariborz Jahanian80aa1cd2010-06-22 23:20:40 +0000288 unsigned PIkind = PIDecl->getPropertyAttributesAsWritten();
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000289 if (isReadWrite && (PIkind & ObjCPropertyDecl::OBJC_PR_readonly)) {
290 unsigned retainCopyNonatomic =
291 (ObjCPropertyDecl::OBJC_PR_retain |
John McCallf85e1932011-06-15 23:02:42 +0000292 ObjCPropertyDecl::OBJC_PR_strong |
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000293 ObjCPropertyDecl::OBJC_PR_copy |
294 ObjCPropertyDecl::OBJC_PR_nonatomic);
295 if ((Attributes & retainCopyNonatomic) !=
296 (PIkind & retainCopyNonatomic)) {
297 Diag(AtLoc, diag::warn_property_attr_mismatch);
298 Diag(PIDecl->getLocation(), diag::note_property_declare);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000299 }
Ted Kremenek9944c762010-03-18 01:22:36 +0000300 DeclContext *DC = cast<DeclContext>(CCPrimary);
301 if (!ObjCPropertyDecl::findPropertyDecl(DC,
302 PIDecl->getDeclName().getAsIdentifierInfo())) {
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000303 // Protocol is not in the primary class. Must build one for it.
304 ObjCDeclSpec ProtocolPropertyODS;
305 // FIXME. Assuming that ObjCDeclSpec::ObjCPropertyAttributeKind
306 // and ObjCPropertyDecl::PropertyAttributeKind have identical
307 // values. Should consolidate both into one enum type.
308 ProtocolPropertyODS.
309 setPropertyAttributes((ObjCDeclSpec::ObjCPropertyAttributeKind)
310 PIkind);
Fariborz Jahaniana28948f2011-08-22 15:54:49 +0000311 // Must re-establish the context from class extension to primary
312 // class context.
Fariborz Jahanian79394182011-08-22 20:15:24 +0000313 ContextRAII SavedContext(*this, CCPrimary);
314
John McCalld226f652010-08-21 09:40:31 +0000315 Decl *ProtocolPtrTy =
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000316 ActOnProperty(S, AtLoc, FD, ProtocolPropertyODS,
317 PIDecl->getGetterName(),
318 PIDecl->getSetterName(),
Fariborz Jahaniana28948f2011-08-22 15:54:49 +0000319 isOverridingProperty,
Ted Kremenek4a2e9ea2010-09-23 21:18:05 +0000320 MethodImplKind,
321 /* lexicalDC = */ CDecl);
John McCalld226f652010-08-21 09:40:31 +0000322 PIDecl = cast<ObjCPropertyDecl>(ProtocolPtrTy);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000323 }
324 PIDecl->makeitReadWriteAttribute();
325 if (Attributes & ObjCDeclSpec::DQ_PR_retain)
326 PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain);
John McCallf85e1932011-06-15 23:02:42 +0000327 if (Attributes & ObjCDeclSpec::DQ_PR_strong)
328 PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000329 if (Attributes & ObjCDeclSpec::DQ_PR_copy)
330 PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy);
331 PIDecl->setSetterName(SetterSel);
332 } else {
Ted Kremenek788f4892010-10-21 18:49:42 +0000333 // Tailor the diagnostics for the common case where a readwrite
334 // property is declared both in the @interface and the continuation.
335 // This is a common error where the user often intended the original
336 // declaration to be readonly.
337 unsigned diag =
338 (Attributes & ObjCDeclSpec::DQ_PR_readwrite) &&
339 (PIkind & ObjCPropertyDecl::OBJC_PR_readwrite)
340 ? diag::err_use_continuation_class_redeclaration_readwrite
341 : diag::err_use_continuation_class;
342 Diag(AtLoc, diag)
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000343 << CCPrimary->getDeclName();
344 Diag(PIDecl->getLocation(), diag::note_property_declare);
345 }
346 *isOverridingProperty = true;
347 // Make sure setter decl is synthesized, and added to primary class's list.
Ted Kremenek8254aa62010-09-21 18:28:43 +0000348 ProcessPropertyDecl(PIDecl, CCPrimary, PDecl, CDecl);
Argyrios Kyrtzidisc80553e2011-11-14 04:52:29 +0000349 if (ASTMutationListener *L = Context.getASTMutationListener())
350 L->AddedObjCPropertyInClassExtension(PDecl, PIDecl, CDecl);
John McCalld226f652010-08-21 09:40:31 +0000351 return 0;
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000352}
353
354ObjCPropertyDecl *Sema::CreatePropertyDecl(Scope *S,
355 ObjCContainerDecl *CDecl,
356 SourceLocation AtLoc,
357 FieldDeclarator &FD,
358 Selector GetterSel,
359 Selector SetterSel,
360 const bool isAssign,
361 const bool isReadWrite,
362 const unsigned Attributes,
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000363 const unsigned AttributesAsWritten,
John McCall83a230c2010-06-04 20:50:08 +0000364 TypeSourceInfo *TInfo,
Ted Kremenek23173d72010-05-18 21:09:07 +0000365 tok::ObjCKeywordKind MethodImplKind,
366 DeclContext *lexicalDC){
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000367 IdentifierInfo *PropertyId = FD.D.getIdentifier();
John McCall83a230c2010-06-04 20:50:08 +0000368 QualType T = TInfo->getType();
Ted Kremenek28685ab2010-03-12 00:46:40 +0000369
370 // Issue a warning if property is 'assign' as default and its object, which is
371 // gc'able conforms to NSCopying protocol
Douglas Gregore289d812011-09-13 17:21:33 +0000372 if (getLangOptions().getGC() != LangOptions::NonGC &&
Ted Kremenek28685ab2010-03-12 00:46:40 +0000373 isAssign && !(Attributes & ObjCDeclSpec::DQ_PR_assign))
John McCallc12c5bb2010-05-15 11:32:37 +0000374 if (const ObjCObjectPointerType *ObjPtrTy =
375 T->getAs<ObjCObjectPointerType>()) {
376 ObjCInterfaceDecl *IDecl = ObjPtrTy->getObjectType()->getInterface();
377 if (IDecl)
378 if (ObjCProtocolDecl* PNSCopying =
379 LookupProtocol(&Context.Idents.get("NSCopying"), AtLoc))
380 if (IDecl->ClassImplementsProtocol(PNSCopying, true))
381 Diag(AtLoc, diag::warn_implements_nscopying) << PropertyId;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000382 }
John McCallc12c5bb2010-05-15 11:32:37 +0000383 if (T->isObjCObjectType())
Ted Kremenek28685ab2010-03-12 00:46:40 +0000384 Diag(FD.D.getIdentifierLoc(), diag::err_statically_allocated_object);
385
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000386 DeclContext *DC = cast<DeclContext>(CDecl);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000387 ObjCPropertyDecl *PDecl = ObjCPropertyDecl::Create(Context, DC,
388 FD.D.getIdentifierLoc(),
John McCall83a230c2010-06-04 20:50:08 +0000389 PropertyId, AtLoc, TInfo);
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000390
Ted Kremenek9f550ff2010-03-15 20:11:46 +0000391 if (ObjCPropertyDecl *prevDecl =
392 ObjCPropertyDecl::findPropertyDecl(DC, PropertyId)) {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000393 Diag(PDecl->getLocation(), diag::err_duplicate_property);
Ted Kremenek894ae6a2010-03-15 18:47:25 +0000394 Diag(prevDecl->getLocation(), diag::note_property_declare);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000395 PDecl->setInvalidDecl();
396 }
Ted Kremenek23173d72010-05-18 21:09:07 +0000397 else {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000398 DC->addDecl(PDecl);
Ted Kremenek23173d72010-05-18 21:09:07 +0000399 if (lexicalDC)
400 PDecl->setLexicalDeclContext(lexicalDC);
401 }
Ted Kremenek28685ab2010-03-12 00:46:40 +0000402
403 if (T->isArrayType() || T->isFunctionType()) {
404 Diag(AtLoc, diag::err_property_type) << T;
405 PDecl->setInvalidDecl();
406 }
407
408 ProcessDeclAttributes(S, PDecl, FD.D);
409
410 // Regardless of setter/getter attribute, we save the default getter/setter
411 // selector names in anticipation of declaration of setter/getter methods.
412 PDecl->setGetterName(GetterSel);
413 PDecl->setSetterName(SetterSel);
Argyrios Kyrtzidis0a68dc72011-07-12 04:30:16 +0000414 PDecl->setPropertyAttributesAsWritten(
Argyrios Kyrtzidisdbbdec92011-11-06 18:58:12 +0000415 makePropertyAttributesAsWritten(AttributesAsWritten));
Argyrios Kyrtzidis0a68dc72011-07-12 04:30:16 +0000416
Ted Kremenek28685ab2010-03-12 00:46:40 +0000417 if (Attributes & ObjCDeclSpec::DQ_PR_readonly)
418 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly);
419
420 if (Attributes & ObjCDeclSpec::DQ_PR_getter)
421 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_getter);
422
423 if (Attributes & ObjCDeclSpec::DQ_PR_setter)
424 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_setter);
425
426 if (isReadWrite)
427 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite);
428
429 if (Attributes & ObjCDeclSpec::DQ_PR_retain)
430 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain);
431
John McCallf85e1932011-06-15 23:02:42 +0000432 if (Attributes & ObjCDeclSpec::DQ_PR_strong)
433 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
434
435 if (Attributes & ObjCDeclSpec::DQ_PR_weak)
436 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_weak);
437
Ted Kremenek28685ab2010-03-12 00:46:40 +0000438 if (Attributes & ObjCDeclSpec::DQ_PR_copy)
439 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy);
440
John McCallf85e1932011-06-15 23:02:42 +0000441 if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
442 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
443
Ted Kremenek28685ab2010-03-12 00:46:40 +0000444 if (isAssign)
445 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign);
446
John McCall265941b2011-09-13 18:31:23 +0000447 // In the semantic attributes, one of nonatomic or atomic is always set.
Ted Kremenek28685ab2010-03-12 00:46:40 +0000448 if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)
449 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_nonatomic);
John McCall265941b2011-09-13 18:31:23 +0000450 else
Fariborz Jahanian45937ae2011-06-11 00:45:12 +0000451 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_atomic);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000452
John McCallf85e1932011-06-15 23:02:42 +0000453 // 'unsafe_unretained' is alias for 'assign'.
454 if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained)
455 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign);
456 if (isAssign)
457 PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_unsafe_unretained);
458
Ted Kremenek28685ab2010-03-12 00:46:40 +0000459 if (MethodImplKind == tok::objc_required)
460 PDecl->setPropertyImplementation(ObjCPropertyDecl::Required);
461 else if (MethodImplKind == tok::objc_optional)
462 PDecl->setPropertyImplementation(ObjCPropertyDecl::Optional);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000463
Ted Kremeneke3d67bc2010-03-12 02:31:10 +0000464 return PDecl;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000465}
466
John McCallf85e1932011-06-15 23:02:42 +0000467static void checkARCPropertyImpl(Sema &S, SourceLocation propertyImplLoc,
468 ObjCPropertyDecl *property,
469 ObjCIvarDecl *ivar) {
470 if (property->isInvalidDecl() || ivar->isInvalidDecl()) return;
471
John McCallf85e1932011-06-15 23:02:42 +0000472 QualType ivarType = ivar->getType();
473 Qualifiers::ObjCLifetime ivarLifetime = ivarType.getObjCLifetime();
John McCallf85e1932011-06-15 23:02:42 +0000474
John McCall265941b2011-09-13 18:31:23 +0000475 // The lifetime implied by the property's attributes.
476 Qualifiers::ObjCLifetime propertyLifetime =
477 getImpliedARCOwnership(property->getPropertyAttributes(),
478 property->getType());
John McCallf85e1932011-06-15 23:02:42 +0000479
John McCall265941b2011-09-13 18:31:23 +0000480 // We're fine if they match.
481 if (propertyLifetime == ivarLifetime) return;
John McCallf85e1932011-06-15 23:02:42 +0000482
John McCall265941b2011-09-13 18:31:23 +0000483 // These aren't valid lifetimes for object ivars; don't diagnose twice.
484 if (ivarLifetime == Qualifiers::OCL_None ||
485 ivarLifetime == Qualifiers::OCL_Autoreleasing)
486 return;
John McCallf85e1932011-06-15 23:02:42 +0000487
John McCall265941b2011-09-13 18:31:23 +0000488 switch (propertyLifetime) {
489 case Qualifiers::OCL_Strong:
490 S.Diag(propertyImplLoc, diag::err_arc_strong_property_ownership)
491 << property->getDeclName()
492 << ivar->getDeclName()
493 << ivarLifetime;
494 break;
John McCallf85e1932011-06-15 23:02:42 +0000495
John McCall265941b2011-09-13 18:31:23 +0000496 case Qualifiers::OCL_Weak:
497 S.Diag(propertyImplLoc, diag::error_weak_property)
498 << property->getDeclName()
499 << ivar->getDeclName();
500 break;
John McCallf85e1932011-06-15 23:02:42 +0000501
John McCall265941b2011-09-13 18:31:23 +0000502 case Qualifiers::OCL_ExplicitNone:
503 S.Diag(propertyImplLoc, diag::err_arc_assign_property_ownership)
504 << property->getDeclName()
505 << ivar->getDeclName()
506 << ((property->getPropertyAttributesAsWritten()
507 & ObjCPropertyDecl::OBJC_PR_assign) != 0);
508 break;
John McCallf85e1932011-06-15 23:02:42 +0000509
John McCall265941b2011-09-13 18:31:23 +0000510 case Qualifiers::OCL_Autoreleasing:
511 llvm_unreachable("properties cannot be autoreleasing");
John McCallf85e1932011-06-15 23:02:42 +0000512
John McCall265941b2011-09-13 18:31:23 +0000513 case Qualifiers::OCL_None:
514 // Any other property should be ignored.
John McCallf85e1932011-06-15 23:02:42 +0000515 return;
516 }
517
518 S.Diag(property->getLocation(), diag::note_property_declare);
519}
520
Ted Kremenek28685ab2010-03-12 00:46:40 +0000521
522/// ActOnPropertyImplDecl - This routine performs semantic checks and
523/// builds the AST node for a property implementation declaration; declared
524/// as @synthesize or @dynamic.
525///
John McCalld226f652010-08-21 09:40:31 +0000526Decl *Sema::ActOnPropertyImplDecl(Scope *S,
527 SourceLocation AtLoc,
528 SourceLocation PropertyLoc,
529 bool Synthesize,
John McCalld226f652010-08-21 09:40:31 +0000530 IdentifierInfo *PropertyId,
Douglas Gregora4ffd852010-11-17 01:03:52 +0000531 IdentifierInfo *PropertyIvar,
532 SourceLocation PropertyIvarLoc) {
Ted Kremeneke9686572010-04-05 23:45:09 +0000533 ObjCContainerDecl *ClassImpDecl =
Fariborz Jahanian84e0ccf2011-09-19 16:32:32 +0000534 dyn_cast<ObjCContainerDecl>(CurContext);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000535 // Make sure we have a context for the property implementation declaration.
536 if (!ClassImpDecl) {
537 Diag(AtLoc, diag::error_missing_property_context);
John McCalld226f652010-08-21 09:40:31 +0000538 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000539 }
540 ObjCPropertyDecl *property = 0;
541 ObjCInterfaceDecl* IDecl = 0;
542 // Find the class or category class where this property must have
543 // a declaration.
544 ObjCImplementationDecl *IC = 0;
545 ObjCCategoryImplDecl* CatImplClass = 0;
546 if ((IC = dyn_cast<ObjCImplementationDecl>(ClassImpDecl))) {
547 IDecl = IC->getClassInterface();
548 // We always synthesize an interface for an implementation
549 // without an interface decl. So, IDecl is always non-zero.
550 assert(IDecl &&
551 "ActOnPropertyImplDecl - @implementation without @interface");
552
553 // Look for this property declaration in the @implementation's @interface
554 property = IDecl->FindPropertyDeclaration(PropertyId);
555 if (!property) {
556 Diag(PropertyLoc, diag::error_bad_property_decl) << IDecl->getDeclName();
John McCalld226f652010-08-21 09:40:31 +0000557 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000558 }
Fariborz Jahaniandd4430e2010-12-17 22:28:16 +0000559 unsigned PIkind = property->getPropertyAttributesAsWritten();
Fariborz Jahanian45937ae2011-06-11 00:45:12 +0000560 if ((PIkind & (ObjCPropertyDecl::OBJC_PR_atomic |
561 ObjCPropertyDecl::OBJC_PR_nonatomic) ) == 0) {
Fariborz Jahaniandd4430e2010-12-17 22:28:16 +0000562 if (AtLoc.isValid())
563 Diag(AtLoc, diag::warn_implicit_atomic_property);
564 else
565 Diag(IC->getLocation(), diag::warn_auto_implicit_atomic_property);
566 Diag(property->getLocation(), diag::note_property_declare);
567 }
568
Ted Kremenek28685ab2010-03-12 00:46:40 +0000569 if (const ObjCCategoryDecl *CD =
570 dyn_cast<ObjCCategoryDecl>(property->getDeclContext())) {
571 if (!CD->IsClassExtension()) {
572 Diag(PropertyLoc, diag::error_category_property) << CD->getDeclName();
573 Diag(property->getLocation(), diag::note_property_declare);
John McCalld226f652010-08-21 09:40:31 +0000574 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000575 }
576 }
577 } else if ((CatImplClass = dyn_cast<ObjCCategoryImplDecl>(ClassImpDecl))) {
578 if (Synthesize) {
579 Diag(AtLoc, diag::error_synthesize_category_decl);
John McCalld226f652010-08-21 09:40:31 +0000580 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000581 }
582 IDecl = CatImplClass->getClassInterface();
583 if (!IDecl) {
584 Diag(AtLoc, diag::error_missing_property_interface);
John McCalld226f652010-08-21 09:40:31 +0000585 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000586 }
587 ObjCCategoryDecl *Category =
588 IDecl->FindCategoryDeclaration(CatImplClass->getIdentifier());
589
590 // If category for this implementation not found, it is an error which
591 // has already been reported eralier.
592 if (!Category)
John McCalld226f652010-08-21 09:40:31 +0000593 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000594 // Look for this property declaration in @implementation's category
595 property = Category->FindPropertyDeclaration(PropertyId);
596 if (!property) {
597 Diag(PropertyLoc, diag::error_bad_category_property_decl)
598 << Category->getDeclName();
John McCalld226f652010-08-21 09:40:31 +0000599 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000600 }
601 } else {
602 Diag(AtLoc, diag::error_bad_property_context);
John McCalld226f652010-08-21 09:40:31 +0000603 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000604 }
605 ObjCIvarDecl *Ivar = 0;
606 // Check that we have a valid, previously declared ivar for @synthesize
607 if (Synthesize) {
608 // @synthesize
609 if (!PropertyIvar)
610 PropertyIvar = PropertyId;
John McCallf85e1932011-06-15 23:02:42 +0000611 ObjCPropertyDecl::PropertyAttributeKind kind
612 = property->getPropertyAttributes();
John McCall265941b2011-09-13 18:31:23 +0000613 QualType PropType = property->getType();
614
615 QualType PropertyIvarType = PropType.getNonReferenceType();
616
617 // Add GC __weak to the ivar type if the property is weak.
618 if ((kind & ObjCPropertyDecl::OBJC_PR_weak) &&
Douglas Gregore289d812011-09-13 17:21:33 +0000619 getLangOptions().getGC() != LangOptions::NonGC) {
John McCall265941b2011-09-13 18:31:23 +0000620 assert(!getLangOptions().ObjCAutoRefCount);
621 if (PropertyIvarType.isObjCGCStrong()) {
622 Diag(PropertyLoc, diag::err_gc_weak_property_strong_type);
623 Diag(property->getLocation(), diag::note_property_declare);
624 } else {
625 PropertyIvarType =
626 Context.getObjCGCQualType(PropertyIvarType, Qualifiers::Weak);
Fariborz Jahanianedc08822011-09-07 16:24:21 +0000627 }
Fariborz Jahanianedc08822011-09-07 16:24:21 +0000628 }
John McCall265941b2011-09-13 18:31:23 +0000629
Ted Kremenek28685ab2010-03-12 00:46:40 +0000630 // Check that this is a previously declared 'ivar' in 'IDecl' interface
631 ObjCInterfaceDecl *ClassDeclared;
632 Ivar = IDecl->lookupInstanceVariable(PropertyIvar, ClassDeclared);
633 if (!Ivar) {
John McCall265941b2011-09-13 18:31:23 +0000634 // In ARC, give the ivar a lifetime qualifier based on the
John McCallf85e1932011-06-15 23:02:42 +0000635 // property attributes.
636 if (getLangOptions().ObjCAutoRefCount &&
John McCall265941b2011-09-13 18:31:23 +0000637 !PropertyIvarType.getObjCLifetime() &&
638 PropertyIvarType->isObjCRetainableType()) {
John McCallf85e1932011-06-15 23:02:42 +0000639
John McCall265941b2011-09-13 18:31:23 +0000640 // It's an error if we have to do this and the user didn't
641 // explicitly write an ownership attribute on the property.
Argyrios Kyrtzidis473506b2011-07-26 21:48:26 +0000642 if (!property->hasWrittenStorageAttribute() &&
John McCall265941b2011-09-13 18:31:23 +0000643 !(kind & ObjCPropertyDecl::OBJC_PR_strong)) {
Argyrios Kyrtzidis473506b2011-07-26 21:48:26 +0000644 Diag(PropertyLoc,
645 diag::err_arc_objc_property_default_assign_on_object);
646 Diag(property->getLocation(), diag::note_property_declare);
John McCall265941b2011-09-13 18:31:23 +0000647 } else {
648 Qualifiers::ObjCLifetime lifetime =
649 getImpliedARCOwnership(kind, PropertyIvarType);
650 assert(lifetime && "no lifetime for property?");
Fariborz Jahanian6dce88d2011-12-09 19:55:11 +0000651 if (lifetime == Qualifiers::OCL_Weak) {
652 bool err = false;
653 if (const ObjCObjectPointerType *ObjT =
654 PropertyIvarType->getAs<ObjCObjectPointerType>())
655 if (ObjT->getInterfaceDecl()->isArcWeakrefUnavailable()) {
656 Diag(PropertyLoc, diag::err_arc_weak_unavailable_property);
657 Diag(property->getLocation(), diag::note_property_declare);
658 err = true;
659 }
660 if (!err && !getLangOptions().ObjCRuntimeHasWeak) {
661 Diag(PropertyLoc, diag::err_arc_weak_no_runtime);
662 Diag(property->getLocation(), diag::note_property_declare);
663 }
John McCallf85e1932011-06-15 23:02:42 +0000664 }
Fariborz Jahanian6dce88d2011-12-09 19:55:11 +0000665
John McCallf85e1932011-06-15 23:02:42 +0000666 Qualifiers qs;
John McCall265941b2011-09-13 18:31:23 +0000667 qs.addObjCLifetime(lifetime);
John McCallf85e1932011-06-15 23:02:42 +0000668 PropertyIvarType = Context.getQualifiedType(PropertyIvarType, qs);
669 }
John McCallf85e1932011-06-15 23:02:42 +0000670 }
671
672 if (kind & ObjCPropertyDecl::OBJC_PR_weak &&
673 !getLangOptions().ObjCAutoRefCount &&
Douglas Gregore289d812011-09-13 17:21:33 +0000674 getLangOptions().getGC() == LangOptions::NonGC) {
John McCallf85e1932011-06-15 23:02:42 +0000675 Diag(PropertyLoc, diag::error_synthesize_weak_non_arc_or_gc);
676 Diag(property->getLocation(), diag::note_property_declare);
677 }
678
Abramo Bagnaraff676cb2011-03-08 08:55:46 +0000679 Ivar = ObjCIvarDecl::Create(Context, ClassImpDecl,
680 PropertyLoc, PropertyLoc, PropertyIvar,
Fariborz Jahanian14086762011-03-28 23:47:18 +0000681 PropertyIvarType, /*Dinfo=*/0,
Fariborz Jahanian75049662010-12-15 23:29:04 +0000682 ObjCIvarDecl::Private,
Fariborz Jahanianad51e742010-07-17 00:59:30 +0000683 (Expr *)0, true);
Daniel Dunbar29fa69a2010-04-02 19:44:54 +0000684 ClassImpDecl->addDecl(Ivar);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000685 IDecl->makeDeclVisibleInContext(Ivar, false);
686 property->setPropertyIvarDecl(Ivar);
687
688 if (!getLangOptions().ObjCNonFragileABI)
689 Diag(PropertyLoc, diag::error_missing_property_ivar_decl) << PropertyId;
690 // Note! I deliberately want it to fall thru so, we have a
691 // a property implementation and to avoid future warnings.
692 } else if (getLangOptions().ObjCNonFragileABI &&
Douglas Gregor60ef3082011-12-15 00:29:59 +0000693 !declaresSameEntity(ClassDeclared, IDecl)) {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000694 Diag(PropertyLoc, diag::error_ivar_in_superclass_use)
695 << property->getDeclName() << Ivar->getDeclName()
696 << ClassDeclared->getDeclName();
697 Diag(Ivar->getLocation(), diag::note_previous_access_declaration)
Daniel Dunbar4087f272010-08-17 22:39:59 +0000698 << Ivar << Ivar->getName();
Ted Kremenek28685ab2010-03-12 00:46:40 +0000699 // Note! I deliberately want it to fall thru so more errors are caught.
700 }
701 QualType IvarType = Context.getCanonicalType(Ivar->getType());
702
703 // Check that type of property and its ivar are type compatible.
John McCall265941b2011-09-13 18:31:23 +0000704 if (Context.getCanonicalType(PropertyIvarType) != IvarType) {
Fariborz Jahanian62ac5d02010-05-18 23:04:17 +0000705 bool compat = false;
Fariborz Jahanian14086762011-03-28 23:47:18 +0000706 if (isa<ObjCObjectPointerType>(PropertyIvarType)
John McCallf85e1932011-06-15 23:02:42 +0000707 && isa<ObjCObjectPointerType>(IvarType))
Fariborz Jahanian62ac5d02010-05-18 23:04:17 +0000708 compat =
709 Context.canAssignObjCInterfaces(
Fariborz Jahanian14086762011-03-28 23:47:18 +0000710 PropertyIvarType->getAs<ObjCObjectPointerType>(),
Fariborz Jahanian62ac5d02010-05-18 23:04:17 +0000711 IvarType->getAs<ObjCObjectPointerType>());
Douglas Gregorb608b982011-01-28 02:26:04 +0000712 else {
713 SourceLocation Loc = PropertyIvarLoc;
714 if (Loc.isInvalid())
715 Loc = PropertyLoc;
Fariborz Jahanian14086762011-03-28 23:47:18 +0000716 compat = (CheckAssignmentConstraints(Loc, PropertyIvarType, IvarType)
John McCalldaa8e4e2010-11-15 09:13:47 +0000717 == Compatible);
Douglas Gregorb608b982011-01-28 02:26:04 +0000718 }
Fariborz Jahanian62ac5d02010-05-18 23:04:17 +0000719 if (!compat) {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000720 Diag(PropertyLoc, diag::error_property_ivar_type)
Ted Kremenekf921a482010-03-23 19:02:22 +0000721 << property->getDeclName() << PropType
722 << Ivar->getDeclName() << IvarType;
723 Diag(Ivar->getLocation(), diag::note_ivar_decl);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000724 // Note! I deliberately want it to fall thru so, we have a
725 // a property implementation and to avoid future warnings.
726 }
727
728 // FIXME! Rules for properties are somewhat different that those
729 // for assignments. Use a new routine to consolidate all cases;
730 // specifically for property redeclarations as well as for ivars.
Fariborz Jahanian14086762011-03-28 23:47:18 +0000731 QualType lhsType =Context.getCanonicalType(PropertyIvarType).getUnqualifiedType();
Ted Kremenek28685ab2010-03-12 00:46:40 +0000732 QualType rhsType =Context.getCanonicalType(IvarType).getUnqualifiedType();
733 if (lhsType != rhsType &&
734 lhsType->isArithmeticType()) {
735 Diag(PropertyLoc, diag::error_property_ivar_type)
Ted Kremenekf921a482010-03-23 19:02:22 +0000736 << property->getDeclName() << PropType
737 << Ivar->getDeclName() << IvarType;
738 Diag(Ivar->getLocation(), diag::note_ivar_decl);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000739 // Fall thru - see previous comment
740 }
741 // __weak is explicit. So it works on Canonical type.
John McCallf85e1932011-06-15 23:02:42 +0000742 if ((PropType.isObjCGCWeak() && !IvarType.isObjCGCWeak() &&
Douglas Gregore289d812011-09-13 17:21:33 +0000743 getLangOptions().getGC() != LangOptions::NonGC)) {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000744 Diag(PropertyLoc, diag::error_weak_property)
745 << property->getDeclName() << Ivar->getDeclName();
Fariborz Jahanianedc08822011-09-07 16:24:21 +0000746 Diag(Ivar->getLocation(), diag::note_ivar_decl);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000747 // Fall thru - see previous comment
748 }
John McCallf85e1932011-06-15 23:02:42 +0000749 // Fall thru - see previous comment
Ted Kremenek28685ab2010-03-12 00:46:40 +0000750 if ((property->getType()->isObjCObjectPointerType() ||
751 PropType.isObjCGCStrong()) && IvarType.isObjCGCWeak() &&
Douglas Gregore289d812011-09-13 17:21:33 +0000752 getLangOptions().getGC() != LangOptions::NonGC) {
Ted Kremenek28685ab2010-03-12 00:46:40 +0000753 Diag(PropertyLoc, diag::error_strong_property)
754 << property->getDeclName() << Ivar->getDeclName();
755 // Fall thru - see previous comment
756 }
757 }
John McCallf85e1932011-06-15 23:02:42 +0000758 if (getLangOptions().ObjCAutoRefCount)
759 checkARCPropertyImpl(*this, PropertyLoc, property, Ivar);
Ted Kremenek28685ab2010-03-12 00:46:40 +0000760 } else if (PropertyIvar)
761 // @dynamic
762 Diag(PropertyLoc, diag::error_dynamic_property_ivar_decl);
John McCallf85e1932011-06-15 23:02:42 +0000763
Ted Kremenek28685ab2010-03-12 00:46:40 +0000764 assert (property && "ActOnPropertyImplDecl - property declaration missing");
765 ObjCPropertyImplDecl *PIDecl =
766 ObjCPropertyImplDecl::Create(Context, CurContext, AtLoc, PropertyLoc,
767 property,
768 (Synthesize ?
769 ObjCPropertyImplDecl::Synthesize
770 : ObjCPropertyImplDecl::Dynamic),
Douglas Gregora4ffd852010-11-17 01:03:52 +0000771 Ivar, PropertyIvarLoc);
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000772 if (ObjCMethodDecl *getterMethod = property->getGetterMethodDecl()) {
773 getterMethod->createImplicitParams(Context, IDecl);
Fariborz Jahanian0313f442010-10-15 22:42:59 +0000774 if (getLangOptions().CPlusPlus && Synthesize &&
775 Ivar->getType()->isRecordType()) {
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000776 // For Objective-C++, need to synthesize the AST for the IVAR object to be
777 // returned by the getter as it must conform to C++'s copy-return rules.
778 // FIXME. Eventually we want to do this for Objective-C as well.
779 ImplicitParamDecl *SelfDecl = getterMethod->getSelfDecl();
780 DeclRefExpr *SelfExpr =
John McCallf89e55a2010-11-18 06:31:45 +0000781 new (Context) DeclRefExpr(SelfDecl, SelfDecl->getType(),
782 VK_RValue, SourceLocation());
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000783 Expr *IvarRefExpr =
784 new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc,
785 SelfExpr, true, true);
John McCall60d7b3a2010-08-24 06:29:42 +0000786 ExprResult Res =
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000787 PerformCopyInitialization(InitializedEntity::InitializeResult(
Douglas Gregor3c9034c2010-05-15 00:13:29 +0000788 SourceLocation(),
789 getterMethod->getResultType(),
790 /*NRVO=*/false),
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000791 SourceLocation(),
792 Owned(IvarRefExpr));
793 if (!Res.isInvalid()) {
794 Expr *ResExpr = Res.takeAs<Expr>();
795 if (ResExpr)
John McCall4765fa02010-12-06 08:20:24 +0000796 ResExpr = MaybeCreateExprWithCleanups(ResExpr);
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000797 PIDecl->setGetterCXXConstructor(ResExpr);
798 }
799 }
Fariborz Jahanian831fb962011-06-25 00:17:46 +0000800 if (property->hasAttr<NSReturnsNotRetainedAttr>() &&
801 !getterMethod->hasAttr<NSReturnsNotRetainedAttr>()) {
802 Diag(getterMethod->getLocation(),
803 diag::warn_property_getter_owning_mismatch);
804 Diag(property->getLocation(), diag::note_property_declare);
805 }
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000806 }
807 if (ObjCMethodDecl *setterMethod = property->getSetterMethodDecl()) {
808 setterMethod->createImplicitParams(Context, IDecl);
Fariborz Jahanian0313f442010-10-15 22:42:59 +0000809 if (getLangOptions().CPlusPlus && Synthesize
810 && Ivar->getType()->isRecordType()) {
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000811 // FIXME. Eventually we want to do this for Objective-C as well.
812 ImplicitParamDecl *SelfDecl = setterMethod->getSelfDecl();
813 DeclRefExpr *SelfExpr =
John McCallf89e55a2010-11-18 06:31:45 +0000814 new (Context) DeclRefExpr(SelfDecl, SelfDecl->getType(),
815 VK_RValue, SourceLocation());
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000816 Expr *lhs =
817 new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc,
818 SelfExpr, true, true);
819 ObjCMethodDecl::param_iterator P = setterMethod->param_begin();
820 ParmVarDecl *Param = (*P);
John McCall3c3b7f92011-10-25 17:37:35 +0000821 QualType T = Param->getType().getNonReferenceType();
Fariborz Jahanian14086762011-03-28 23:47:18 +0000822 Expr *rhs = new (Context) DeclRefExpr(Param, T,
John McCallf89e55a2010-11-18 06:31:45 +0000823 VK_LValue, SourceLocation());
Fariborz Jahanianfa432392010-10-14 21:30:10 +0000824 ExprResult Res = BuildBinOp(S, lhs->getLocEnd(),
John McCall2de56d12010-08-25 11:45:40 +0000825 BO_Assign, lhs, rhs);
Fariborz Jahanian57e264e2011-10-06 18:38:18 +0000826 if (property->getPropertyAttributes() &
827 ObjCPropertyDecl::OBJC_PR_atomic) {
828 Expr *callExpr = Res.takeAs<Expr>();
829 if (const CXXOperatorCallExpr *CXXCE =
Fariborz Jahanian13bf6332011-10-07 21:08:14 +0000830 dyn_cast_or_null<CXXOperatorCallExpr>(callExpr))
831 if (const FunctionDecl *FuncDecl = CXXCE->getDirectCallee())
Fariborz Jahanian57e264e2011-10-06 18:38:18 +0000832 if (!FuncDecl->isTrivial())
Fariborz Jahanian20abee62012-01-10 00:37:01 +0000833 if (property->getType()->isReferenceType()) {
834 Diag(PropertyLoc,
835 diag::err_atomic_property_nontrivial_assign_op)
Fariborz Jahanian57e264e2011-10-06 18:38:18 +0000836 << property->getType();
Fariborz Jahanian20abee62012-01-10 00:37:01 +0000837 Diag(FuncDecl->getLocStart(),
838 diag::note_callee_decl) << FuncDecl;
839 }
Fariborz Jahanian57e264e2011-10-06 18:38:18 +0000840 }
Fariborz Jahanian17cb3262010-05-05 21:52:17 +0000841 PIDecl->setSetterCXXAssignment(Res.takeAs<Expr>());
842 }
843 }
844
Ted Kremenek28685ab2010-03-12 00:46:40 +0000845 if (IC) {
846 if (Synthesize)
847 if (ObjCPropertyImplDecl *PPIDecl =
848 IC->FindPropertyImplIvarDecl(PropertyIvar)) {
849 Diag(PropertyLoc, diag::error_duplicate_ivar_use)
850 << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier()
851 << PropertyIvar;
852 Diag(PPIDecl->getLocation(), diag::note_previous_use);
853 }
854
855 if (ObjCPropertyImplDecl *PPIDecl
856 = IC->FindPropertyImplDecl(PropertyId)) {
857 Diag(PropertyLoc, diag::error_property_implemented) << PropertyId;
858 Diag(PPIDecl->getLocation(), diag::note_previous_declaration);
John McCalld226f652010-08-21 09:40:31 +0000859 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000860 }
861 IC->addPropertyImplementation(PIDecl);
Fariborz Jahaniane776f882011-01-03 18:08:02 +0000862 if (getLangOptions().ObjCDefaultSynthProperties &&
Fariborz Jahanianeb4f2c52012-01-03 19:46:00 +0000863 getLangOptions().ObjCNonFragileABI2 &&
Ted Kremenek71207fc2012-01-05 22:47:47 +0000864 !IDecl->isObjCRequiresPropertyDefs()) {
Fariborz Jahanianad51e742010-07-17 00:59:30 +0000865 // Diagnose if an ivar was lazily synthesdized due to a previous
866 // use and if 1) property is @dynamic or 2) property is synthesized
Fariborz Jahaniancdaa6a82010-08-24 18:48:05 +0000867 // but it requires an ivar of different name.
Fariborz Jahanian411c25c2011-01-20 23:34:25 +0000868 ObjCInterfaceDecl *ClassDeclared=0;
Fariborz Jahanianad51e742010-07-17 00:59:30 +0000869 ObjCIvarDecl *Ivar = 0;
870 if (!Synthesize)
871 Ivar = IDecl->lookupInstanceVariable(PropertyId, ClassDeclared);
872 else {
873 if (PropertyIvar && PropertyIvar != PropertyId)
874 Ivar = IDecl->lookupInstanceVariable(PropertyId, ClassDeclared);
875 }
Fariborz Jahaniancdaa6a82010-08-24 18:48:05 +0000876 // Issue diagnostics only if Ivar belongs to current class.
877 if (Ivar && Ivar->getSynthesize() &&
Douglas Gregor60ef3082011-12-15 00:29:59 +0000878 declaresSameEntity(IC->getClassInterface(), ClassDeclared)) {
Fariborz Jahanianad51e742010-07-17 00:59:30 +0000879 Diag(Ivar->getLocation(), diag::err_undeclared_var_use)
880 << PropertyId;
881 Ivar->setInvalidDecl();
882 }
883 }
Ted Kremenek28685ab2010-03-12 00:46:40 +0000884 } else {
885 if (Synthesize)
886 if (ObjCPropertyImplDecl *PPIDecl =
887 CatImplClass->FindPropertyImplIvarDecl(PropertyIvar)) {
888 Diag(PropertyLoc, diag::error_duplicate_ivar_use)
889 << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier()
890 << PropertyIvar;
891 Diag(PPIDecl->getLocation(), diag::note_previous_use);
892 }
893
894 if (ObjCPropertyImplDecl *PPIDecl =
895 CatImplClass->FindPropertyImplDecl(PropertyId)) {
896 Diag(PropertyLoc, diag::error_property_implemented) << PropertyId;
897 Diag(PPIDecl->getLocation(), diag::note_previous_declaration);
John McCalld226f652010-08-21 09:40:31 +0000898 return 0;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000899 }
900 CatImplClass->addPropertyImplementation(PIDecl);
901 }
902
John McCalld226f652010-08-21 09:40:31 +0000903 return PIDecl;
Ted Kremenek28685ab2010-03-12 00:46:40 +0000904}
905
906//===----------------------------------------------------------------------===//
907// Helper methods.
908//===----------------------------------------------------------------------===//
909
Ted Kremenek9d64c152010-03-12 00:38:38 +0000910/// DiagnosePropertyMismatch - Compares two properties for their
911/// attributes and types and warns on a variety of inconsistencies.
912///
913void
914Sema::DiagnosePropertyMismatch(ObjCPropertyDecl *Property,
915 ObjCPropertyDecl *SuperProperty,
916 const IdentifierInfo *inheritedName) {
917 ObjCPropertyDecl::PropertyAttributeKind CAttr =
918 Property->getPropertyAttributes();
919 ObjCPropertyDecl::PropertyAttributeKind SAttr =
920 SuperProperty->getPropertyAttributes();
921 if ((CAttr & ObjCPropertyDecl::OBJC_PR_readonly)
922 && (SAttr & ObjCPropertyDecl::OBJC_PR_readwrite))
923 Diag(Property->getLocation(), diag::warn_readonly_property)
924 << Property->getDeclName() << inheritedName;
925 if ((CAttr & ObjCPropertyDecl::OBJC_PR_copy)
926 != (SAttr & ObjCPropertyDecl::OBJC_PR_copy))
927 Diag(Property->getLocation(), diag::warn_property_attribute)
928 << Property->getDeclName() << "copy" << inheritedName;
Fariborz Jahanian1b46d8d2011-10-08 17:45:33 +0000929 else if (!(SAttr & ObjCPropertyDecl::OBJC_PR_readonly)){
John McCallf85e1932011-06-15 23:02:42 +0000930 unsigned CAttrRetain =
931 (CAttr &
932 (ObjCPropertyDecl::OBJC_PR_retain | ObjCPropertyDecl::OBJC_PR_strong));
933 unsigned SAttrRetain =
934 (SAttr &
935 (ObjCPropertyDecl::OBJC_PR_retain | ObjCPropertyDecl::OBJC_PR_strong));
936 bool CStrong = (CAttrRetain != 0);
937 bool SStrong = (SAttrRetain != 0);
938 if (CStrong != SStrong)
939 Diag(Property->getLocation(), diag::warn_property_attribute)
940 << Property->getDeclName() << "retain (or strong)" << inheritedName;
941 }
Ted Kremenek9d64c152010-03-12 00:38:38 +0000942
943 if ((CAttr & ObjCPropertyDecl::OBJC_PR_nonatomic)
944 != (SAttr & ObjCPropertyDecl::OBJC_PR_nonatomic))
945 Diag(Property->getLocation(), diag::warn_property_attribute)
946 << Property->getDeclName() << "atomic" << inheritedName;
947 if (Property->getSetterName() != SuperProperty->getSetterName())
948 Diag(Property->getLocation(), diag::warn_property_attribute)
949 << Property->getDeclName() << "setter" << inheritedName;
950 if (Property->getGetterName() != SuperProperty->getGetterName())
951 Diag(Property->getLocation(), diag::warn_property_attribute)
952 << Property->getDeclName() << "getter" << inheritedName;
953
954 QualType LHSType =
955 Context.getCanonicalType(SuperProperty->getType());
956 QualType RHSType =
957 Context.getCanonicalType(Property->getType());
958
Fariborz Jahanianc286f382011-07-12 22:05:16 +0000959 if (!Context.propertyTypesAreCompatible(LHSType, RHSType)) {
Fariborz Jahanian8beb6a22011-07-13 17:55:01 +0000960 // Do cases not handled in above.
961 // FIXME. For future support of covariant property types, revisit this.
962 bool IncompatibleObjC = false;
963 QualType ConvertedType;
964 if (!isObjCPointerConversion(RHSType, LHSType,
965 ConvertedType, IncompatibleObjC) ||
Fariborz Jahanian13546a82011-10-12 00:00:57 +0000966 IncompatibleObjC) {
Fariborz Jahanian8beb6a22011-07-13 17:55:01 +0000967 Diag(Property->getLocation(), diag::warn_property_types_are_incompatible)
968 << Property->getType() << SuperProperty->getType() << inheritedName;
Fariborz Jahanian13546a82011-10-12 00:00:57 +0000969 Diag(SuperProperty->getLocation(), diag::note_property_declare);
970 }
Ted Kremenek9d64c152010-03-12 00:38:38 +0000971 }
972}
973
974bool Sema::DiagnosePropertyAccessorMismatch(ObjCPropertyDecl *property,
975 ObjCMethodDecl *GetterMethod,
976 SourceLocation Loc) {
977 if (GetterMethod &&
John McCall3c3b7f92011-10-25 17:37:35 +0000978 !Context.hasSameType(GetterMethod->getResultType().getNonReferenceType(),
979 property->getType().getNonReferenceType())) {
Ted Kremenek9d64c152010-03-12 00:38:38 +0000980 AssignConvertType result = Incompatible;
John McCall1c23e912010-11-16 02:32:08 +0000981 if (property->getType()->isObjCObjectPointerType())
Douglas Gregorb608b982011-01-28 02:26:04 +0000982 result = CheckAssignmentConstraints(Loc, GetterMethod->getResultType(),
John McCall1c23e912010-11-16 02:32:08 +0000983 property->getType());
Ted Kremenek9d64c152010-03-12 00:38:38 +0000984 if (result != Compatible) {
985 Diag(Loc, diag::warn_accessor_property_type_mismatch)
986 << property->getDeclName()
987 << GetterMethod->getSelector();
988 Diag(GetterMethod->getLocation(), diag::note_declared_at);
989 return true;
990 }
991 }
992 return false;
993}
994
995/// ComparePropertiesInBaseAndSuper - This routine compares property
996/// declarations in base and its super class, if any, and issues
Chris Lattnerfc8f0e12011-04-15 05:22:18 +0000997/// diagnostics in a variety of inconsistent situations.
Ted Kremenek9d64c152010-03-12 00:38:38 +0000998///
999void Sema::ComparePropertiesInBaseAndSuper(ObjCInterfaceDecl *IDecl) {
1000 ObjCInterfaceDecl *SDecl = IDecl->getSuperClass();
1001 if (!SDecl)
1002 return;
1003 // FIXME: O(N^2)
1004 for (ObjCInterfaceDecl::prop_iterator S = SDecl->prop_begin(),
1005 E = SDecl->prop_end(); S != E; ++S) {
1006 ObjCPropertyDecl *SuperPDecl = (*S);
1007 // Does property in super class has declaration in current class?
1008 for (ObjCInterfaceDecl::prop_iterator I = IDecl->prop_begin(),
1009 E = IDecl->prop_end(); I != E; ++I) {
1010 ObjCPropertyDecl *PDecl = (*I);
1011 if (SuperPDecl->getIdentifier() == PDecl->getIdentifier())
1012 DiagnosePropertyMismatch(PDecl, SuperPDecl,
1013 SDecl->getIdentifier());
1014 }
1015 }
1016}
1017
1018/// MatchOneProtocolPropertiesInClass - This routine goes thru the list
1019/// of properties declared in a protocol and compares their attribute against
1020/// the same property declared in the class or category.
1021void
1022Sema::MatchOneProtocolPropertiesInClass(Decl *CDecl,
1023 ObjCProtocolDecl *PDecl) {
1024 ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl);
1025 if (!IDecl) {
1026 // Category
1027 ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl);
1028 assert (CatDecl && "MatchOneProtocolPropertiesInClass");
1029 if (!CatDecl->IsClassExtension())
1030 for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1031 E = PDecl->prop_end(); P != E; ++P) {
1032 ObjCPropertyDecl *Pr = (*P);
1033 ObjCCategoryDecl::prop_iterator CP, CE;
1034 // Is this property already in category's list of properties?
Ted Kremenek2d2f9362010-03-12 00:49:00 +00001035 for (CP = CatDecl->prop_begin(), CE = CatDecl->prop_end(); CP!=CE; ++CP)
Ted Kremenek9d64c152010-03-12 00:38:38 +00001036 if ((*CP)->getIdentifier() == Pr->getIdentifier())
1037 break;
1038 if (CP != CE)
1039 // Property protocol already exist in class. Diagnose any mismatch.
1040 DiagnosePropertyMismatch((*CP), Pr, PDecl->getIdentifier());
1041 }
1042 return;
1043 }
1044 for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1045 E = PDecl->prop_end(); P != E; ++P) {
1046 ObjCPropertyDecl *Pr = (*P);
1047 ObjCInterfaceDecl::prop_iterator CP, CE;
1048 // Is this property already in class's list of properties?
1049 for (CP = IDecl->prop_begin(), CE = IDecl->prop_end(); CP != CE; ++CP)
1050 if ((*CP)->getIdentifier() == Pr->getIdentifier())
1051 break;
1052 if (CP != CE)
1053 // Property protocol already exist in class. Diagnose any mismatch.
1054 DiagnosePropertyMismatch((*CP), Pr, PDecl->getIdentifier());
1055 }
1056}
1057
1058/// CompareProperties - This routine compares properties
1059/// declared in 'ClassOrProtocol' objects (which can be a class or an
1060/// inherited protocol with the list of properties for class/category 'CDecl'
1061///
John McCalld226f652010-08-21 09:40:31 +00001062void Sema::CompareProperties(Decl *CDecl, Decl *ClassOrProtocol) {
1063 Decl *ClassDecl = ClassOrProtocol;
Ted Kremenek9d64c152010-03-12 00:38:38 +00001064 ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl);
1065
1066 if (!IDecl) {
1067 // Category
1068 ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl);
1069 assert (CatDecl && "CompareProperties");
1070 if (ObjCCategoryDecl *MDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl)) {
1071 for (ObjCCategoryDecl::protocol_iterator P = MDecl->protocol_begin(),
1072 E = MDecl->protocol_end(); P != E; ++P)
1073 // Match properties of category with those of protocol (*P)
1074 MatchOneProtocolPropertiesInClass(CatDecl, *P);
1075
1076 // Go thru the list of protocols for this category and recursively match
1077 // their properties with those in the category.
1078 for (ObjCCategoryDecl::protocol_iterator P = CatDecl->protocol_begin(),
1079 E = CatDecl->protocol_end(); P != E; ++P)
John McCalld226f652010-08-21 09:40:31 +00001080 CompareProperties(CatDecl, *P);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001081 } else {
1082 ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl);
1083 for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(),
1084 E = MD->protocol_end(); P != E; ++P)
1085 MatchOneProtocolPropertiesInClass(CatDecl, *P);
1086 }
1087 return;
1088 }
1089
1090 if (ObjCInterfaceDecl *MDecl = dyn_cast<ObjCInterfaceDecl>(ClassDecl)) {
Ted Kremenek53b94412010-09-01 01:21:15 +00001091 for (ObjCInterfaceDecl::all_protocol_iterator
1092 P = MDecl->all_referenced_protocol_begin(),
1093 E = MDecl->all_referenced_protocol_end(); P != E; ++P)
Ted Kremenek9d64c152010-03-12 00:38:38 +00001094 // Match properties of class IDecl with those of protocol (*P).
1095 MatchOneProtocolPropertiesInClass(IDecl, *P);
1096
1097 // Go thru the list of protocols for this class and recursively match
1098 // their properties with those declared in the class.
Ted Kremenek53b94412010-09-01 01:21:15 +00001099 for (ObjCInterfaceDecl::all_protocol_iterator
1100 P = IDecl->all_referenced_protocol_begin(),
1101 E = IDecl->all_referenced_protocol_end(); P != E; ++P)
John McCalld226f652010-08-21 09:40:31 +00001102 CompareProperties(IDecl, *P);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001103 } else {
1104 ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl);
1105 for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(),
1106 E = MD->protocol_end(); P != E; ++P)
1107 MatchOneProtocolPropertiesInClass(IDecl, *P);
1108 }
1109}
1110
1111/// isPropertyReadonly - Return true if property is readonly, by searching
1112/// for the property in the class and in its categories and implementations
1113///
1114bool Sema::isPropertyReadonly(ObjCPropertyDecl *PDecl,
1115 ObjCInterfaceDecl *IDecl) {
1116 // by far the most common case.
1117 if (!PDecl->isReadOnly())
1118 return false;
1119 // Even if property is ready only, if interface has a user defined setter,
1120 // it is not considered read only.
1121 if (IDecl->getInstanceMethod(PDecl->getSetterName()))
1122 return false;
1123
1124 // Main class has the property as 'readonly'. Must search
1125 // through the category list to see if the property's
1126 // attribute has been over-ridden to 'readwrite'.
1127 for (ObjCCategoryDecl *Category = IDecl->getCategoryList();
1128 Category; Category = Category->getNextClassCategory()) {
1129 // Even if property is ready only, if a category has a user defined setter,
1130 // it is not considered read only.
1131 if (Category->getInstanceMethod(PDecl->getSetterName()))
1132 return false;
1133 ObjCPropertyDecl *P =
1134 Category->FindPropertyDeclaration(PDecl->getIdentifier());
1135 if (P && !P->isReadOnly())
1136 return false;
1137 }
1138
1139 // Also, check for definition of a setter method in the implementation if
1140 // all else failed.
1141 if (ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(CurContext)) {
1142 if (ObjCImplementationDecl *IMD =
1143 dyn_cast<ObjCImplementationDecl>(OMD->getDeclContext())) {
1144 if (IMD->getInstanceMethod(PDecl->getSetterName()))
1145 return false;
1146 } else if (ObjCCategoryImplDecl *CIMD =
1147 dyn_cast<ObjCCategoryImplDecl>(OMD->getDeclContext())) {
1148 if (CIMD->getInstanceMethod(PDecl->getSetterName()))
1149 return false;
1150 }
1151 }
1152 // Lastly, look through the implementation (if one is in scope).
1153 if (ObjCImplementationDecl *ImpDecl = IDecl->getImplementation())
1154 if (ImpDecl->getInstanceMethod(PDecl->getSetterName()))
1155 return false;
1156 // If all fails, look at the super class.
1157 if (ObjCInterfaceDecl *SIDecl = IDecl->getSuperClass())
1158 return isPropertyReadonly(PDecl, SIDecl);
1159 return true;
1160}
1161
1162/// CollectImmediateProperties - This routine collects all properties in
1163/// the class and its conforming protocols; but not those it its super class.
1164void Sema::CollectImmediateProperties(ObjCContainerDecl *CDecl,
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001165 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap,
1166 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& SuperPropMap) {
Ted Kremenek9d64c152010-03-12 00:38:38 +00001167 if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) {
1168 for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
1169 E = IDecl->prop_end(); P != E; ++P) {
1170 ObjCPropertyDecl *Prop = (*P);
1171 PropMap[Prop->getIdentifier()] = Prop;
1172 }
1173 // scan through class's protocols.
Ted Kremenek53b94412010-09-01 01:21:15 +00001174 for (ObjCInterfaceDecl::all_protocol_iterator
1175 PI = IDecl->all_referenced_protocol_begin(),
1176 E = IDecl->all_referenced_protocol_end(); PI != E; ++PI)
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001177 CollectImmediateProperties((*PI), PropMap, SuperPropMap);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001178 }
1179 if (ObjCCategoryDecl *CATDecl = dyn_cast<ObjCCategoryDecl>(CDecl)) {
1180 if (!CATDecl->IsClassExtension())
1181 for (ObjCContainerDecl::prop_iterator P = CATDecl->prop_begin(),
1182 E = CATDecl->prop_end(); P != E; ++P) {
1183 ObjCPropertyDecl *Prop = (*P);
1184 PropMap[Prop->getIdentifier()] = Prop;
1185 }
1186 // scan through class's protocols.
Ted Kremenek53b94412010-09-01 01:21:15 +00001187 for (ObjCCategoryDecl::protocol_iterator PI = CATDecl->protocol_begin(),
Ted Kremenek9d64c152010-03-12 00:38:38 +00001188 E = CATDecl->protocol_end(); PI != E; ++PI)
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001189 CollectImmediateProperties((*PI), PropMap, SuperPropMap);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001190 }
1191 else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) {
1192 for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1193 E = PDecl->prop_end(); P != E; ++P) {
1194 ObjCPropertyDecl *Prop = (*P);
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001195 ObjCPropertyDecl *PropertyFromSuper = SuperPropMap[Prop->getIdentifier()];
1196 // Exclude property for protocols which conform to class's super-class,
1197 // as super-class has to implement the property.
Fariborz Jahaniana929ec72011-09-27 00:23:52 +00001198 if (!PropertyFromSuper ||
1199 PropertyFromSuper->getIdentifier() != Prop->getIdentifier()) {
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001200 ObjCPropertyDecl *&PropEntry = PropMap[Prop->getIdentifier()];
1201 if (!PropEntry)
1202 PropEntry = Prop;
1203 }
Ted Kremenek9d64c152010-03-12 00:38:38 +00001204 }
1205 // scan through protocol's protocols.
1206 for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
1207 E = PDecl->protocol_end(); PI != E; ++PI)
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001208 CollectImmediateProperties((*PI), PropMap, SuperPropMap);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001209 }
1210}
1211
Fariborz Jahanian509d4772010-05-14 18:35:57 +00001212/// CollectClassPropertyImplementations - This routine collects list of
1213/// properties to be implemented in the class. This includes, class's
1214/// and its conforming protocols' properties.
1215static void CollectClassPropertyImplementations(ObjCContainerDecl *CDecl,
1216 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) {
1217 if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) {
1218 for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
1219 E = IDecl->prop_end(); P != E; ++P) {
1220 ObjCPropertyDecl *Prop = (*P);
1221 PropMap[Prop->getIdentifier()] = Prop;
1222 }
Ted Kremenek53b94412010-09-01 01:21:15 +00001223 for (ObjCInterfaceDecl::all_protocol_iterator
1224 PI = IDecl->all_referenced_protocol_begin(),
1225 E = IDecl->all_referenced_protocol_end(); PI != E; ++PI)
Fariborz Jahanian509d4772010-05-14 18:35:57 +00001226 CollectClassPropertyImplementations((*PI), PropMap);
1227 }
1228 else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) {
1229 for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1230 E = PDecl->prop_end(); P != E; ++P) {
1231 ObjCPropertyDecl *Prop = (*P);
1232 PropMap[Prop->getIdentifier()] = Prop;
1233 }
1234 // scan through protocol's protocols.
1235 for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
1236 E = PDecl->protocol_end(); PI != E; ++PI)
1237 CollectClassPropertyImplementations((*PI), PropMap);
1238 }
1239}
1240
1241/// CollectSuperClassPropertyImplementations - This routine collects list of
1242/// properties to be implemented in super class(s) and also coming from their
1243/// conforming protocols.
1244static void CollectSuperClassPropertyImplementations(ObjCInterfaceDecl *CDecl,
1245 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) {
1246 if (ObjCInterfaceDecl *SDecl = CDecl->getSuperClass()) {
1247 while (SDecl) {
1248 CollectClassPropertyImplementations(SDecl, PropMap);
1249 SDecl = SDecl->getSuperClass();
1250 }
1251 }
1252}
1253
Ted Kremenek9d64c152010-03-12 00:38:38 +00001254/// LookupPropertyDecl - Looks up a property in the current class and all
1255/// its protocols.
1256ObjCPropertyDecl *Sema::LookupPropertyDecl(const ObjCContainerDecl *CDecl,
1257 IdentifierInfo *II) {
1258 if (const ObjCInterfaceDecl *IDecl =
1259 dyn_cast<ObjCInterfaceDecl>(CDecl)) {
1260 for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(),
1261 E = IDecl->prop_end(); P != E; ++P) {
1262 ObjCPropertyDecl *Prop = (*P);
1263 if (Prop->getIdentifier() == II)
1264 return Prop;
1265 }
1266 // scan through class's protocols.
Ted Kremenek53b94412010-09-01 01:21:15 +00001267 for (ObjCInterfaceDecl::all_protocol_iterator
1268 PI = IDecl->all_referenced_protocol_begin(),
1269 E = IDecl->all_referenced_protocol_end(); PI != E; ++PI) {
Ted Kremenek9d64c152010-03-12 00:38:38 +00001270 ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II);
1271 if (Prop)
1272 return Prop;
1273 }
1274 }
1275 else if (const ObjCProtocolDecl *PDecl =
1276 dyn_cast<ObjCProtocolDecl>(CDecl)) {
1277 for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(),
1278 E = PDecl->prop_end(); P != E; ++P) {
1279 ObjCPropertyDecl *Prop = (*P);
1280 if (Prop->getIdentifier() == II)
1281 return Prop;
1282 }
1283 // scan through protocol's protocols.
1284 for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
1285 E = PDecl->protocol_end(); PI != E; ++PI) {
1286 ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II);
1287 if (Prop)
1288 return Prop;
1289 }
1290 }
1291 return 0;
1292}
1293
Ted Kremenekd2ee8092011-09-27 23:39:40 +00001294static IdentifierInfo * getDefaultSynthIvarName(ObjCPropertyDecl *Prop,
1295 ASTContext &Ctx) {
1296 llvm::SmallString<128> ivarName;
1297 {
1298 llvm::raw_svector_ostream os(ivarName);
1299 os << '_' << Prop->getIdentifier()->getName();
1300 }
1301 return &Ctx.Idents.get(ivarName.str());
1302}
1303
Fariborz Jahanian509d4772010-05-14 18:35:57 +00001304/// DefaultSynthesizeProperties - This routine default synthesizes all
1305/// properties which must be synthesized in class's @implementation.
Ted Kremenekd2ee8092011-09-27 23:39:40 +00001306void Sema::DefaultSynthesizeProperties(Scope *S, ObjCImplDecl* IMPDecl,
1307 ObjCInterfaceDecl *IDecl) {
Fariborz Jahanian509d4772010-05-14 18:35:57 +00001308
1309 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap;
1310 CollectClassPropertyImplementations(IDecl, PropMap);
1311 if (PropMap.empty())
1312 return;
1313 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap;
1314 CollectSuperClassPropertyImplementations(IDecl, SuperPropMap);
1315
1316 for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator
1317 P = PropMap.begin(), E = PropMap.end(); P != E; ++P) {
1318 ObjCPropertyDecl *Prop = P->second;
1319 // If property to be implemented in the super class, ignore.
1320 if (SuperPropMap[Prop->getIdentifier()])
1321 continue;
1322 // Is there a matching propery synthesize/dynamic?
1323 if (Prop->isInvalidDecl() ||
1324 Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional ||
1325 IMPDecl->FindPropertyImplIvarDecl(Prop->getIdentifier()))
1326 continue;
Fariborz Jahaniand3635b92010-07-14 18:11:52 +00001327 // Property may have been synthesized by user.
1328 if (IMPDecl->FindPropertyImplDecl(Prop->getIdentifier()))
1329 continue;
Fariborz Jahanian95f1b862010-08-25 00:31:58 +00001330 if (IMPDecl->getInstanceMethod(Prop->getGetterName())) {
1331 if (Prop->getPropertyAttributes() & ObjCPropertyDecl::OBJC_PR_readonly)
1332 continue;
1333 if (IMPDecl->getInstanceMethod(Prop->getSetterName()))
1334 continue;
1335 }
Fariborz Jahanianf8aba8c2011-12-15 01:03:18 +00001336 if (isa<ObjCProtocolDecl>(Prop->getDeclContext())) {
1337 // We won't auto-synthesize properties declared in protocols.
1338 Diag(IMPDecl->getLocation(),
1339 diag::warn_auto_synthesizing_protocol_property);
1340 Diag(Prop->getLocation(), diag::note_property_declare);
1341 continue;
1342 }
Ted Kremenek2a6af6b2010-09-24 01:23:01 +00001343
1344 // We use invalid SourceLocations for the synthesized ivars since they
1345 // aren't really synthesized at a particular location; they just exist.
1346 // Saying that they are located at the @implementation isn't really going
1347 // to help users.
1348 ActOnPropertyImplDecl(S, SourceLocation(), SourceLocation(),
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00001349 true,
Ted Kremenekd2ee8092011-09-27 23:39:40 +00001350 /* property = */ Prop->getIdentifier(),
1351 /* ivar = */ getDefaultSynthIvarName(Prop, Context),
Douglas Gregora4ffd852010-11-17 01:03:52 +00001352 SourceLocation());
Ted Kremenek2a6af6b2010-09-24 01:23:01 +00001353 }
Fariborz Jahanian509d4772010-05-14 18:35:57 +00001354}
Ted Kremenek9d64c152010-03-12 00:38:38 +00001355
Fariborz Jahanian8697d302011-08-31 22:24:06 +00001356void Sema::DefaultSynthesizeProperties(Scope *S, Decl *D) {
1357 if (!LangOpts.ObjCDefaultSynthProperties || !LangOpts.ObjCNonFragileABI2)
1358 return;
1359 ObjCImplementationDecl *IC=dyn_cast_or_null<ObjCImplementationDecl>(D);
1360 if (!IC)
1361 return;
1362 if (ObjCInterfaceDecl* IDecl = IC->getClassInterface())
Ted Kremenek71207fc2012-01-05 22:47:47 +00001363 if (!IDecl->isObjCRequiresPropertyDefs())
Fariborz Jahanianeb4f2c52012-01-03 19:46:00 +00001364 DefaultSynthesizeProperties(S, IC, IDecl);
Fariborz Jahanian8697d302011-08-31 22:24:06 +00001365}
1366
Fariborz Jahanian17cb3262010-05-05 21:52:17 +00001367void Sema::DiagnoseUnimplementedProperties(Scope *S, ObjCImplDecl* IMPDecl,
Ted Kremenek9d64c152010-03-12 00:38:38 +00001368 ObjCContainerDecl *CDecl,
1369 const llvm::DenseSet<Selector>& InsMap) {
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001370 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap;
1371 if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl))
1372 CollectSuperClassPropertyImplementations(IDecl, SuperPropMap);
1373
Ted Kremenek9d64c152010-03-12 00:38:38 +00001374 llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap;
Fariborz Jahaniancfa6a272010-06-29 18:12:32 +00001375 CollectImmediateProperties(CDecl, PropMap, SuperPropMap);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001376 if (PropMap.empty())
1377 return;
1378
1379 llvm::DenseSet<ObjCPropertyDecl *> PropImplMap;
1380 for (ObjCImplDecl::propimpl_iterator
1381 I = IMPDecl->propimpl_begin(),
1382 EI = IMPDecl->propimpl_end(); I != EI; ++I)
1383 PropImplMap.insert((*I)->getPropertyDecl());
1384
1385 for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator
1386 P = PropMap.begin(), E = PropMap.end(); P != E; ++P) {
1387 ObjCPropertyDecl *Prop = P->second;
1388 // Is there a matching propery synthesize/dynamic?
1389 if (Prop->isInvalidDecl() ||
1390 Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional ||
Fariborz Jahanian327126e2011-06-24 20:31:37 +00001391 PropImplMap.count(Prop) || Prop->hasAttr<UnavailableAttr>())
Ted Kremenek9d64c152010-03-12 00:38:38 +00001392 continue;
Ted Kremenek9d64c152010-03-12 00:38:38 +00001393 if (!InsMap.count(Prop->getGetterName())) {
Fariborz Jahanianb8607392011-08-27 21:55:47 +00001394 Diag(IMPDecl->getLocation(),
Ted Kremenek9d64c152010-03-12 00:38:38 +00001395 isa<ObjCCategoryDecl>(CDecl) ?
1396 diag::warn_setter_getter_impl_required_in_category :
1397 diag::warn_setter_getter_impl_required)
1398 << Prop->getDeclName() << Prop->getGetterName();
Fariborz Jahanianb8607392011-08-27 21:55:47 +00001399 Diag(Prop->getLocation(),
1400 diag::note_property_declare);
Fariborz Jahanianda611a72012-01-04 23:16:13 +00001401 if (LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCNonFragileABI2)
1402 if (ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(CDecl))
Ted Kremenek71207fc2012-01-05 22:47:47 +00001403 if (const ObjCInterfaceDecl *RID = ID->isObjCRequiresPropertyDefs())
Fariborz Jahanianda611a72012-01-04 23:16:13 +00001404 Diag(RID->getLocation(), diag::note_suppressed_class_declare);
1405
Ted Kremenek9d64c152010-03-12 00:38:38 +00001406 }
1407
1408 if (!Prop->isReadOnly() && !InsMap.count(Prop->getSetterName())) {
Fariborz Jahanianb8607392011-08-27 21:55:47 +00001409 Diag(IMPDecl->getLocation(),
Ted Kremenek9d64c152010-03-12 00:38:38 +00001410 isa<ObjCCategoryDecl>(CDecl) ?
1411 diag::warn_setter_getter_impl_required_in_category :
1412 diag::warn_setter_getter_impl_required)
1413 << Prop->getDeclName() << Prop->getSetterName();
Fariborz Jahanianb8607392011-08-27 21:55:47 +00001414 Diag(Prop->getLocation(),
1415 diag::note_property_declare);
Fariborz Jahanianda611a72012-01-04 23:16:13 +00001416 if (LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCNonFragileABI2)
1417 if (ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(CDecl))
Ted Kremenek71207fc2012-01-05 22:47:47 +00001418 if (const ObjCInterfaceDecl *RID = ID->isObjCRequiresPropertyDefs())
Fariborz Jahanianda611a72012-01-04 23:16:13 +00001419 Diag(RID->getLocation(), diag::note_suppressed_class_declare);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001420 }
1421 }
1422}
1423
1424void
1425Sema::AtomicPropertySetterGetterRules (ObjCImplDecl* IMPDecl,
1426 ObjCContainerDecl* IDecl) {
1427 // Rules apply in non-GC mode only
Douglas Gregore289d812011-09-13 17:21:33 +00001428 if (getLangOptions().getGC() != LangOptions::NonGC)
Ted Kremenek9d64c152010-03-12 00:38:38 +00001429 return;
1430 for (ObjCContainerDecl::prop_iterator I = IDecl->prop_begin(),
1431 E = IDecl->prop_end();
1432 I != E; ++I) {
1433 ObjCPropertyDecl *Property = (*I);
Argyrios Kyrtzidis94659e42011-01-31 21:34:11 +00001434 ObjCMethodDecl *GetterMethod = 0;
1435 ObjCMethodDecl *SetterMethod = 0;
1436 bool LookedUpGetterSetter = false;
1437
Ted Kremenek9d64c152010-03-12 00:38:38 +00001438 unsigned Attributes = Property->getPropertyAttributes();
John McCall265941b2011-09-13 18:31:23 +00001439 unsigned AttributesAsWritten = Property->getPropertyAttributesAsWritten();
Argyrios Kyrtzidis94659e42011-01-31 21:34:11 +00001440
John McCall265941b2011-09-13 18:31:23 +00001441 if (!(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_atomic) &&
1442 !(AttributesAsWritten & ObjCPropertyDecl::OBJC_PR_nonatomic)) {
Argyrios Kyrtzidis94659e42011-01-31 21:34:11 +00001443 GetterMethod = IMPDecl->getInstanceMethod(Property->getGetterName());
1444 SetterMethod = IMPDecl->getInstanceMethod(Property->getSetterName());
1445 LookedUpGetterSetter = true;
1446 if (GetterMethod) {
1447 Diag(GetterMethod->getLocation(),
1448 diag::warn_default_atomic_custom_getter_setter)
Argyrios Kyrtzidis293a45e2011-01-31 23:20:03 +00001449 << Property->getIdentifier() << 0;
Argyrios Kyrtzidis94659e42011-01-31 21:34:11 +00001450 Diag(Property->getLocation(), diag::note_property_declare);
1451 }
1452 if (SetterMethod) {
1453 Diag(SetterMethod->getLocation(),
1454 diag::warn_default_atomic_custom_getter_setter)
Argyrios Kyrtzidis293a45e2011-01-31 23:20:03 +00001455 << Property->getIdentifier() << 1;
Argyrios Kyrtzidis94659e42011-01-31 21:34:11 +00001456 Diag(Property->getLocation(), diag::note_property_declare);
1457 }
1458 }
1459
Ted Kremenek9d64c152010-03-12 00:38:38 +00001460 // We only care about readwrite atomic property.
1461 if ((Attributes & ObjCPropertyDecl::OBJC_PR_nonatomic) ||
1462 !(Attributes & ObjCPropertyDecl::OBJC_PR_readwrite))
1463 continue;
1464 if (const ObjCPropertyImplDecl *PIDecl
1465 = IMPDecl->FindPropertyImplDecl(Property->getIdentifier())) {
1466 if (PIDecl->getPropertyImplementation() == ObjCPropertyImplDecl::Dynamic)
1467 continue;
Argyrios Kyrtzidis94659e42011-01-31 21:34:11 +00001468 if (!LookedUpGetterSetter) {
1469 GetterMethod = IMPDecl->getInstanceMethod(Property->getGetterName());
1470 SetterMethod = IMPDecl->getInstanceMethod(Property->getSetterName());
1471 LookedUpGetterSetter = true;
1472 }
Ted Kremenek9d64c152010-03-12 00:38:38 +00001473 if ((GetterMethod && !SetterMethod) || (!GetterMethod && SetterMethod)) {
1474 SourceLocation MethodLoc =
1475 (GetterMethod ? GetterMethod->getLocation()
1476 : SetterMethod->getLocation());
1477 Diag(MethodLoc, diag::warn_atomic_property_rule)
Fariborz Jahanian7d65f692011-10-06 23:47:58 +00001478 << Property->getIdentifier() << (GetterMethod != 0)
1479 << (SetterMethod != 0);
1480 Diag(MethodLoc, diag::note_atomic_property_fixup_suggest);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001481 Diag(Property->getLocation(), diag::note_property_declare);
1482 }
1483 }
1484 }
1485}
1486
John McCallf85e1932011-06-15 23:02:42 +00001487void Sema::DiagnoseOwningPropertyGetterSynthesis(const ObjCImplementationDecl *D) {
Douglas Gregore289d812011-09-13 17:21:33 +00001488 if (getLangOptions().getGC() == LangOptions::GCOnly)
John McCallf85e1932011-06-15 23:02:42 +00001489 return;
1490
1491 for (ObjCImplementationDecl::propimpl_iterator
1492 i = D->propimpl_begin(), e = D->propimpl_end(); i != e; ++i) {
1493 ObjCPropertyImplDecl *PID = *i;
1494 if (PID->getPropertyImplementation() != ObjCPropertyImplDecl::Synthesize)
1495 continue;
1496
1497 const ObjCPropertyDecl *PD = PID->getPropertyDecl();
Fariborz Jahanian831fb962011-06-25 00:17:46 +00001498 if (PD && !PD->hasAttr<NSReturnsNotRetainedAttr>() &&
1499 !D->getInstanceMethod(PD->getGetterName())) {
John McCallf85e1932011-06-15 23:02:42 +00001500 ObjCMethodDecl *method = PD->getGetterMethodDecl();
1501 if (!method)
1502 continue;
1503 ObjCMethodFamily family = method->getMethodFamily();
1504 if (family == OMF_alloc || family == OMF_copy ||
1505 family == OMF_mutableCopy || family == OMF_new) {
1506 if (getLangOptions().ObjCAutoRefCount)
1507 Diag(PID->getLocation(), diag::err_ownin_getter_rule);
1508 else
Ted Kremenek920c9c12011-10-12 18:03:37 +00001509 Diag(PID->getLocation(), diag::warn_owning_getter_rule);
John McCallf85e1932011-06-15 23:02:42 +00001510 Diag(PD->getLocation(), diag::note_property_declare);
1511 }
1512 }
1513 }
1514}
1515
John McCall5de74d12010-11-10 07:01:40 +00001516/// AddPropertyAttrs - Propagates attributes from a property to the
1517/// implicitly-declared getter or setter for that property.
1518static void AddPropertyAttrs(Sema &S, ObjCMethodDecl *PropertyMethod,
1519 ObjCPropertyDecl *Property) {
1520 // Should we just clone all attributes over?
Douglas Gregor0a0d2b12011-03-23 00:50:03 +00001521 for (Decl::attr_iterator A = Property->attr_begin(),
1522 AEnd = Property->attr_end();
1523 A != AEnd; ++A) {
1524 if (isa<DeprecatedAttr>(*A) ||
1525 isa<UnavailableAttr>(*A) ||
1526 isa<AvailabilityAttr>(*A))
1527 PropertyMethod->addAttr((*A)->clone(S.Context));
1528 }
John McCall5de74d12010-11-10 07:01:40 +00001529}
1530
Ted Kremenek9d64c152010-03-12 00:38:38 +00001531/// ProcessPropertyDecl - Make sure that any user-defined setter/getter methods
1532/// have the property type and issue diagnostics if they don't.
1533/// Also synthesize a getter/setter method if none exist (and update the
1534/// appropriate lookup tables. FIXME: Should reconsider if adding synthesized
1535/// methods is the "right" thing to do.
1536void Sema::ProcessPropertyDecl(ObjCPropertyDecl *property,
Ted Kremenek8254aa62010-09-21 18:28:43 +00001537 ObjCContainerDecl *CD,
1538 ObjCPropertyDecl *redeclaredProperty,
1539 ObjCContainerDecl *lexicalDC) {
1540
Ted Kremenek9d64c152010-03-12 00:38:38 +00001541 ObjCMethodDecl *GetterMethod, *SetterMethod;
1542
1543 GetterMethod = CD->getInstanceMethod(property->getGetterName());
1544 SetterMethod = CD->getInstanceMethod(property->getSetterName());
1545 DiagnosePropertyAccessorMismatch(property, GetterMethod,
1546 property->getLocation());
1547
1548 if (SetterMethod) {
1549 ObjCPropertyDecl::PropertyAttributeKind CAttr =
1550 property->getPropertyAttributes();
1551 if ((!(CAttr & ObjCPropertyDecl::OBJC_PR_readonly)) &&
1552 Context.getCanonicalType(SetterMethod->getResultType()) !=
1553 Context.VoidTy)
1554 Diag(SetterMethod->getLocation(), diag::err_setter_type_void);
1555 if (SetterMethod->param_size() != 1 ||
Fariborz Jahanian2aac0c92011-09-26 22:59:09 +00001556 !Context.hasSameUnqualifiedType(
Fariborz Jahanianbb13c322011-10-15 17:36:49 +00001557 (*SetterMethod->param_begin())->getType().getNonReferenceType(),
1558 property->getType().getNonReferenceType())) {
Ted Kremenek9d64c152010-03-12 00:38:38 +00001559 Diag(property->getLocation(),
1560 diag::warn_accessor_property_type_mismatch)
1561 << property->getDeclName()
1562 << SetterMethod->getSelector();
1563 Diag(SetterMethod->getLocation(), diag::note_declared_at);
1564 }
1565 }
1566
1567 // Synthesize getter/setter methods if none exist.
1568 // Find the default getter and if one not found, add one.
1569 // FIXME: The synthesized property we set here is misleading. We almost always
1570 // synthesize these methods unless the user explicitly provided prototypes
1571 // (which is odd, but allowed). Sema should be typechecking that the
1572 // declarations jive in that situation (which it is not currently).
1573 if (!GetterMethod) {
1574 // No instance method of same name as property getter name was found.
1575 // Declare a getter method and add it to the list of methods
1576 // for this class.
Ted Kremeneka054fb42010-09-21 20:52:59 +00001577 SourceLocation Loc = redeclaredProperty ?
1578 redeclaredProperty->getLocation() :
1579 property->getLocation();
1580
1581 GetterMethod = ObjCMethodDecl::Create(Context, Loc, Loc,
1582 property->getGetterName(),
Argyrios Kyrtzidis75cf3e82011-08-17 19:25:08 +00001583 property->getType(), 0, CD, /*isInstance=*/true,
1584 /*isVariadic=*/false, /*isSynthesized=*/true,
1585 /*isImplicitlyDeclared=*/true, /*isDefined=*/false,
Ted Kremenek9d64c152010-03-12 00:38:38 +00001586 (property->getPropertyImplementation() ==
1587 ObjCPropertyDecl::Optional) ?
1588 ObjCMethodDecl::Optional :
1589 ObjCMethodDecl::Required);
1590 CD->addDecl(GetterMethod);
John McCall5de74d12010-11-10 07:01:40 +00001591
1592 AddPropertyAttrs(*this, GetterMethod, property);
1593
Ted Kremenek23173d72010-05-18 21:09:07 +00001594 // FIXME: Eventually this shouldn't be needed, as the lexical context
1595 // and the real context should be the same.
Ted Kremeneka054fb42010-09-21 20:52:59 +00001596 if (lexicalDC)
Ted Kremenek23173d72010-05-18 21:09:07 +00001597 GetterMethod->setLexicalDeclContext(lexicalDC);
Fariborz Jahanian831fb962011-06-25 00:17:46 +00001598 if (property->hasAttr<NSReturnsNotRetainedAttr>())
1599 GetterMethod->addAttr(
1600 ::new (Context) NSReturnsNotRetainedAttr(Loc, Context));
Ted Kremenek9d64c152010-03-12 00:38:38 +00001601 } else
1602 // A user declared getter will be synthesize when @synthesize of
1603 // the property with the same name is seen in the @implementation
1604 GetterMethod->setSynthesized(true);
1605 property->setGetterMethodDecl(GetterMethod);
1606
1607 // Skip setter if property is read-only.
1608 if (!property->isReadOnly()) {
1609 // Find the default setter and if one not found, add one.
1610 if (!SetterMethod) {
1611 // No instance method of same name as property setter name was found.
1612 // Declare a setter method and add it to the list of methods
1613 // for this class.
Ted Kremenek8254aa62010-09-21 18:28:43 +00001614 SourceLocation Loc = redeclaredProperty ?
1615 redeclaredProperty->getLocation() :
1616 property->getLocation();
1617
1618 SetterMethod =
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +00001619 ObjCMethodDecl::Create(Context, Loc, Loc,
Ted Kremenek8254aa62010-09-21 18:28:43 +00001620 property->getSetterName(), Context.VoidTy, 0,
Argyrios Kyrtzidis75cf3e82011-08-17 19:25:08 +00001621 CD, /*isInstance=*/true, /*isVariadic=*/false,
1622 /*isSynthesized=*/true,
1623 /*isImplicitlyDeclared=*/true,
1624 /*isDefined=*/false,
Ted Kremenek9d64c152010-03-12 00:38:38 +00001625 (property->getPropertyImplementation() ==
1626 ObjCPropertyDecl::Optional) ?
Ted Kremenek8254aa62010-09-21 18:28:43 +00001627 ObjCMethodDecl::Optional :
1628 ObjCMethodDecl::Required);
1629
Ted Kremenek9d64c152010-03-12 00:38:38 +00001630 // Invent the arguments for the setter. We don't bother making a
1631 // nice name for the argument.
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00001632 ParmVarDecl *Argument = ParmVarDecl::Create(Context, SetterMethod,
1633 Loc, Loc,
Ted Kremenek9d64c152010-03-12 00:38:38 +00001634 property->getIdentifier(),
John McCallf85e1932011-06-15 23:02:42 +00001635 property->getType().getUnqualifiedType(),
Ted Kremenek9d64c152010-03-12 00:38:38 +00001636 /*TInfo=*/0,
John McCalld931b082010-08-26 03:08:43 +00001637 SC_None,
1638 SC_None,
Ted Kremenek9d64c152010-03-12 00:38:38 +00001639 0);
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +00001640 SetterMethod->setMethodParams(Context, Argument,
1641 ArrayRef<SourceLocation>());
John McCall5de74d12010-11-10 07:01:40 +00001642
1643 AddPropertyAttrs(*this, SetterMethod, property);
1644
Ted Kremenek9d64c152010-03-12 00:38:38 +00001645 CD->addDecl(SetterMethod);
Ted Kremenek23173d72010-05-18 21:09:07 +00001646 // FIXME: Eventually this shouldn't be needed, as the lexical context
1647 // and the real context should be the same.
Ted Kremenek8254aa62010-09-21 18:28:43 +00001648 if (lexicalDC)
Ted Kremenek23173d72010-05-18 21:09:07 +00001649 SetterMethod->setLexicalDeclContext(lexicalDC);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001650 } else
1651 // A user declared setter will be synthesize when @synthesize of
1652 // the property with the same name is seen in the @implementation
1653 SetterMethod->setSynthesized(true);
1654 property->setSetterMethodDecl(SetterMethod);
1655 }
1656 // Add any synthesized methods to the global pool. This allows us to
1657 // handle the following, which is supported by GCC (and part of the design).
1658 //
1659 // @interface Foo
1660 // @property double bar;
1661 // @end
1662 //
1663 // void thisIsUnfortunate() {
1664 // id foo;
1665 // double bar = [foo bar];
1666 // }
1667 //
1668 if (GetterMethod)
1669 AddInstanceMethodToGlobalPool(GetterMethod);
1670 if (SetterMethod)
1671 AddInstanceMethodToGlobalPool(SetterMethod);
1672}
1673
John McCalld226f652010-08-21 09:40:31 +00001674void Sema::CheckObjCPropertyAttributes(Decl *PDecl,
Ted Kremenek9d64c152010-03-12 00:38:38 +00001675 SourceLocation Loc,
1676 unsigned &Attributes) {
1677 // FIXME: Improve the reported location.
John McCallf85e1932011-06-15 23:02:42 +00001678 if (!PDecl || PDecl->isInvalidDecl())
Ted Kremenek5fcd52a2010-04-05 22:39:42 +00001679 return;
1680
1681 ObjCPropertyDecl *PropertyDecl = cast<ObjCPropertyDecl>(PDecl);
Fariborz Jahanian842f07b2010-03-30 22:40:11 +00001682 QualType PropertyTy = PropertyDecl->getType();
Ted Kremenek9d64c152010-03-12 00:38:38 +00001683
1684 // readonly and readwrite/assign/retain/copy conflict.
1685 if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
1686 (Attributes & (ObjCDeclSpec::DQ_PR_readwrite |
1687 ObjCDeclSpec::DQ_PR_assign |
John McCallf85e1932011-06-15 23:02:42 +00001688 ObjCDeclSpec::DQ_PR_unsafe_unretained |
Ted Kremenek9d64c152010-03-12 00:38:38 +00001689 ObjCDeclSpec::DQ_PR_copy |
John McCallf85e1932011-06-15 23:02:42 +00001690 ObjCDeclSpec::DQ_PR_retain |
1691 ObjCDeclSpec::DQ_PR_strong))) {
Ted Kremenek9d64c152010-03-12 00:38:38 +00001692 const char * which = (Attributes & ObjCDeclSpec::DQ_PR_readwrite) ?
1693 "readwrite" :
1694 (Attributes & ObjCDeclSpec::DQ_PR_assign) ?
1695 "assign" :
John McCallf85e1932011-06-15 23:02:42 +00001696 (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) ?
1697 "unsafe_unretained" :
Ted Kremenek9d64c152010-03-12 00:38:38 +00001698 (Attributes & ObjCDeclSpec::DQ_PR_copy) ?
1699 "copy" : "retain";
1700
1701 Diag(Loc, (Attributes & (ObjCDeclSpec::DQ_PR_readwrite)) ?
1702 diag::err_objc_property_attr_mutually_exclusive :
1703 diag::warn_objc_property_attr_mutually_exclusive)
1704 << "readonly" << which;
1705 }
1706
1707 // Check for copy or retain on non-object types.
John McCallf85e1932011-06-15 23:02:42 +00001708 if ((Attributes & (ObjCDeclSpec::DQ_PR_weak | ObjCDeclSpec::DQ_PR_copy |
1709 ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong)) &&
1710 !PropertyTy->isObjCRetainableType() &&
Fariborz Jahanian842f07b2010-03-30 22:40:11 +00001711 !PropertyDecl->getAttr<ObjCNSObjectAttr>()) {
Ted Kremenek9d64c152010-03-12 00:38:38 +00001712 Diag(Loc, diag::err_objc_property_requires_object)
John McCallf85e1932011-06-15 23:02:42 +00001713 << (Attributes & ObjCDeclSpec::DQ_PR_weak ? "weak" :
1714 Attributes & ObjCDeclSpec::DQ_PR_copy ? "copy" : "retain (or strong)");
1715 Attributes &= ~(ObjCDeclSpec::DQ_PR_weak | ObjCDeclSpec::DQ_PR_copy |
1716 ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001717 }
1718
1719 // Check for more than one of { assign, copy, retain }.
1720 if (Attributes & ObjCDeclSpec::DQ_PR_assign) {
1721 if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
1722 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1723 << "assign" << "copy";
1724 Attributes &= ~ObjCDeclSpec::DQ_PR_copy;
1725 }
1726 if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
1727 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1728 << "assign" << "retain";
1729 Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
1730 }
John McCallf85e1932011-06-15 23:02:42 +00001731 if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
1732 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1733 << "assign" << "strong";
1734 Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
1735 }
1736 if (getLangOptions().ObjCAutoRefCount &&
1737 (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1738 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1739 << "assign" << "weak";
1740 Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
1741 }
1742 } else if (Attributes & ObjCDeclSpec::DQ_PR_unsafe_unretained) {
1743 if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
1744 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1745 << "unsafe_unretained" << "copy";
1746 Attributes &= ~ObjCDeclSpec::DQ_PR_copy;
1747 }
1748 if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
1749 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1750 << "unsafe_unretained" << "retain";
1751 Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
1752 }
1753 if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
1754 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1755 << "unsafe_unretained" << "strong";
1756 Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
1757 }
1758 if (getLangOptions().ObjCAutoRefCount &&
1759 (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1760 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1761 << "unsafe_unretained" << "weak";
1762 Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
1763 }
Ted Kremenek9d64c152010-03-12 00:38:38 +00001764 } else if (Attributes & ObjCDeclSpec::DQ_PR_copy) {
1765 if (Attributes & ObjCDeclSpec::DQ_PR_retain) {
1766 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1767 << "copy" << "retain";
1768 Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
1769 }
John McCallf85e1932011-06-15 23:02:42 +00001770 if (Attributes & ObjCDeclSpec::DQ_PR_strong) {
1771 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1772 << "copy" << "strong";
1773 Attributes &= ~ObjCDeclSpec::DQ_PR_strong;
1774 }
1775 if (Attributes & ObjCDeclSpec::DQ_PR_weak) {
1776 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1777 << "copy" << "weak";
1778 Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
1779 }
1780 }
1781 else if ((Attributes & ObjCDeclSpec::DQ_PR_retain) &&
1782 (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1783 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1784 << "retain" << "weak";
Fariborz Jahanian528a4992011-09-14 18:03:46 +00001785 Attributes &= ~ObjCDeclSpec::DQ_PR_retain;
John McCallf85e1932011-06-15 23:02:42 +00001786 }
1787 else if ((Attributes & ObjCDeclSpec::DQ_PR_strong) &&
1788 (Attributes & ObjCDeclSpec::DQ_PR_weak)) {
1789 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1790 << "strong" << "weak";
1791 Attributes &= ~ObjCDeclSpec::DQ_PR_weak;
Ted Kremenek9d64c152010-03-12 00:38:38 +00001792 }
1793
Fariborz Jahanian9d1bbea2011-10-10 21:53:24 +00001794 if ((Attributes & ObjCDeclSpec::DQ_PR_atomic) &&
1795 (Attributes & ObjCDeclSpec::DQ_PR_nonatomic)) {
1796 Diag(Loc, diag::err_objc_property_attr_mutually_exclusive)
1797 << "atomic" << "nonatomic";
1798 Attributes &= ~ObjCDeclSpec::DQ_PR_atomic;
1799 }
1800
Ted Kremenek9d64c152010-03-12 00:38:38 +00001801 // Warn if user supplied no assignment attribute, property is
1802 // readwrite, and this is an object type.
1803 if (!(Attributes & (ObjCDeclSpec::DQ_PR_assign | ObjCDeclSpec::DQ_PR_copy |
John McCallf85e1932011-06-15 23:02:42 +00001804 ObjCDeclSpec::DQ_PR_unsafe_unretained |
1805 ObjCDeclSpec::DQ_PR_retain | ObjCDeclSpec::DQ_PR_strong |
1806 ObjCDeclSpec::DQ_PR_weak)) &&
Ted Kremenek9d64c152010-03-12 00:38:38 +00001807 PropertyTy->isObjCObjectPointerType()) {
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001808 if (getLangOptions().ObjCAutoRefCount)
1809 // With arc, @property definitions should default to (strong) when
Fariborz Jahanianf21a92d2011-11-08 20:58:53 +00001810 // not specified; including when property is 'readonly'.
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001811 PropertyDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_strong);
Fariborz Jahanianf21a92d2011-11-08 20:58:53 +00001812 else if (!(Attributes & ObjCDeclSpec::DQ_PR_readonly)) {
Fariborz Jahanian9f37cd12012-01-04 00:31:53 +00001813 bool isAnyClassTy =
1814 (PropertyTy->isObjCClassType() ||
1815 PropertyTy->isObjCQualifiedClassType());
1816 // In non-gc, non-arc mode, 'Class' is treated as a 'void *' no need to
1817 // issue any warning.
1818 if (isAnyClassTy && getLangOptions().getGC() == LangOptions::NonGC)
1819 ;
1820 else {
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001821 // Skip this warning in gc-only mode.
Douglas Gregore289d812011-09-13 17:21:33 +00001822 if (getLangOptions().getGC() != LangOptions::GCOnly)
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001823 Diag(Loc, diag::warn_objc_property_no_assignment_attribute);
Ted Kremenek9d64c152010-03-12 00:38:38 +00001824
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001825 // If non-gc code warn that this is likely inappropriate.
Douglas Gregore289d812011-09-13 17:21:33 +00001826 if (getLangOptions().getGC() == LangOptions::NonGC)
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001827 Diag(Loc, diag::warn_objc_property_default_assign_on_object);
Fariborz Jahanian9f37cd12012-01-04 00:31:53 +00001828 }
Fariborz Jahanianbc03aea2011-08-19 19:28:44 +00001829 }
Ted Kremenek9d64c152010-03-12 00:38:38 +00001830
1831 // FIXME: Implement warning dependent on NSCopying being
1832 // implemented. See also:
1833 // <rdar://5168496&4855821&5607453&5096644&4947311&5698469&4947014&5168496>
1834 // (please trim this list while you are at it).
1835 }
1836
1837 if (!(Attributes & ObjCDeclSpec::DQ_PR_copy)
Fariborz Jahanian2b77cb82011-01-05 23:00:04 +00001838 &&!(Attributes & ObjCDeclSpec::DQ_PR_readonly)
Douglas Gregore289d812011-09-13 17:21:33 +00001839 && getLangOptions().getGC() == LangOptions::GCOnly
Ted Kremenek9d64c152010-03-12 00:38:38 +00001840 && PropertyTy->isBlockPointerType())
1841 Diag(Loc, diag::warn_objc_property_copy_missing_on_block);
Fariborz Jahanian528a4992011-09-14 18:03:46 +00001842 else if (getLangOptions().ObjCAutoRefCount &&
1843 (Attributes & ObjCDeclSpec::DQ_PR_retain) &&
1844 !(Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
1845 !(Attributes & ObjCDeclSpec::DQ_PR_strong) &&
1846 PropertyTy->isBlockPointerType())
1847 Diag(Loc, diag::warn_objc_property_retain_of_block);
Fariborz Jahanian48a98c72011-11-01 23:02:16 +00001848
1849 if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) &&
1850 (Attributes & ObjCDeclSpec::DQ_PR_setter))
1851 Diag(Loc, diag::warn_objc_readonly_property_has_setter);
1852
Ted Kremenek9d64c152010-03-12 00:38:38 +00001853}