blob: 168ccfc3f578cb03cede886852964457042358e8 [file] [log] [blame]
Chris Lattner4d391482007-12-12 07:09:47 +00001//===--- SemaDeclObjC.cpp - Semantic Analysis for ObjC Declarations -------===//
2//
3// The LLVM Compiler Infrastructure
4//
Chris Lattner0bc735f2007-12-29 19:59:25 +00005// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
Chris Lattner4d391482007-12-12 07:09:47 +00007//
8//===----------------------------------------------------------------------===//
9//
10// This file implements semantic analysis for Objective C declarations.
11//
12//===----------------------------------------------------------------------===//
13
John McCall2d887082010-08-25 22:03:47 +000014#include "clang/Sema/SemaInternal.h"
Douglas Gregore737f502010-08-12 20:07:10 +000015#include "clang/Sema/Lookup.h"
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +000016#include "clang/Sema/ExternalSemaSource.h"
John McCall5f1e0942010-08-24 08:50:51 +000017#include "clang/Sema/Scope.h"
John McCall781472f2010-08-25 08:40:02 +000018#include "clang/Sema/ScopeInfo.h"
John McCallf85e1932011-06-15 23:02:42 +000019#include "clang/AST/ASTConsumer.h"
Steve Naroffca331292009-03-03 14:49:36 +000020#include "clang/AST/Expr.h"
John McCallf85e1932011-06-15 23:02:42 +000021#include "clang/AST/ExprObjC.h"
Chris Lattner4d391482007-12-12 07:09:47 +000022#include "clang/AST/ASTContext.h"
23#include "clang/AST/DeclObjC.h"
Argyrios Kyrtzidis1a434152011-11-12 21:07:52 +000024#include "clang/AST/ASTMutationListener.h"
John McCallf85e1932011-06-15 23:02:42 +000025#include "clang/Basic/SourceManager.h"
John McCall19510852010-08-20 18:27:03 +000026#include "clang/Sema/DeclSpec.h"
John McCall50df6ae2010-08-25 07:03:20 +000027#include "llvm/ADT/DenseSet.h"
28
Chris Lattner4d391482007-12-12 07:09:47 +000029using namespace clang;
30
John McCallf85e1932011-06-15 23:02:42 +000031/// Check whether the given method, which must be in the 'init'
32/// family, is a valid member of that family.
33///
34/// \param receiverTypeIfCall - if null, check this as if declaring it;
35/// if non-null, check this as if making a call to it with the given
36/// receiver type
37///
38/// \return true to indicate that there was an error and appropriate
39/// actions were taken
40bool Sema::checkInitMethod(ObjCMethodDecl *method,
41 QualType receiverTypeIfCall) {
42 if (method->isInvalidDecl()) return true;
43
44 // This castAs is safe: methods that don't return an object
45 // pointer won't be inferred as inits and will reject an explicit
46 // objc_method_family(init).
47
48 // We ignore protocols here. Should we? What about Class?
49
50 const ObjCObjectType *result = method->getResultType()
51 ->castAs<ObjCObjectPointerType>()->getObjectType();
52
53 if (result->isObjCId()) {
54 return false;
55 } else if (result->isObjCClass()) {
56 // fall through: always an error
57 } else {
58 ObjCInterfaceDecl *resultClass = result->getInterface();
59 assert(resultClass && "unexpected object type!");
60
61 // It's okay for the result type to still be a forward declaration
62 // if we're checking an interface declaration.
Douglas Gregor7723fec2011-12-15 20:29:51 +000063 if (!resultClass->hasDefinition()) {
John McCallf85e1932011-06-15 23:02:42 +000064 if (receiverTypeIfCall.isNull() &&
65 !isa<ObjCImplementationDecl>(method->getDeclContext()))
66 return false;
67
68 // Otherwise, we try to compare class types.
69 } else {
70 // If this method was declared in a protocol, we can't check
71 // anything unless we have a receiver type that's an interface.
72 const ObjCInterfaceDecl *receiverClass = 0;
73 if (isa<ObjCProtocolDecl>(method->getDeclContext())) {
74 if (receiverTypeIfCall.isNull())
75 return false;
76
77 receiverClass = receiverTypeIfCall->castAs<ObjCObjectPointerType>()
78 ->getInterfaceDecl();
79
80 // This can be null for calls to e.g. id<Foo>.
81 if (!receiverClass) return false;
82 } else {
83 receiverClass = method->getClassInterface();
84 assert(receiverClass && "method not associated with a class!");
85 }
86
87 // If either class is a subclass of the other, it's fine.
88 if (receiverClass->isSuperClassOf(resultClass) ||
89 resultClass->isSuperClassOf(receiverClass))
90 return false;
91 }
92 }
93
94 SourceLocation loc = method->getLocation();
95
96 // If we're in a system header, and this is not a call, just make
97 // the method unusable.
98 if (receiverTypeIfCall.isNull() && getSourceManager().isInSystemHeader(loc)) {
99 method->addAttr(new (Context) UnavailableAttr(loc, Context,
100 "init method returns a type unrelated to its receiver type"));
101 return true;
102 }
103
104 // Otherwise, it's an error.
105 Diag(loc, diag::err_arc_init_method_unrelated_result_type);
106 method->setInvalidDecl();
107 return true;
108}
109
Fariborz Jahanian3240fe32011-09-27 22:35:36 +0000110void Sema::CheckObjCMethodOverride(ObjCMethodDecl *NewMethod,
Douglas Gregor926df6c2011-06-11 01:09:30 +0000111 const ObjCMethodDecl *Overridden,
112 bool IsImplementation) {
113 if (Overridden->hasRelatedResultType() &&
114 !NewMethod->hasRelatedResultType()) {
115 // This can only happen when the method follows a naming convention that
116 // implies a related result type, and the original (overridden) method has
117 // a suitable return type, but the new (overriding) method does not have
118 // a suitable return type.
119 QualType ResultType = NewMethod->getResultType();
120 SourceRange ResultTypeRange;
121 if (const TypeSourceInfo *ResultTypeInfo
John McCallf85e1932011-06-15 23:02:42 +0000122 = NewMethod->getResultTypeSourceInfo())
Douglas Gregor926df6c2011-06-11 01:09:30 +0000123 ResultTypeRange = ResultTypeInfo->getTypeLoc().getSourceRange();
124
125 // Figure out which class this method is part of, if any.
126 ObjCInterfaceDecl *CurrentClass
127 = dyn_cast<ObjCInterfaceDecl>(NewMethod->getDeclContext());
128 if (!CurrentClass) {
129 DeclContext *DC = NewMethod->getDeclContext();
130 if (ObjCCategoryDecl *Cat = dyn_cast<ObjCCategoryDecl>(DC))
131 CurrentClass = Cat->getClassInterface();
132 else if (ObjCImplDecl *Impl = dyn_cast<ObjCImplDecl>(DC))
133 CurrentClass = Impl->getClassInterface();
134 else if (ObjCCategoryImplDecl *CatImpl
135 = dyn_cast<ObjCCategoryImplDecl>(DC))
136 CurrentClass = CatImpl->getClassInterface();
137 }
138
139 if (CurrentClass) {
140 Diag(NewMethod->getLocation(),
141 diag::warn_related_result_type_compatibility_class)
142 << Context.getObjCInterfaceType(CurrentClass)
143 << ResultType
144 << ResultTypeRange;
145 } else {
146 Diag(NewMethod->getLocation(),
147 diag::warn_related_result_type_compatibility_protocol)
148 << ResultType
149 << ResultTypeRange;
150 }
151
Douglas Gregore97179c2011-09-08 01:46:34 +0000152 if (ObjCMethodFamily Family = Overridden->getMethodFamily())
153 Diag(Overridden->getLocation(),
154 diag::note_related_result_type_overridden_family)
155 << Family;
156 else
157 Diag(Overridden->getLocation(),
158 diag::note_related_result_type_overridden);
Douglas Gregor926df6c2011-06-11 01:09:30 +0000159 }
Fariborz Jahanian3240fe32011-09-27 22:35:36 +0000160 if (getLangOptions().ObjCAutoRefCount) {
161 if ((NewMethod->hasAttr<NSReturnsRetainedAttr>() !=
162 Overridden->hasAttr<NSReturnsRetainedAttr>())) {
163 Diag(NewMethod->getLocation(),
164 diag::err_nsreturns_retained_attribute_mismatch) << 1;
165 Diag(Overridden->getLocation(), diag::note_previous_decl)
166 << "method";
167 }
168 if ((NewMethod->hasAttr<NSReturnsNotRetainedAttr>() !=
169 Overridden->hasAttr<NSReturnsNotRetainedAttr>())) {
170 Diag(NewMethod->getLocation(),
171 diag::err_nsreturns_retained_attribute_mismatch) << 0;
172 Diag(Overridden->getLocation(), diag::note_previous_decl)
173 << "method";
174 }
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +0000175 ObjCMethodDecl::param_const_iterator oi = Overridden->param_begin();
176 for (ObjCMethodDecl::param_iterator
177 ni = NewMethod->param_begin(), ne = NewMethod->param_end();
Fariborz Jahanian3240fe32011-09-27 22:35:36 +0000178 ni != ne; ++ni, ++oi) {
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +0000179 const ParmVarDecl *oldDecl = (*oi);
Fariborz Jahanian3240fe32011-09-27 22:35:36 +0000180 ParmVarDecl *newDecl = (*ni);
181 if (newDecl->hasAttr<NSConsumedAttr>() !=
182 oldDecl->hasAttr<NSConsumedAttr>()) {
183 Diag(newDecl->getLocation(),
184 diag::err_nsconsumed_attribute_mismatch);
185 Diag(oldDecl->getLocation(), diag::note_previous_decl)
186 << "parameter";
187 }
188 }
189 }
Douglas Gregor926df6c2011-06-11 01:09:30 +0000190}
191
John McCallf85e1932011-06-15 23:02:42 +0000192/// \brief Check a method declaration for compatibility with the Objective-C
193/// ARC conventions.
194static bool CheckARCMethodDecl(Sema &S, ObjCMethodDecl *method) {
195 ObjCMethodFamily family = method->getMethodFamily();
196 switch (family) {
197 case OMF_None:
198 case OMF_dealloc:
Nico Weber80cb6e62011-08-28 22:35:17 +0000199 case OMF_finalize:
John McCallf85e1932011-06-15 23:02:42 +0000200 case OMF_retain:
201 case OMF_release:
202 case OMF_autorelease:
203 case OMF_retainCount:
204 case OMF_self:
John McCall6c2c2502011-07-22 02:45:48 +0000205 case OMF_performSelector:
John McCallf85e1932011-06-15 23:02:42 +0000206 return false;
207
208 case OMF_init:
209 // If the method doesn't obey the init rules, don't bother annotating it.
210 if (S.checkInitMethod(method, QualType()))
211 return true;
212
213 method->addAttr(new (S.Context) NSConsumesSelfAttr(SourceLocation(),
214 S.Context));
215
216 // Don't add a second copy of this attribute, but otherwise don't
217 // let it be suppressed.
218 if (method->hasAttr<NSReturnsRetainedAttr>())
219 return false;
220 break;
221
222 case OMF_alloc:
223 case OMF_copy:
224 case OMF_mutableCopy:
225 case OMF_new:
226 if (method->hasAttr<NSReturnsRetainedAttr>() ||
227 method->hasAttr<NSReturnsNotRetainedAttr>() ||
228 method->hasAttr<NSReturnsAutoreleasedAttr>())
229 return false;
230 break;
231 }
232
233 method->addAttr(new (S.Context) NSReturnsRetainedAttr(SourceLocation(),
234 S.Context));
235 return false;
236}
237
Fariborz Jahanian5ac96d52011-02-15 17:49:58 +0000238static void DiagnoseObjCImplementedDeprecations(Sema &S,
239 NamedDecl *ND,
240 SourceLocation ImplLoc,
241 int select) {
Douglas Gregor0a0d2b12011-03-23 00:50:03 +0000242 if (ND && ND->isDeprecated()) {
Fariborz Jahanian98d810e2011-02-16 00:30:31 +0000243 S.Diag(ImplLoc, diag::warn_deprecated_def) << select;
Fariborz Jahanian5ac96d52011-02-15 17:49:58 +0000244 if (select == 0)
245 S.Diag(ND->getLocation(), diag::note_method_declared_at);
246 else
247 S.Diag(ND->getLocation(), diag::note_previous_decl) << "class";
248 }
249}
250
Fariborz Jahanian140ab232011-08-31 17:37:55 +0000251/// AddAnyMethodToGlobalPool - Add any method, instance or factory to global
252/// pool.
253void Sema::AddAnyMethodToGlobalPool(Decl *D) {
254 ObjCMethodDecl *MDecl = dyn_cast_or_null<ObjCMethodDecl>(D);
255
256 // If we don't have a valid method decl, simply return.
257 if (!MDecl)
258 return;
259 if (MDecl->isInstanceMethod())
260 AddInstanceMethodToGlobalPool(MDecl, true);
261 else
262 AddFactoryMethodToGlobalPool(MDecl, true);
263}
264
Steve Naroffebf64432009-02-28 16:59:13 +0000265/// ActOnStartOfObjCMethodDef - This routine sets up parameters; invisible
Chris Lattner4d391482007-12-12 07:09:47 +0000266/// and user declared, in the method definition's AST.
John McCalld226f652010-08-21 09:40:31 +0000267void Sema::ActOnStartOfObjCMethodDef(Scope *FnBodyScope, Decl *D) {
Argyrios Kyrtzidis53d0ea52008-06-28 06:07:14 +0000268 assert(getCurMethodDecl() == 0 && "Method parsing confused");
John McCalld226f652010-08-21 09:40:31 +0000269 ObjCMethodDecl *MDecl = dyn_cast_or_null<ObjCMethodDecl>(D);
Mike Stump1eb44332009-09-09 15:08:12 +0000270
Steve Naroff394f3f42008-07-25 17:57:26 +0000271 // If we don't have a valid method decl, simply return.
272 if (!MDecl)
273 return;
Steve Naroffa56f6162007-12-18 01:30:32 +0000274
Chris Lattner4d391482007-12-12 07:09:47 +0000275 // Allow all of Sema to see that we are entering a method definition.
Douglas Gregor44b43212008-12-11 16:49:14 +0000276 PushDeclContext(FnBodyScope, MDecl);
Douglas Gregor9ea9bdb2010-03-01 23:15:13 +0000277 PushFunctionScope();
278
Chris Lattner4d391482007-12-12 07:09:47 +0000279 // Create Decl objects for each parameter, entrring them in the scope for
280 // binding to their use.
Chris Lattner4d391482007-12-12 07:09:47 +0000281
282 // Insert the invisible arguments, self and _cmd!
Fariborz Jahanianfef30b52008-12-09 20:23:04 +0000283 MDecl->createImplicitParams(Context, MDecl->getClassInterface());
Mike Stump1eb44332009-09-09 15:08:12 +0000284
Daniel Dunbar451318c2008-08-26 06:07:48 +0000285 PushOnScopeChains(MDecl->getSelfDecl(), FnBodyScope);
286 PushOnScopeChains(MDecl->getCmdDecl(), FnBodyScope);
Chris Lattner04421082008-04-08 04:40:51 +0000287
Chris Lattner8123a952008-04-10 02:22:51 +0000288 // Introduce all of the other parameters into this scope.
Chris Lattner89951a82009-02-20 18:43:26 +0000289 for (ObjCMethodDecl::param_iterator PI = MDecl->param_begin(),
Fariborz Jahanian23c01042010-09-17 22:07:07 +0000290 E = MDecl->param_end(); PI != E; ++PI) {
291 ParmVarDecl *Param = (*PI);
292 if (!Param->isInvalidDecl() &&
293 RequireCompleteType(Param->getLocation(), Param->getType(),
294 diag::err_typecheck_decl_incomplete_type))
295 Param->setInvalidDecl();
Chris Lattner89951a82009-02-20 18:43:26 +0000296 if ((*PI)->getIdentifier())
297 PushOnScopeChains(*PI, FnBodyScope);
Fariborz Jahanian23c01042010-09-17 22:07:07 +0000298 }
John McCallf85e1932011-06-15 23:02:42 +0000299
300 // In ARC, disallow definition of retain/release/autorelease/retainCount
301 if (getLangOptions().ObjCAutoRefCount) {
302 switch (MDecl->getMethodFamily()) {
303 case OMF_retain:
304 case OMF_retainCount:
305 case OMF_release:
306 case OMF_autorelease:
307 Diag(MDecl->getLocation(), diag::err_arc_illegal_method_def)
308 << MDecl->getSelector();
309 break;
310
311 case OMF_None:
312 case OMF_dealloc:
Nico Weber80cb6e62011-08-28 22:35:17 +0000313 case OMF_finalize:
John McCallf85e1932011-06-15 23:02:42 +0000314 case OMF_alloc:
315 case OMF_init:
316 case OMF_mutableCopy:
317 case OMF_copy:
318 case OMF_new:
319 case OMF_self:
Fariborz Jahanian9670e172011-07-05 22:38:59 +0000320 case OMF_performSelector:
John McCallf85e1932011-06-15 23:02:42 +0000321 break;
322 }
323 }
324
Nico Weber9a1ecf02011-08-22 17:25:57 +0000325 // Warn on deprecated methods under -Wdeprecated-implementations,
326 // and prepare for warning on missing super calls.
327 if (ObjCInterfaceDecl *IC = MDecl->getClassInterface()) {
Fariborz Jahanian5ac96d52011-02-15 17:49:58 +0000328 if (ObjCMethodDecl *IMD =
Fariborz Jahanianb1224f62011-02-15 00:59:30 +0000329 IC->lookupMethod(MDecl->getSelector(), MDecl->isInstanceMethod()))
Fariborz Jahanian5ac96d52011-02-15 17:49:58 +0000330 DiagnoseObjCImplementedDeprecations(*this,
331 dyn_cast<NamedDecl>(IMD),
332 MDecl->getLocation(), 0);
Nico Weber9a1ecf02011-08-22 17:25:57 +0000333
Nico Weber80cb6e62011-08-28 22:35:17 +0000334 // If this is "dealloc" or "finalize", set some bit here.
Nico Weber9a1ecf02011-08-22 17:25:57 +0000335 // Then in ActOnSuperMessage() (SemaExprObjC), set it back to false.
336 // Finally, in ActOnFinishFunctionBody() (SemaDecl), warn if flag is set.
337 // Only do this if the current class actually has a superclass.
Nico Weber80cb6e62011-08-28 22:35:17 +0000338 if (IC->getSuperClass()) {
Ted Kremenek4eb14ca2011-08-22 19:07:43 +0000339 ObjCShouldCallSuperDealloc =
Ted Kremenek8cd8de42011-09-28 19:32:29 +0000340 !(Context.getLangOptions().ObjCAutoRefCount ||
341 Context.getLangOptions().getGC() == LangOptions::GCOnly) &&
Ted Kremenek4eb14ca2011-08-22 19:07:43 +0000342 MDecl->getMethodFamily() == OMF_dealloc;
Nico Weber27f07762011-08-29 22:59:14 +0000343 ObjCShouldCallSuperFinalize =
Ted Kremenek8cd8de42011-09-28 19:32:29 +0000344 Context.getLangOptions().getGC() != LangOptions::NonGC &&
Nico Weber27f07762011-08-29 22:59:14 +0000345 MDecl->getMethodFamily() == OMF_finalize;
Nico Weber80cb6e62011-08-28 22:35:17 +0000346 }
Nico Weber9a1ecf02011-08-22 17:25:57 +0000347 }
Chris Lattner4d391482007-12-12 07:09:47 +0000348}
349
John McCalld226f652010-08-21 09:40:31 +0000350Decl *Sema::
Chris Lattner7caeabd2008-07-21 22:17:28 +0000351ActOnStartClassInterface(SourceLocation AtInterfaceLoc,
352 IdentifierInfo *ClassName, SourceLocation ClassLoc,
353 IdentifierInfo *SuperName, SourceLocation SuperLoc,
John McCalld226f652010-08-21 09:40:31 +0000354 Decl * const *ProtoRefs, unsigned NumProtoRefs,
Douglas Gregor18df52b2010-01-16 15:02:53 +0000355 const SourceLocation *ProtoLocs,
Chris Lattner7caeabd2008-07-21 22:17:28 +0000356 SourceLocation EndProtoLoc, AttributeList *AttrList) {
Chris Lattner4d391482007-12-12 07:09:47 +0000357 assert(ClassName && "Missing class identifier");
Mike Stump1eb44332009-09-09 15:08:12 +0000358
Chris Lattner4d391482007-12-12 07:09:47 +0000359 // Check for another declaration kind with the same name.
Douglas Gregorc83c6872010-04-15 22:33:43 +0000360 NamedDecl *PrevDecl = LookupSingleName(TUScope, ClassName, ClassLoc,
Douglas Gregorc0b39642010-04-15 23:40:53 +0000361 LookupOrdinaryName, ForRedeclaration);
Douglas Gregor72c3f312008-12-05 18:15:24 +0000362
Ted Kremeneka526c5c2008-01-07 19:49:32 +0000363 if (PrevDecl && !isa<ObjCInterfaceDecl>(PrevDecl)) {
Chris Lattner3c73c412008-11-19 08:23:25 +0000364 Diag(ClassLoc, diag::err_redefinition_different_kind) << ClassName;
Chris Lattner5f4a6822008-11-23 23:12:31 +0000365 Diag(PrevDecl->getLocation(), diag::note_previous_definition);
Chris Lattner4d391482007-12-12 07:09:47 +0000366 }
Mike Stump1eb44332009-09-09 15:08:12 +0000367
Douglas Gregor7723fec2011-12-15 20:29:51 +0000368 // Create a declaration to describe this @interface.
Douglas Gregor0af55012011-12-16 03:12:41 +0000369 ObjCInterfaceDecl* PrevIDecl = dyn_cast_or_null<ObjCInterfaceDecl>(PrevDecl);
Douglas Gregor7723fec2011-12-15 20:29:51 +0000370 ObjCInterfaceDecl *IDecl
371 = ObjCInterfaceDecl::Create(Context, CurContext, AtInterfaceLoc, ClassName,
Douglas Gregor0af55012011-12-16 03:12:41 +0000372 PrevIDecl, ClassLoc);
Douglas Gregor7723fec2011-12-15 20:29:51 +0000373
Douglas Gregor7723fec2011-12-15 20:29:51 +0000374 if (PrevIDecl) {
375 // Class already seen. Was it a definition?
376 if (ObjCInterfaceDecl *Def = PrevIDecl->getDefinition()) {
377 Diag(AtInterfaceLoc, diag::err_duplicate_class_def)
378 << PrevIDecl->getDeclName();
Douglas Gregor2e5c15b2011-12-15 05:27:12 +0000379 Diag(Def->getLocation(), diag::note_previous_definition);
Douglas Gregor7723fec2011-12-15 20:29:51 +0000380 IDecl->setInvalidDecl();
Chris Lattner4d391482007-12-12 07:09:47 +0000381 }
Chris Lattner4d391482007-12-12 07:09:47 +0000382 }
Douglas Gregor7723fec2011-12-15 20:29:51 +0000383
384 if (AttrList)
385 ProcessDeclAttributeList(TUScope, IDecl, AttrList);
386 PushOnScopeChains(IDecl, TUScope);
Mike Stump1eb44332009-09-09 15:08:12 +0000387
Douglas Gregor7723fec2011-12-15 20:29:51 +0000388 // Start the definition of this class. If we're in a redefinition case, there
389 // may already be a definition, so we'll end up adding to it.
Douglas Gregor2e5c15b2011-12-15 05:27:12 +0000390 if (!IDecl->hasDefinition())
391 IDecl->startDefinition();
392
Chris Lattner4d391482007-12-12 07:09:47 +0000393 if (SuperName) {
Chris Lattner4d391482007-12-12 07:09:47 +0000394 // Check if a different kind of symbol declared in this scope.
Douglas Gregorc83c6872010-04-15 22:33:43 +0000395 PrevDecl = LookupSingleName(TUScope, SuperName, SuperLoc,
396 LookupOrdinaryName);
Douglas Gregorf06cdae2010-01-03 18:01:57 +0000397
398 if (!PrevDecl) {
399 // Try to correct for a typo in the superclass name.
Douglas Gregord8bba9c2011-06-28 16:20:02 +0000400 TypoCorrection Corrected = CorrectTypo(
401 DeclarationNameInfo(SuperName, SuperLoc), LookupOrdinaryName, TUScope,
402 NULL, NULL, false, CTC_NoKeywords);
403 if ((PrevDecl = Corrected.getCorrectionDeclAs<ObjCInterfaceDecl>())) {
Douglas Gregor60ef3082011-12-15 00:29:59 +0000404 if (declaresSameEntity(PrevDecl, IDecl)) {
Douglas Gregora38c4732011-12-01 15:37:53 +0000405 // Don't correct to the class we're defining.
406 PrevDecl = 0;
407 } else {
408 Diag(SuperLoc, diag::err_undef_superclass_suggest)
409 << SuperName << ClassName << PrevDecl->getDeclName();
410 Diag(PrevDecl->getLocation(), diag::note_previous_decl)
411 << PrevDecl->getDeclName();
412 }
Douglas Gregorf06cdae2010-01-03 18:01:57 +0000413 }
414 }
415
Douglas Gregor60ef3082011-12-15 00:29:59 +0000416 if (declaresSameEntity(PrevDecl, IDecl)) {
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000417 Diag(SuperLoc, diag::err_recursive_superclass)
418 << SuperName << ClassName << SourceRange(AtInterfaceLoc, ClassLoc);
Douglas Gregor05c272f2011-12-15 22:34:59 +0000419 IDecl->setEndOfDefinitionLoc(ClassLoc);
Mike Stumpac5fc7c2009-08-04 21:02:39 +0000420 } else {
Mike Stump1eb44332009-09-09 15:08:12 +0000421 ObjCInterfaceDecl *SuperClassDecl =
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000422 dyn_cast_or_null<ObjCInterfaceDecl>(PrevDecl);
Chris Lattner3c73c412008-11-19 08:23:25 +0000423
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000424 // Diagnose classes that inherit from deprecated classes.
425 if (SuperClassDecl)
426 (void)DiagnoseUseOfDecl(SuperClassDecl, SuperLoc);
Mike Stump1eb44332009-09-09 15:08:12 +0000427
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000428 if (PrevDecl && SuperClassDecl == 0) {
429 // The previous declaration was not a class decl. Check if we have a
430 // typedef. If we do, get the underlying class type.
Richard Smith162e1c12011-04-15 14:24:37 +0000431 if (const TypedefNameDecl *TDecl =
432 dyn_cast_or_null<TypedefNameDecl>(PrevDecl)) {
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000433 QualType T = TDecl->getUnderlyingType();
John McCallc12c5bb2010-05-15 11:32:37 +0000434 if (T->isObjCObjectType()) {
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000435 if (NamedDecl *IDecl = T->getAs<ObjCObjectType>()->getInterface())
436 SuperClassDecl = dyn_cast<ObjCInterfaceDecl>(IDecl);
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000437 }
438 }
Mike Stump1eb44332009-09-09 15:08:12 +0000439
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000440 // This handles the following case:
441 //
442 // typedef int SuperClass;
443 // @interface MyClass : SuperClass {} @end
444 //
445 if (!SuperClassDecl) {
446 Diag(SuperLoc, diag::err_redefinition_different_kind) << SuperName;
447 Diag(PrevDecl->getLocation(), diag::note_previous_definition);
Steve Naroff818cb9e2009-02-04 17:14:05 +0000448 }
449 }
Mike Stump1eb44332009-09-09 15:08:12 +0000450
Richard Smith162e1c12011-04-15 14:24:37 +0000451 if (!dyn_cast_or_null<TypedefNameDecl>(PrevDecl)) {
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000452 if (!SuperClassDecl)
453 Diag(SuperLoc, diag::err_undef_superclass)
454 << SuperName << ClassName << SourceRange(AtInterfaceLoc, ClassLoc);
Douglas Gregorb3029962011-11-14 22:10:01 +0000455 else if (RequireCompleteType(SuperLoc,
456 Context.getObjCInterfaceType(SuperClassDecl),
457 PDiag(diag::err_forward_superclass)
458 << SuperClassDecl->getDeclName()
459 << ClassName
460 << SourceRange(AtInterfaceLoc, ClassLoc))) {
Fariborz Jahaniana8139732011-06-23 23:16:19 +0000461 SuperClassDecl = 0;
462 }
Steve Naroff818cb9e2009-02-04 17:14:05 +0000463 }
Fariborz Jahanianfdee0892009-07-09 22:08:26 +0000464 IDecl->setSuperClass(SuperClassDecl);
465 IDecl->setSuperClassLoc(SuperLoc);
Douglas Gregor05c272f2011-12-15 22:34:59 +0000466 IDecl->setEndOfDefinitionLoc(SuperLoc);
Steve Naroff818cb9e2009-02-04 17:14:05 +0000467 }
Chris Lattner4d391482007-12-12 07:09:47 +0000468 } else { // we have a root class.
Douglas Gregor05c272f2011-12-15 22:34:59 +0000469 IDecl->setEndOfDefinitionLoc(ClassLoc);
Chris Lattner4d391482007-12-12 07:09:47 +0000470 }
Mike Stump1eb44332009-09-09 15:08:12 +0000471
Sebastian Redl0b17c612010-08-13 00:28:03 +0000472 // Check then save referenced protocols.
Chris Lattner06036d32008-07-26 04:13:19 +0000473 if (NumProtoRefs) {
Chris Lattner38af2de2009-02-20 21:35:13 +0000474 IDecl->setProtocolList((ObjCProtocolDecl**)ProtoRefs, NumProtoRefs,
Douglas Gregor18df52b2010-01-16 15:02:53 +0000475 ProtoLocs, Context);
Douglas Gregor05c272f2011-12-15 22:34:59 +0000476 IDecl->setEndOfDefinitionLoc(EndProtoLoc);
Chris Lattner4d391482007-12-12 07:09:47 +0000477 }
Mike Stump1eb44332009-09-09 15:08:12 +0000478
Anders Carlsson15281452008-11-04 16:57:32 +0000479 CheckObjCDeclScope(IDecl);
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000480 return ActOnObjCContainerStartDefinition(IDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000481}
482
483/// ActOnCompatiblityAlias - this action is called after complete parsing of
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +0000484/// @compatibility_alias declaration. It sets up the alias relationships.
John McCalld226f652010-08-21 09:40:31 +0000485Decl *Sema::ActOnCompatiblityAlias(SourceLocation AtLoc,
486 IdentifierInfo *AliasName,
487 SourceLocation AliasLocation,
488 IdentifierInfo *ClassName,
489 SourceLocation ClassLocation) {
Chris Lattner4d391482007-12-12 07:09:47 +0000490 // Look for previous declaration of alias name
Douglas Gregorc83c6872010-04-15 22:33:43 +0000491 NamedDecl *ADecl = LookupSingleName(TUScope, AliasName, AliasLocation,
Douglas Gregorc0b39642010-04-15 23:40:53 +0000492 LookupOrdinaryName, ForRedeclaration);
Chris Lattner4d391482007-12-12 07:09:47 +0000493 if (ADecl) {
Chris Lattner8b265bd2008-11-23 23:20:13 +0000494 if (isa<ObjCCompatibleAliasDecl>(ADecl))
Chris Lattner4d391482007-12-12 07:09:47 +0000495 Diag(AliasLocation, diag::warn_previous_alias_decl);
Chris Lattner8b265bd2008-11-23 23:20:13 +0000496 else
Chris Lattner3c73c412008-11-19 08:23:25 +0000497 Diag(AliasLocation, diag::err_conflicting_aliasing_type) << AliasName;
Chris Lattner8b265bd2008-11-23 23:20:13 +0000498 Diag(ADecl->getLocation(), diag::note_previous_declaration);
John McCalld226f652010-08-21 09:40:31 +0000499 return 0;
Chris Lattner4d391482007-12-12 07:09:47 +0000500 }
501 // Check for class declaration
Douglas Gregorc83c6872010-04-15 22:33:43 +0000502 NamedDecl *CDeclU = LookupSingleName(TUScope, ClassName, ClassLocation,
Douglas Gregorc0b39642010-04-15 23:40:53 +0000503 LookupOrdinaryName, ForRedeclaration);
Richard Smith162e1c12011-04-15 14:24:37 +0000504 if (const TypedefNameDecl *TDecl =
505 dyn_cast_or_null<TypedefNameDecl>(CDeclU)) {
Fariborz Jahanian305c6582009-01-08 01:10:55 +0000506 QualType T = TDecl->getUnderlyingType();
John McCallc12c5bb2010-05-15 11:32:37 +0000507 if (T->isObjCObjectType()) {
508 if (NamedDecl *IDecl = T->getAs<ObjCObjectType>()->getInterface()) {
Fariborz Jahanian305c6582009-01-08 01:10:55 +0000509 ClassName = IDecl->getIdentifier();
Douglas Gregorc83c6872010-04-15 22:33:43 +0000510 CDeclU = LookupSingleName(TUScope, ClassName, ClassLocation,
Douglas Gregorc0b39642010-04-15 23:40:53 +0000511 LookupOrdinaryName, ForRedeclaration);
Fariborz Jahanian305c6582009-01-08 01:10:55 +0000512 }
513 }
514 }
Chris Lattnerf8d17a52008-03-16 21:17:37 +0000515 ObjCInterfaceDecl *CDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDeclU);
516 if (CDecl == 0) {
Chris Lattner3c73c412008-11-19 08:23:25 +0000517 Diag(ClassLocation, diag::warn_undef_interface) << ClassName;
Chris Lattnerf8d17a52008-03-16 21:17:37 +0000518 if (CDeclU)
Chris Lattner8b265bd2008-11-23 23:20:13 +0000519 Diag(CDeclU->getLocation(), diag::note_previous_declaration);
John McCalld226f652010-08-21 09:40:31 +0000520 return 0;
Chris Lattner4d391482007-12-12 07:09:47 +0000521 }
Mike Stump1eb44332009-09-09 15:08:12 +0000522
Chris Lattnerf8d17a52008-03-16 21:17:37 +0000523 // Everything checked out, instantiate a new alias declaration AST.
Mike Stump1eb44332009-09-09 15:08:12 +0000524 ObjCCompatibleAliasDecl *AliasDecl =
Douglas Gregord0434102009-01-09 00:49:46 +0000525 ObjCCompatibleAliasDecl::Create(Context, CurContext, AtLoc, AliasName, CDecl);
Mike Stump1eb44332009-09-09 15:08:12 +0000526
Anders Carlsson15281452008-11-04 16:57:32 +0000527 if (!CheckObjCDeclScope(AliasDecl))
Douglas Gregor516ff432009-04-24 02:57:34 +0000528 PushOnScopeChains(AliasDecl, TUScope);
Douglas Gregord0434102009-01-09 00:49:46 +0000529
John McCalld226f652010-08-21 09:40:31 +0000530 return AliasDecl;
Chris Lattner4d391482007-12-12 07:09:47 +0000531}
532
Fariborz Jahanian819e9bf2011-05-13 18:02:08 +0000533bool Sema::CheckForwardProtocolDeclarationForCircularDependency(
Steve Naroff61d68522009-03-05 15:22:01 +0000534 IdentifierInfo *PName,
535 SourceLocation &Ploc, SourceLocation PrevLoc,
Fariborz Jahanian819e9bf2011-05-13 18:02:08 +0000536 const ObjCList<ObjCProtocolDecl> &PList) {
537
538 bool res = false;
Steve Naroff61d68522009-03-05 15:22:01 +0000539 for (ObjCList<ObjCProtocolDecl>::iterator I = PList.begin(),
540 E = PList.end(); I != E; ++I) {
Douglas Gregorc83c6872010-04-15 22:33:43 +0000541 if (ObjCProtocolDecl *PDecl = LookupProtocol((*I)->getIdentifier(),
542 Ploc)) {
Steve Naroff61d68522009-03-05 15:22:01 +0000543 if (PDecl->getIdentifier() == PName) {
544 Diag(Ploc, diag::err_protocol_has_circular_dependency);
545 Diag(PrevLoc, diag::note_previous_definition);
Fariborz Jahanian819e9bf2011-05-13 18:02:08 +0000546 res = true;
Steve Naroff61d68522009-03-05 15:22:01 +0000547 }
Fariborz Jahanian819e9bf2011-05-13 18:02:08 +0000548 if (CheckForwardProtocolDeclarationForCircularDependency(PName, Ploc,
549 PDecl->getLocation(), PDecl->getReferencedProtocols()))
550 res = true;
Steve Naroff61d68522009-03-05 15:22:01 +0000551 }
552 }
Fariborz Jahanian819e9bf2011-05-13 18:02:08 +0000553 return res;
Steve Naroff61d68522009-03-05 15:22:01 +0000554}
555
John McCalld226f652010-08-21 09:40:31 +0000556Decl *
Chris Lattnere13b9592008-07-26 04:03:38 +0000557Sema::ActOnStartProtocolInterface(SourceLocation AtProtoInterfaceLoc,
558 IdentifierInfo *ProtocolName,
559 SourceLocation ProtocolLoc,
John McCalld226f652010-08-21 09:40:31 +0000560 Decl * const *ProtoRefs,
Chris Lattnere13b9592008-07-26 04:03:38 +0000561 unsigned NumProtoRefs,
Douglas Gregor18df52b2010-01-16 15:02:53 +0000562 const SourceLocation *ProtoLocs,
Daniel Dunbar246e70f2008-09-26 04:48:09 +0000563 SourceLocation EndProtoLoc,
564 AttributeList *AttrList) {
Fariborz Jahanian96b69a72011-05-12 22:04:39 +0000565 bool err = false;
Daniel Dunbar246e70f2008-09-26 04:48:09 +0000566 // FIXME: Deal with AttrList.
Chris Lattner4d391482007-12-12 07:09:47 +0000567 assert(ProtocolName && "Missing protocol identifier");
Douglas Gregorc83c6872010-04-15 22:33:43 +0000568 ObjCProtocolDecl *PDecl = LookupProtocol(ProtocolName, ProtocolLoc);
Chris Lattner4d391482007-12-12 07:09:47 +0000569 if (PDecl) {
570 // Protocol already seen. Better be a forward protocol declaration
Chris Lattner439e71f2008-03-16 01:25:17 +0000571 if (!PDecl->isForwardDecl()) {
Fariborz Jahaniane2573e52009-04-06 23:43:32 +0000572 Diag(ProtocolLoc, diag::warn_duplicate_protocol_def) << ProtocolName;
Chris Lattnerb8b96af2008-11-23 22:46:27 +0000573 Diag(PDecl->getLocation(), diag::note_previous_definition);
Mike Stump1eb44332009-09-09 15:08:12 +0000574
Argyrios Kyrtzidis4fc04da2011-11-13 22:08:30 +0000575 // Create a new one; the other may be in a different DeclContex, (e.g.
576 // this one may be in a LinkageSpecDecl while the other is not) which
577 // will break invariants.
578 // We will not add it to scope chains to ignore it as the warning says.
579 PDecl = ObjCProtocolDecl::Create(Context, CurContext, ProtocolName,
580 ProtocolLoc, AtProtoInterfaceLoc,
581 /*isForwardDecl=*/false);
582
583 } else {
584 ObjCList<ObjCProtocolDecl> PList;
585 PList.set((ObjCProtocolDecl *const*)ProtoRefs, NumProtoRefs, Context);
586 err = CheckForwardProtocolDeclarationForCircularDependency(
587 ProtocolName, ProtocolLoc, PDecl->getLocation(), PList);
588
589 // Make sure the cached decl gets a valid start location.
590 PDecl->setAtStartLoc(AtProtoInterfaceLoc);
591 PDecl->setLocation(ProtocolLoc);
592 // Since this ObjCProtocolDecl was created by a forward declaration,
593 // we now add it to the DeclContext since it wasn't added before
594 PDecl->setLexicalDeclContext(CurContext);
595 CurContext->addDecl(PDecl);
596 PDecl->completedForwardDecl();
597 }
Chris Lattner439e71f2008-03-16 01:25:17 +0000598 } else {
Argyrios Kyrtzidis1711fc92011-10-04 04:48:02 +0000599 PDecl = ObjCProtocolDecl::Create(Context, CurContext, ProtocolName,
Argyrios Kyrtzidisb05d7b22011-10-17 19:48:06 +0000600 ProtocolLoc, AtProtoInterfaceLoc,
601 /*isForwardDecl=*/false);
Douglas Gregor6e378de2009-04-23 23:18:26 +0000602 PushOnScopeChains(PDecl, TUScope);
Chris Lattnercca59d72008-03-16 01:23:04 +0000603 }
Fariborz Jahanianbc1c8772008-12-17 01:07:27 +0000604 if (AttrList)
Douglas Gregor9cdda0c2009-06-17 21:51:59 +0000605 ProcessDeclAttributeList(TUScope, PDecl, AttrList);
Fariborz Jahanian96b69a72011-05-12 22:04:39 +0000606 if (!err && NumProtoRefs ) {
Chris Lattnerc8581052008-03-16 20:19:15 +0000607 /// Check then save referenced protocols.
Douglas Gregor18df52b2010-01-16 15:02:53 +0000608 PDecl->setProtocolList((ObjCProtocolDecl**)ProtoRefs, NumProtoRefs,
609 ProtoLocs, Context);
Chris Lattner4d391482007-12-12 07:09:47 +0000610 PDecl->setLocEnd(EndProtoLoc);
611 }
Mike Stump1eb44332009-09-09 15:08:12 +0000612
613 CheckObjCDeclScope(PDecl);
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000614 return ActOnObjCContainerStartDefinition(PDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000615}
616
617/// FindProtocolDeclaration - This routine looks up protocols and
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +0000618/// issues an error if they are not declared. It returns list of
619/// protocol declarations in its 'Protocols' argument.
Chris Lattner4d391482007-12-12 07:09:47 +0000620void
Chris Lattnere13b9592008-07-26 04:03:38 +0000621Sema::FindProtocolDeclaration(bool WarnOnDeclarations,
Chris Lattner7caeabd2008-07-21 22:17:28 +0000622 const IdentifierLocPair *ProtocolId,
Chris Lattner4d391482007-12-12 07:09:47 +0000623 unsigned NumProtocols,
Chris Lattner5f9e2722011-07-23 10:55:15 +0000624 SmallVectorImpl<Decl *> &Protocols) {
Chris Lattner4d391482007-12-12 07:09:47 +0000625 for (unsigned i = 0; i != NumProtocols; ++i) {
Douglas Gregorc83c6872010-04-15 22:33:43 +0000626 ObjCProtocolDecl *PDecl = LookupProtocol(ProtocolId[i].first,
627 ProtocolId[i].second);
Chris Lattnereacc3922008-07-26 03:47:43 +0000628 if (!PDecl) {
Douglas Gregord8bba9c2011-06-28 16:20:02 +0000629 TypoCorrection Corrected = CorrectTypo(
630 DeclarationNameInfo(ProtocolId[i].first, ProtocolId[i].second),
631 LookupObjCProtocolName, TUScope, NULL, NULL, false, CTC_NoKeywords);
632 if ((PDecl = Corrected.getCorrectionDeclAs<ObjCProtocolDecl>())) {
Douglas Gregorf06cdae2010-01-03 18:01:57 +0000633 Diag(ProtocolId[i].second, diag::err_undeclared_protocol_suggest)
Douglas Gregord8bba9c2011-06-28 16:20:02 +0000634 << ProtocolId[i].first << Corrected.getCorrection();
Douglas Gregor67dd1d42010-01-07 00:17:44 +0000635 Diag(PDecl->getLocation(), diag::note_previous_decl)
636 << PDecl->getDeclName();
Douglas Gregorf06cdae2010-01-03 18:01:57 +0000637 }
638 }
639
640 if (!PDecl) {
Chris Lattnerfa25bbb2008-11-19 05:08:23 +0000641 Diag(ProtocolId[i].second, diag::err_undeclared_protocol)
Chris Lattner3c73c412008-11-19 08:23:25 +0000642 << ProtocolId[i].first;
Chris Lattnereacc3922008-07-26 03:47:43 +0000643 continue;
644 }
Mike Stump1eb44332009-09-09 15:08:12 +0000645
Douglas Gregor48f3bb92009-02-18 21:56:37 +0000646 (void)DiagnoseUseOfDecl(PDecl, ProtocolId[i].second);
Chris Lattnereacc3922008-07-26 03:47:43 +0000647
648 // If this is a forward declaration and we are supposed to warn in this
649 // case, do it.
650 if (WarnOnDeclarations && PDecl->isForwardDecl())
Chris Lattnerfa25bbb2008-11-19 05:08:23 +0000651 Diag(ProtocolId[i].second, diag::warn_undef_protocolref)
Chris Lattner3c73c412008-11-19 08:23:25 +0000652 << ProtocolId[i].first;
John McCalld226f652010-08-21 09:40:31 +0000653 Protocols.push_back(PDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000654 }
655}
656
Fariborz Jahanian78c39c72009-03-02 19:06:08 +0000657/// DiagnoseClassExtensionDupMethods - Check for duplicate declaration of
Fariborz Jahanianb7f95f52009-03-02 19:05:07 +0000658/// a class method in its extension.
659///
Mike Stump1eb44332009-09-09 15:08:12 +0000660void Sema::DiagnoseClassExtensionDupMethods(ObjCCategoryDecl *CAT,
Fariborz Jahanianb7f95f52009-03-02 19:05:07 +0000661 ObjCInterfaceDecl *ID) {
662 if (!ID)
663 return; // Possibly due to previous error
664
665 llvm::DenseMap<Selector, const ObjCMethodDecl*> MethodMap;
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +0000666 for (ObjCInterfaceDecl::method_iterator i = ID->meth_begin(),
667 e = ID->meth_end(); i != e; ++i) {
Fariborz Jahanianb7f95f52009-03-02 19:05:07 +0000668 ObjCMethodDecl *MD = *i;
669 MethodMap[MD->getSelector()] = MD;
670 }
671
672 if (MethodMap.empty())
673 return;
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +0000674 for (ObjCCategoryDecl::method_iterator i = CAT->meth_begin(),
675 e = CAT->meth_end(); i != e; ++i) {
Fariborz Jahanianb7f95f52009-03-02 19:05:07 +0000676 ObjCMethodDecl *Method = *i;
677 const ObjCMethodDecl *&PrevMethod = MethodMap[Method->getSelector()];
678 if (PrevMethod && !MatchTwoMethodDeclarations(Method, PrevMethod)) {
679 Diag(Method->getLocation(), diag::err_duplicate_method_decl)
680 << Method->getDeclName();
681 Diag(PrevMethod->getLocation(), diag::note_previous_declaration);
682 }
683 }
684}
685
Chris Lattner58fe03b2009-04-12 08:43:13 +0000686/// ActOnForwardProtocolDeclaration - Handle @protocol foo;
John McCalld226f652010-08-21 09:40:31 +0000687Decl *
Chris Lattner4d391482007-12-12 07:09:47 +0000688Sema::ActOnForwardProtocolDeclaration(SourceLocation AtProtocolLoc,
Chris Lattner7caeabd2008-07-21 22:17:28 +0000689 const IdentifierLocPair *IdentList,
Fariborz Jahanianbc1c8772008-12-17 01:07:27 +0000690 unsigned NumElts,
691 AttributeList *attrList) {
Chris Lattner5f9e2722011-07-23 10:55:15 +0000692 SmallVector<ObjCProtocolDecl*, 32> Protocols;
693 SmallVector<SourceLocation, 8> ProtoLocs;
Mike Stump1eb44332009-09-09 15:08:12 +0000694
Chris Lattner4d391482007-12-12 07:09:47 +0000695 for (unsigned i = 0; i != NumElts; ++i) {
Chris Lattner7caeabd2008-07-21 22:17:28 +0000696 IdentifierInfo *Ident = IdentList[i].first;
Douglas Gregorc83c6872010-04-15 22:33:43 +0000697 ObjCProtocolDecl *PDecl = LookupProtocol(Ident, IdentList[i].second);
Sebastian Redl0b17c612010-08-13 00:28:03 +0000698 bool isNew = false;
Douglas Gregord0434102009-01-09 00:49:46 +0000699 if (PDecl == 0) { // Not already seen?
Argyrios Kyrtzidis1711fc92011-10-04 04:48:02 +0000700 PDecl = ObjCProtocolDecl::Create(Context, CurContext, Ident,
Argyrios Kyrtzidisb05d7b22011-10-17 19:48:06 +0000701 IdentList[i].second, AtProtocolLoc,
702 /*isForwardDecl=*/true);
Sebastian Redl0b17c612010-08-13 00:28:03 +0000703 PushOnScopeChains(PDecl, TUScope, false);
704 isNew = true;
Douglas Gregord0434102009-01-09 00:49:46 +0000705 }
Sebastian Redl0b17c612010-08-13 00:28:03 +0000706 if (attrList) {
Douglas Gregor9cdda0c2009-06-17 21:51:59 +0000707 ProcessDeclAttributeList(TUScope, PDecl, attrList);
Argyrios Kyrtzidis1a434152011-11-12 21:07:52 +0000708 if (!isNew) {
709 if (ASTMutationListener *L = Context.getASTMutationListener())
710 L->UpdatedAttributeList(PDecl);
711 }
Sebastian Redl0b17c612010-08-13 00:28:03 +0000712 }
Chris Lattner4d391482007-12-12 07:09:47 +0000713 Protocols.push_back(PDecl);
Douglas Gregor18df52b2010-01-16 15:02:53 +0000714 ProtoLocs.push_back(IdentList[i].second);
Chris Lattner4d391482007-12-12 07:09:47 +0000715 }
Mike Stump1eb44332009-09-09 15:08:12 +0000716
717 ObjCForwardProtocolDecl *PDecl =
Douglas Gregord0434102009-01-09 00:49:46 +0000718 ObjCForwardProtocolDecl::Create(Context, CurContext, AtProtocolLoc,
Douglas Gregor18df52b2010-01-16 15:02:53 +0000719 Protocols.data(), Protocols.size(),
720 ProtoLocs.data());
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +0000721 CurContext->addDecl(PDecl);
Anders Carlsson15281452008-11-04 16:57:32 +0000722 CheckObjCDeclScope(PDecl);
John McCalld226f652010-08-21 09:40:31 +0000723 return PDecl;
Chris Lattner4d391482007-12-12 07:09:47 +0000724}
725
John McCalld226f652010-08-21 09:40:31 +0000726Decl *Sema::
Chris Lattner7caeabd2008-07-21 22:17:28 +0000727ActOnStartCategoryInterface(SourceLocation AtInterfaceLoc,
728 IdentifierInfo *ClassName, SourceLocation ClassLoc,
729 IdentifierInfo *CategoryName,
730 SourceLocation CategoryLoc,
John McCalld226f652010-08-21 09:40:31 +0000731 Decl * const *ProtoRefs,
Chris Lattner7caeabd2008-07-21 22:17:28 +0000732 unsigned NumProtoRefs,
Douglas Gregor18df52b2010-01-16 15:02:53 +0000733 const SourceLocation *ProtoLocs,
Chris Lattner7caeabd2008-07-21 22:17:28 +0000734 SourceLocation EndProtoLoc) {
Fariborz Jahanian80aa1cd2010-06-22 23:20:40 +0000735 ObjCCategoryDecl *CDecl;
Douglas Gregorc83c6872010-04-15 22:33:43 +0000736 ObjCInterfaceDecl *IDecl = getObjCInterfaceDecl(ClassName, ClassLoc, true);
Ted Kremenek09b68972010-02-23 19:39:46 +0000737
738 /// Check that class of this category is already completely declared.
Douglas Gregorb3029962011-11-14 22:10:01 +0000739
740 if (!IDecl
741 || RequireCompleteType(ClassLoc, Context.getObjCInterfaceType(IDecl),
742 PDiag(diag::err_category_forward_interface)
743 << (CategoryName == 0))) {
Ted Kremenek09b68972010-02-23 19:39:46 +0000744 // Create an invalid ObjCCategoryDecl to serve as context for
745 // the enclosing method declarations. We mark the decl invalid
746 // to make it clear that this isn't a valid AST.
747 CDecl = ObjCCategoryDecl::Create(Context, CurContext, AtInterfaceLoc,
Argyrios Kyrtzidis955fadb2011-08-30 19:43:26 +0000748 ClassLoc, CategoryLoc, CategoryName,IDecl);
Ted Kremenek09b68972010-02-23 19:39:46 +0000749 CDecl->setInvalidDecl();
Douglas Gregorb3029962011-11-14 22:10:01 +0000750
751 if (!IDecl)
752 Diag(ClassLoc, diag::err_undef_interface) << ClassName;
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000753 return ActOnObjCContainerStartDefinition(CDecl);
Ted Kremenek09b68972010-02-23 19:39:46 +0000754 }
755
Fariborz Jahanian80aa1cd2010-06-22 23:20:40 +0000756 if (!CategoryName && IDecl->getImplementation()) {
757 Diag(ClassLoc, diag::err_class_extension_after_impl) << ClassName;
758 Diag(IDecl->getImplementation()->getLocation(),
759 diag::note_implementation_declared);
Ted Kremenek09b68972010-02-23 19:39:46 +0000760 }
761
Fariborz Jahanian25760612010-02-15 21:55:26 +0000762 if (CategoryName) {
763 /// Check for duplicate interface declaration for this category
764 ObjCCategoryDecl *CDeclChain;
765 for (CDeclChain = IDecl->getCategoryList(); CDeclChain;
766 CDeclChain = CDeclChain->getNextClassCategory()) {
767 if (CDeclChain->getIdentifier() == CategoryName) {
768 // Class extensions can be declared multiple times.
769 Diag(CategoryLoc, diag::warn_dup_category_def)
770 << ClassName << CategoryName;
771 Diag(CDeclChain->getLocation(), diag::note_previous_definition);
772 break;
773 }
Chris Lattner70f19542009-02-16 21:26:43 +0000774 }
775 }
Chris Lattner70f19542009-02-16 21:26:43 +0000776
Argyrios Kyrtzidis955fadb2011-08-30 19:43:26 +0000777 CDecl = ObjCCategoryDecl::Create(Context, CurContext, AtInterfaceLoc,
778 ClassLoc, CategoryLoc, CategoryName, IDecl);
779 // FIXME: PushOnScopeChains?
780 CurContext->addDecl(CDecl);
781
Chris Lattner4d391482007-12-12 07:09:47 +0000782 if (NumProtoRefs) {
Fariborz Jahanianb106fc62009-10-05 21:32:49 +0000783 CDecl->setProtocolList((ObjCProtocolDecl**)ProtoRefs, NumProtoRefs,
Douglas Gregor18df52b2010-01-16 15:02:53 +0000784 ProtoLocs, Context);
Fariborz Jahanian339798e2009-10-05 20:41:32 +0000785 // Protocols in the class extension belong to the class.
Fariborz Jahanian25760612010-02-15 21:55:26 +0000786 if (CDecl->IsClassExtension())
Fariborz Jahanian339798e2009-10-05 20:41:32 +0000787 IDecl->mergeClassExtensionProtocolList((ObjCProtocolDecl**)ProtoRefs,
Ted Kremenek53b94412010-09-01 01:21:15 +0000788 NumProtoRefs, Context);
Chris Lattner4d391482007-12-12 07:09:47 +0000789 }
Mike Stump1eb44332009-09-09 15:08:12 +0000790
Anders Carlsson15281452008-11-04 16:57:32 +0000791 CheckObjCDeclScope(CDecl);
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000792 return ActOnObjCContainerStartDefinition(CDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000793}
794
795/// ActOnStartCategoryImplementation - Perform semantic checks on the
Ted Kremeneka526c5c2008-01-07 19:49:32 +0000796/// category implementation declaration and build an ObjCCategoryImplDecl
Chris Lattner4d391482007-12-12 07:09:47 +0000797/// object.
John McCalld226f652010-08-21 09:40:31 +0000798Decl *Sema::ActOnStartCategoryImplementation(
Chris Lattner4d391482007-12-12 07:09:47 +0000799 SourceLocation AtCatImplLoc,
800 IdentifierInfo *ClassName, SourceLocation ClassLoc,
801 IdentifierInfo *CatName, SourceLocation CatLoc) {
Douglas Gregorc83c6872010-04-15 22:33:43 +0000802 ObjCInterfaceDecl *IDecl = getObjCInterfaceDecl(ClassName, ClassLoc, true);
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000803 ObjCCategoryDecl *CatIDecl = 0;
804 if (IDecl) {
805 CatIDecl = IDecl->FindCategoryDeclaration(CatName);
806 if (!CatIDecl) {
807 // Category @implementation with no corresponding @interface.
808 // Create and install one.
Argyrios Kyrtzidis37f40572011-11-23 20:27:26 +0000809 CatIDecl = ObjCCategoryDecl::Create(Context, CurContext, AtCatImplLoc,
810 ClassLoc, CatLoc,
Argyrios Kyrtzidis955fadb2011-08-30 19:43:26 +0000811 CatName, IDecl);
Argyrios Kyrtzidis37f40572011-11-23 20:27:26 +0000812 CatIDecl->setImplicit();
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000813 }
814 }
815
Mike Stump1eb44332009-09-09 15:08:12 +0000816 ObjCCategoryImplDecl *CDecl =
Argyrios Kyrtzidis1711fc92011-10-04 04:48:02 +0000817 ObjCCategoryImplDecl::Create(Context, CurContext, CatName, IDecl,
Argyrios Kyrtzidisc6994002011-12-09 00:31:40 +0000818 ClassLoc, AtCatImplLoc, CatLoc);
Chris Lattner4d391482007-12-12 07:09:47 +0000819 /// Check that class of this category is already completely declared.
Douglas Gregorb3029962011-11-14 22:10:01 +0000820 if (!IDecl) {
Chris Lattner3c73c412008-11-19 08:23:25 +0000821 Diag(ClassLoc, diag::err_undef_interface) << ClassName;
John McCall6c2c2502011-07-22 02:45:48 +0000822 CDecl->setInvalidDecl();
Douglas Gregorb3029962011-11-14 22:10:01 +0000823 } else if (RequireCompleteType(ClassLoc, Context.getObjCInterfaceType(IDecl),
824 diag::err_undef_interface)) {
825 CDecl->setInvalidDecl();
John McCall6c2c2502011-07-22 02:45:48 +0000826 }
Chris Lattner4d391482007-12-12 07:09:47 +0000827
Douglas Gregord0434102009-01-09 00:49:46 +0000828 // FIXME: PushOnScopeChains?
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +0000829 CurContext->addDecl(CDecl);
Douglas Gregord0434102009-01-09 00:49:46 +0000830
Argyrios Kyrtzidisc076e372011-10-06 23:23:27 +0000831 // If the interface is deprecated/unavailable, warn/error about it.
832 if (IDecl)
833 DiagnoseUseOfDecl(IDecl, ClassLoc);
834
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000835 /// Check that CatName, category name, is not used in another implementation.
836 if (CatIDecl) {
837 if (CatIDecl->getImplementation()) {
838 Diag(ClassLoc, diag::err_dup_implementation_category) << ClassName
839 << CatName;
840 Diag(CatIDecl->getImplementation()->getLocation(),
841 diag::note_previous_definition);
Fariborz Jahanianb1224f62011-02-15 00:59:30 +0000842 } else {
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000843 CatIDecl->setImplementation(CDecl);
Fariborz Jahanianb1224f62011-02-15 00:59:30 +0000844 // Warn on implementating category of deprecated class under
845 // -Wdeprecated-implementations flag.
Fariborz Jahanian5ac96d52011-02-15 17:49:58 +0000846 DiagnoseObjCImplementedDeprecations(*this,
847 dyn_cast<NamedDecl>(IDecl),
848 CDecl->getLocation(), 2);
Fariborz Jahanianb1224f62011-02-15 00:59:30 +0000849 }
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000850 }
Mike Stump1eb44332009-09-09 15:08:12 +0000851
Anders Carlsson15281452008-11-04 16:57:32 +0000852 CheckObjCDeclScope(CDecl);
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000853 return ActOnObjCContainerStartDefinition(CDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000854}
855
John McCalld226f652010-08-21 09:40:31 +0000856Decl *Sema::ActOnStartClassImplementation(
Chris Lattner4d391482007-12-12 07:09:47 +0000857 SourceLocation AtClassImplLoc,
858 IdentifierInfo *ClassName, SourceLocation ClassLoc,
Mike Stump1eb44332009-09-09 15:08:12 +0000859 IdentifierInfo *SuperClassname,
Chris Lattner4d391482007-12-12 07:09:47 +0000860 SourceLocation SuperClassLoc) {
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000861 ObjCInterfaceDecl* IDecl = 0;
Chris Lattner4d391482007-12-12 07:09:47 +0000862 // Check for another declaration kind with the same name.
John McCallf36e02d2009-10-09 21:13:30 +0000863 NamedDecl *PrevDecl
Douglas Gregorc0b39642010-04-15 23:40:53 +0000864 = LookupSingleName(TUScope, ClassName, ClassLoc, LookupOrdinaryName,
865 ForRedeclaration);
Ted Kremeneka526c5c2008-01-07 19:49:32 +0000866 if (PrevDecl && !isa<ObjCInterfaceDecl>(PrevDecl)) {
Chris Lattner3c73c412008-11-19 08:23:25 +0000867 Diag(ClassLoc, diag::err_redefinition_different_kind) << ClassName;
Chris Lattner5f4a6822008-11-23 23:12:31 +0000868 Diag(PrevDecl->getLocation(), diag::note_previous_definition);
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000869 } else if ((IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(PrevDecl))) {
Douglas Gregor0af55012011-12-16 03:12:41 +0000870 RequireCompleteType(ClassLoc, Context.getObjCInterfaceType(IDecl),
871 diag::warn_undef_interface);
Douglas Gregor95ff7422010-01-04 17:27:12 +0000872 } else {
873 // We did not find anything with the name ClassName; try to correct for
874 // typos in the class name.
Douglas Gregord8bba9c2011-06-28 16:20:02 +0000875 TypoCorrection Corrected = CorrectTypo(
876 DeclarationNameInfo(ClassName, ClassLoc), LookupOrdinaryName, TUScope,
877 NULL, NULL, false, CTC_NoKeywords);
878 if ((IDecl = Corrected.getCorrectionDeclAs<ObjCInterfaceDecl>())) {
Douglas Gregora6f26382010-01-06 23:44:25 +0000879 // Suggest the (potentially) correct interface name. However, put the
880 // fix-it hint itself in a separate note, since changing the name in
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000881 // the warning would make the fix-it change semantics.However, don't
Douglas Gregor95ff7422010-01-04 17:27:12 +0000882 // provide a code-modification hint or use the typo name for recovery,
883 // because this is just a warning. The program may actually be correct.
Douglas Gregord8bba9c2011-06-28 16:20:02 +0000884 DeclarationName CorrectedName = Corrected.getCorrection();
Douglas Gregor95ff7422010-01-04 17:27:12 +0000885 Diag(ClassLoc, diag::warn_undef_interface_suggest)
Douglas Gregord8bba9c2011-06-28 16:20:02 +0000886 << ClassName << CorrectedName;
887 Diag(IDecl->getLocation(), diag::note_previous_decl) << CorrectedName
888 << FixItHint::CreateReplacement(ClassLoc, CorrectedName.getAsString());
Douglas Gregor95ff7422010-01-04 17:27:12 +0000889 IDecl = 0;
890 } else {
891 Diag(ClassLoc, diag::warn_undef_interface) << ClassName;
892 }
Chris Lattner4d391482007-12-12 07:09:47 +0000893 }
Mike Stump1eb44332009-09-09 15:08:12 +0000894
Chris Lattner4d391482007-12-12 07:09:47 +0000895 // Check that super class name is valid class name
Ted Kremeneka526c5c2008-01-07 19:49:32 +0000896 ObjCInterfaceDecl* SDecl = 0;
Chris Lattner4d391482007-12-12 07:09:47 +0000897 if (SuperClassname) {
898 // Check if a different kind of symbol declared in this scope.
Douglas Gregorc83c6872010-04-15 22:33:43 +0000899 PrevDecl = LookupSingleName(TUScope, SuperClassname, SuperClassLoc,
900 LookupOrdinaryName);
Ted Kremeneka526c5c2008-01-07 19:49:32 +0000901 if (PrevDecl && !isa<ObjCInterfaceDecl>(PrevDecl)) {
Chris Lattner3c73c412008-11-19 08:23:25 +0000902 Diag(SuperClassLoc, diag::err_redefinition_different_kind)
903 << SuperClassname;
Chris Lattner5f4a6822008-11-23 23:12:31 +0000904 Diag(PrevDecl->getLocation(), diag::note_previous_definition);
Chris Lattner3c73c412008-11-19 08:23:25 +0000905 } else {
Mike Stump1eb44332009-09-09 15:08:12 +0000906 SDecl = dyn_cast_or_null<ObjCInterfaceDecl>(PrevDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000907 if (!SDecl)
Chris Lattner3c73c412008-11-19 08:23:25 +0000908 Diag(SuperClassLoc, diag::err_undef_superclass)
909 << SuperClassname << ClassName;
Douglas Gregor60ef3082011-12-15 00:29:59 +0000910 else if (IDecl && !declaresSameEntity(IDecl->getSuperClass(), SDecl)) {
Chris Lattner4d391482007-12-12 07:09:47 +0000911 // This implementation and its interface do not have the same
912 // super class.
Chris Lattner3c73c412008-11-19 08:23:25 +0000913 Diag(SuperClassLoc, diag::err_conflicting_super_class)
Chris Lattner08631c52008-11-23 21:45:46 +0000914 << SDecl->getDeclName();
Chris Lattner5f4a6822008-11-23 23:12:31 +0000915 Diag(SDecl->getLocation(), diag::note_previous_definition);
Chris Lattner4d391482007-12-12 07:09:47 +0000916 }
917 }
918 }
Mike Stump1eb44332009-09-09 15:08:12 +0000919
Chris Lattner4d391482007-12-12 07:09:47 +0000920 if (!IDecl) {
921 // Legacy case of @implementation with no corresponding @interface.
922 // Build, chain & install the interface decl into the identifier.
Daniel Dunbarf6414922008-08-20 18:02:42 +0000923
Mike Stump390b4cc2009-05-16 07:39:55 +0000924 // FIXME: Do we support attributes on the @implementation? If so we should
925 // copy them over.
Mike Stump1eb44332009-09-09 15:08:12 +0000926 IDecl = ObjCInterfaceDecl::Create(Context, CurContext, AtClassImplLoc,
Douglas Gregor0af55012011-12-16 03:12:41 +0000927 ClassName, /*PrevDecl=*/0, ClassLoc,
928 true);
Douglas Gregor2e5c15b2011-12-15 05:27:12 +0000929 IDecl->startDefinition();
Douglas Gregor05c272f2011-12-15 22:34:59 +0000930 if (SDecl) {
931 IDecl->setSuperClass(SDecl);
932 IDecl->setSuperClassLoc(SuperClassLoc);
933 IDecl->setEndOfDefinitionLoc(SuperClassLoc);
934 } else {
935 IDecl->setEndOfDefinitionLoc(ClassLoc);
936 }
937
Douglas Gregor8b9fb302009-04-24 00:16:12 +0000938 PushOnScopeChains(IDecl, TUScope);
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000939 } else {
940 // Mark the interface as being completed, even if it was just as
941 // @class ....;
942 // declaration; the user cannot reopen it.
Douglas Gregor2e5c15b2011-12-15 05:27:12 +0000943 if (!IDecl->hasDefinition())
944 IDecl->startDefinition();
Chris Lattner4d391482007-12-12 07:09:47 +0000945 }
Mike Stump1eb44332009-09-09 15:08:12 +0000946
947 ObjCImplementationDecl* IMPDecl =
Argyrios Kyrtzidis1711fc92011-10-04 04:48:02 +0000948 ObjCImplementationDecl::Create(Context, CurContext, IDecl, SDecl,
949 ClassLoc, AtClassImplLoc);
Mike Stump1eb44332009-09-09 15:08:12 +0000950
Anders Carlsson15281452008-11-04 16:57:32 +0000951 if (CheckObjCDeclScope(IMPDecl))
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000952 return ActOnObjCContainerStartDefinition(IMPDecl);
Mike Stump1eb44332009-09-09 15:08:12 +0000953
Chris Lattner4d391482007-12-12 07:09:47 +0000954 // Check that there is no duplicate implementation of this class.
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000955 if (IDecl->getImplementation()) {
956 // FIXME: Don't leak everything!
Chris Lattner3c73c412008-11-19 08:23:25 +0000957 Diag(ClassLoc, diag::err_dup_implementation_class) << ClassName;
Argyrios Kyrtzidis87018772009-07-21 00:06:04 +0000958 Diag(IDecl->getImplementation()->getLocation(),
959 diag::note_previous_definition);
Douglas Gregordeacbdc2010-08-11 12:19:30 +0000960 } else { // add it to the list.
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000961 IDecl->setImplementation(IMPDecl);
Douglas Gregor8fc463a2009-04-24 00:11:27 +0000962 PushOnScopeChains(IMPDecl, TUScope);
Fariborz Jahanianb1224f62011-02-15 00:59:30 +0000963 // Warn on implementating deprecated class under
964 // -Wdeprecated-implementations flag.
Fariborz Jahanian5ac96d52011-02-15 17:49:58 +0000965 DiagnoseObjCImplementedDeprecations(*this,
966 dyn_cast<NamedDecl>(IDecl),
967 IMPDecl->getLocation(), 1);
Argyrios Kyrtzidis8a1d7222009-07-21 00:05:53 +0000968 }
Argyrios Kyrtzidis3a387442011-10-06 23:23:20 +0000969 return ActOnObjCContainerStartDefinition(IMPDecl);
Chris Lattner4d391482007-12-12 07:09:47 +0000970}
971
Ted Kremeneka526c5c2008-01-07 19:49:32 +0000972void Sema::CheckImplementationIvars(ObjCImplementationDecl *ImpDecl,
973 ObjCIvarDecl **ivars, unsigned numIvars,
Chris Lattner4d391482007-12-12 07:09:47 +0000974 SourceLocation RBrace) {
975 assert(ImpDecl && "missing implementation decl");
Douglas Gregor4afa39d2009-01-20 01:17:11 +0000976 ObjCInterfaceDecl* IDecl = ImpDecl->getClassInterface();
Chris Lattner4d391482007-12-12 07:09:47 +0000977 if (!IDecl)
978 return;
979 /// Check case of non-existing @interface decl.
980 /// (legacy objective-c @implementation decl without an @interface decl).
981 /// Add implementations's ivar to the synthesize class's ivar list.
Steve Naroff33feeb02009-04-20 20:09:33 +0000982 if (IDecl->isImplicitInterfaceDecl()) {
Douglas Gregor05c272f2011-12-15 22:34:59 +0000983 IDecl->setEndOfDefinitionLoc(RBrace);
Fariborz Jahanian3a21cd92010-02-17 17:00:07 +0000984 // Add ivar's to class's DeclContext.
985 for (unsigned i = 0, e = numIvars; i != e; ++i) {
Fariborz Jahanian2f14c4d2010-02-17 18:10:54 +0000986 ivars[i]->setLexicalDeclContext(ImpDecl);
987 IDecl->makeDeclVisibleInContext(ivars[i], false);
Fariborz Jahanian11062e12010-02-19 00:31:17 +0000988 ImpDecl->addDecl(ivars[i]);
Fariborz Jahanian3a21cd92010-02-17 17:00:07 +0000989 }
990
Chris Lattner4d391482007-12-12 07:09:47 +0000991 return;
992 }
993 // If implementation has empty ivar list, just return.
994 if (numIvars == 0)
995 return;
Mike Stump1eb44332009-09-09 15:08:12 +0000996
Chris Lattner4d391482007-12-12 07:09:47 +0000997 assert(ivars && "missing @implementation ivars");
Fariborz Jahanianbd94d442010-02-19 20:58:54 +0000998 if (LangOpts.ObjCNonFragileABI2) {
999 if (ImpDecl->getSuperClass())
1000 Diag(ImpDecl->getLocation(), diag::warn_on_superclass_use);
1001 for (unsigned i = 0; i < numIvars; i++) {
1002 ObjCIvarDecl* ImplIvar = ivars[i];
1003 if (const ObjCIvarDecl *ClsIvar =
1004 IDecl->getIvarDecl(ImplIvar->getIdentifier())) {
1005 Diag(ImplIvar->getLocation(), diag::err_duplicate_ivar_declaration);
1006 Diag(ClsIvar->getLocation(), diag::note_previous_definition);
1007 continue;
1008 }
Fariborz Jahanianbd94d442010-02-19 20:58:54 +00001009 // Instance ivar to Implementation's DeclContext.
1010 ImplIvar->setLexicalDeclContext(ImpDecl);
1011 IDecl->makeDeclVisibleInContext(ImplIvar, false);
1012 ImpDecl->addDecl(ImplIvar);
1013 }
1014 return;
1015 }
Chris Lattner4d391482007-12-12 07:09:47 +00001016 // Check interface's Ivar list against those in the implementation.
1017 // names and types must match.
1018 //
Chris Lattner4d391482007-12-12 07:09:47 +00001019 unsigned j = 0;
Mike Stump1eb44332009-09-09 15:08:12 +00001020 ObjCInterfaceDecl::ivar_iterator
Chris Lattner4c525092007-12-12 17:58:05 +00001021 IVI = IDecl->ivar_begin(), IVE = IDecl->ivar_end();
1022 for (; numIvars > 0 && IVI != IVE; ++IVI) {
Ted Kremeneka526c5c2008-01-07 19:49:32 +00001023 ObjCIvarDecl* ImplIvar = ivars[j++];
1024 ObjCIvarDecl* ClsIvar = *IVI;
Chris Lattner4d391482007-12-12 07:09:47 +00001025 assert (ImplIvar && "missing implementation ivar");
1026 assert (ClsIvar && "missing class ivar");
Mike Stump1eb44332009-09-09 15:08:12 +00001027
Steve Naroffca331292009-03-03 14:49:36 +00001028 // First, make sure the types match.
Richard Smitha6b8b2c2011-10-10 18:28:20 +00001029 if (!Context.hasSameType(ImplIvar->getType(), ClsIvar->getType())) {
Chris Lattnerfa25bbb2008-11-19 05:08:23 +00001030 Diag(ImplIvar->getLocation(), diag::err_conflicting_ivar_type)
Chris Lattner08631c52008-11-23 21:45:46 +00001031 << ImplIvar->getIdentifier()
1032 << ImplIvar->getType() << ClsIvar->getType();
Chris Lattner5f4a6822008-11-23 23:12:31 +00001033 Diag(ClsIvar->getLocation(), diag::note_previous_definition);
Richard Smitha6b8b2c2011-10-10 18:28:20 +00001034 } else if (ImplIvar->isBitField() && ClsIvar->isBitField() &&
1035 ImplIvar->getBitWidthValue(Context) !=
1036 ClsIvar->getBitWidthValue(Context)) {
1037 Diag(ImplIvar->getBitWidth()->getLocStart(),
1038 diag::err_conflicting_ivar_bitwidth) << ImplIvar->getIdentifier();
1039 Diag(ClsIvar->getBitWidth()->getLocStart(),
1040 diag::note_previous_definition);
Mike Stump1eb44332009-09-09 15:08:12 +00001041 }
Steve Naroffca331292009-03-03 14:49:36 +00001042 // Make sure the names are identical.
1043 if (ImplIvar->getIdentifier() != ClsIvar->getIdentifier()) {
Chris Lattnerfa25bbb2008-11-19 05:08:23 +00001044 Diag(ImplIvar->getLocation(), diag::err_conflicting_ivar_name)
Chris Lattner08631c52008-11-23 21:45:46 +00001045 << ImplIvar->getIdentifier() << ClsIvar->getIdentifier();
Chris Lattner5f4a6822008-11-23 23:12:31 +00001046 Diag(ClsIvar->getLocation(), diag::note_previous_definition);
Chris Lattner4d391482007-12-12 07:09:47 +00001047 }
1048 --numIvars;
Chris Lattner4d391482007-12-12 07:09:47 +00001049 }
Mike Stump1eb44332009-09-09 15:08:12 +00001050
Chris Lattner609e4c72007-12-12 18:11:49 +00001051 if (numIvars > 0)
Chris Lattner0e391052007-12-12 18:19:52 +00001052 Diag(ivars[j]->getLocation(), diag::err_inconsistant_ivar_count);
Chris Lattner609e4c72007-12-12 18:11:49 +00001053 else if (IVI != IVE)
Chris Lattner0e391052007-12-12 18:19:52 +00001054 Diag((*IVI)->getLocation(), diag::err_inconsistant_ivar_count);
Chris Lattner4d391482007-12-12 07:09:47 +00001055}
1056
Steve Naroff3c2eb662008-02-10 21:38:56 +00001057void Sema::WarnUndefinedMethod(SourceLocation ImpLoc, ObjCMethodDecl *method,
Fariborz Jahanian52146832010-03-31 18:23:33 +00001058 bool &IncompleteImpl, unsigned DiagID) {
Fariborz Jahanian327126e2011-06-24 20:31:37 +00001059 // No point warning no definition of method which is 'unavailable'.
1060 if (method->hasAttr<UnavailableAttr>())
1061 return;
Steve Naroff3c2eb662008-02-10 21:38:56 +00001062 if (!IncompleteImpl) {
1063 Diag(ImpLoc, diag::warn_incomplete_impl);
1064 IncompleteImpl = true;
1065 }
Fariborz Jahanian61c8d3e2010-10-29 23:20:05 +00001066 if (DiagID == diag::warn_unimplemented_protocol_method)
1067 Diag(ImpLoc, DiagID) << method->getDeclName();
1068 else
1069 Diag(method->getLocation(), DiagID) << method->getDeclName();
Steve Naroff3c2eb662008-02-10 21:38:56 +00001070}
1071
David Chisnalle8a2d4c2010-10-25 17:23:52 +00001072/// Determines if type B can be substituted for type A. Returns true if we can
1073/// guarantee that anything that the user will do to an object of type A can
1074/// also be done to an object of type B. This is trivially true if the two
1075/// types are the same, or if B is a subclass of A. It becomes more complex
1076/// in cases where protocols are involved.
1077///
1078/// Object types in Objective-C describe the minimum requirements for an
1079/// object, rather than providing a complete description of a type. For
1080/// example, if A is a subclass of B, then B* may refer to an instance of A.
1081/// The principle of substitutability means that we may use an instance of A
1082/// anywhere that we may use an instance of B - it will implement all of the
1083/// ivars of B and all of the methods of B.
1084///
1085/// This substitutability is important when type checking methods, because
1086/// the implementation may have stricter type definitions than the interface.
1087/// The interface specifies minimum requirements, but the implementation may
1088/// have more accurate ones. For example, a method may privately accept
1089/// instances of B, but only publish that it accepts instances of A. Any
1090/// object passed to it will be type checked against B, and so will implicitly
1091/// by a valid A*. Similarly, a method may return a subclass of the class that
1092/// it is declared as returning.
1093///
1094/// This is most important when considering subclassing. A method in a
1095/// subclass must accept any object as an argument that its superclass's
1096/// implementation accepts. It may, however, accept a more general type
1097/// without breaking substitutability (i.e. you can still use the subclass
1098/// anywhere that you can use the superclass, but not vice versa). The
1099/// converse requirement applies to return types: the return type for a
1100/// subclass method must be a valid object of the kind that the superclass
1101/// advertises, but it may be specified more accurately. This avoids the need
1102/// for explicit down-casting by callers.
1103///
1104/// Note: This is a stricter requirement than for assignment.
John McCall10302c02010-10-28 02:34:38 +00001105static bool isObjCTypeSubstitutable(ASTContext &Context,
1106 const ObjCObjectPointerType *A,
1107 const ObjCObjectPointerType *B,
1108 bool rejectId) {
1109 // Reject a protocol-unqualified id.
1110 if (rejectId && B->isObjCIdType()) return false;
David Chisnalle8a2d4c2010-10-25 17:23:52 +00001111
1112 // If B is a qualified id, then A must also be a qualified id and it must
1113 // implement all of the protocols in B. It may not be a qualified class.
1114 // For example, MyClass<A> can be assigned to id<A>, but MyClass<A> is a
1115 // stricter definition so it is not substitutable for id<A>.
1116 if (B->isObjCQualifiedIdType()) {
1117 return A->isObjCQualifiedIdType() &&
John McCall10302c02010-10-28 02:34:38 +00001118 Context.ObjCQualifiedIdTypesAreCompatible(QualType(A, 0),
1119 QualType(B,0),
1120 false);
David Chisnalle8a2d4c2010-10-25 17:23:52 +00001121 }
1122
1123 /*
1124 // id is a special type that bypasses type checking completely. We want a
1125 // warning when it is used in one place but not another.
1126 if (C.isObjCIdType(A) || C.isObjCIdType(B)) return false;
1127
1128
1129 // If B is a qualified id, then A must also be a qualified id (which it isn't
1130 // if we've got this far)
1131 if (B->isObjCQualifiedIdType()) return false;
1132 */
1133
1134 // Now we know that A and B are (potentially-qualified) class types. The
1135 // normal rules for assignment apply.
John McCall10302c02010-10-28 02:34:38 +00001136 return Context.canAssignObjCInterfaces(A, B);
David Chisnalle8a2d4c2010-10-25 17:23:52 +00001137}
1138
John McCall10302c02010-10-28 02:34:38 +00001139static SourceRange getTypeRange(TypeSourceInfo *TSI) {
1140 return (TSI ? TSI->getTypeLoc().getSourceRange() : SourceRange());
1141}
1142
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001143static bool CheckMethodOverrideReturn(Sema &S,
John McCall10302c02010-10-28 02:34:38 +00001144 ObjCMethodDecl *MethodImpl,
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001145 ObjCMethodDecl *MethodDecl,
Fariborz Jahanianeee3ef12011-07-24 20:53:26 +00001146 bool IsProtocolMethodDecl,
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001147 bool IsOverridingMode,
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001148 bool Warn) {
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001149 if (IsProtocolMethodDecl &&
1150 (MethodDecl->getObjCDeclQualifier() !=
1151 MethodImpl->getObjCDeclQualifier())) {
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001152 if (Warn) {
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001153 S.Diag(MethodImpl->getLocation(),
1154 (IsOverridingMode ?
1155 diag::warn_conflicting_overriding_ret_type_modifiers
1156 : diag::warn_conflicting_ret_type_modifiers))
1157 << MethodImpl->getDeclName()
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001158 << getTypeRange(MethodImpl->getResultTypeSourceInfo());
1159 S.Diag(MethodDecl->getLocation(), diag::note_previous_declaration)
1160 << getTypeRange(MethodDecl->getResultTypeSourceInfo());
1161 }
1162 else
1163 return false;
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001164 }
1165
John McCall10302c02010-10-28 02:34:38 +00001166 if (S.Context.hasSameUnqualifiedType(MethodImpl->getResultType(),
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001167 MethodDecl->getResultType()))
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001168 return true;
1169 if (!Warn)
1170 return false;
John McCall10302c02010-10-28 02:34:38 +00001171
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001172 unsigned DiagID =
1173 IsOverridingMode ? diag::warn_conflicting_overriding_ret_types
1174 : diag::warn_conflicting_ret_types;
John McCall10302c02010-10-28 02:34:38 +00001175
1176 // Mismatches between ObjC pointers go into a different warning
1177 // category, and sometimes they're even completely whitelisted.
1178 if (const ObjCObjectPointerType *ImplPtrTy =
1179 MethodImpl->getResultType()->getAs<ObjCObjectPointerType>()) {
1180 if (const ObjCObjectPointerType *IfacePtrTy =
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001181 MethodDecl->getResultType()->getAs<ObjCObjectPointerType>()) {
John McCall10302c02010-10-28 02:34:38 +00001182 // Allow non-matching return types as long as they don't violate
1183 // the principle of substitutability. Specifically, we permit
1184 // return types that are subclasses of the declared return type,
1185 // or that are more-qualified versions of the declared type.
1186 if (isObjCTypeSubstitutable(S.Context, IfacePtrTy, ImplPtrTy, false))
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001187 return false;
John McCall10302c02010-10-28 02:34:38 +00001188
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001189 DiagID =
1190 IsOverridingMode ? diag::warn_non_covariant_overriding_ret_types
1191 : diag::warn_non_covariant_ret_types;
John McCall10302c02010-10-28 02:34:38 +00001192 }
1193 }
1194
1195 S.Diag(MethodImpl->getLocation(), DiagID)
1196 << MethodImpl->getDeclName()
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001197 << MethodDecl->getResultType()
John McCall10302c02010-10-28 02:34:38 +00001198 << MethodImpl->getResultType()
1199 << getTypeRange(MethodImpl->getResultTypeSourceInfo());
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001200 S.Diag(MethodDecl->getLocation(),
1201 IsOverridingMode ? diag::note_previous_declaration
1202 : diag::note_previous_definition)
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001203 << getTypeRange(MethodDecl->getResultTypeSourceInfo());
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001204 return false;
John McCall10302c02010-10-28 02:34:38 +00001205}
1206
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001207static bool CheckMethodOverrideParam(Sema &S,
John McCall10302c02010-10-28 02:34:38 +00001208 ObjCMethodDecl *MethodImpl,
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001209 ObjCMethodDecl *MethodDecl,
John McCall10302c02010-10-28 02:34:38 +00001210 ParmVarDecl *ImplVar,
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001211 ParmVarDecl *IfaceVar,
Fariborz Jahanianeee3ef12011-07-24 20:53:26 +00001212 bool IsProtocolMethodDecl,
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001213 bool IsOverridingMode,
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001214 bool Warn) {
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001215 if (IsProtocolMethodDecl &&
1216 (ImplVar->getObjCDeclQualifier() !=
1217 IfaceVar->getObjCDeclQualifier())) {
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001218 if (Warn) {
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001219 if (IsOverridingMode)
1220 S.Diag(ImplVar->getLocation(),
1221 diag::warn_conflicting_overriding_param_modifiers)
1222 << getTypeRange(ImplVar->getTypeSourceInfo())
1223 << MethodImpl->getDeclName();
1224 else S.Diag(ImplVar->getLocation(),
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001225 diag::warn_conflicting_param_modifiers)
1226 << getTypeRange(ImplVar->getTypeSourceInfo())
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001227 << MethodImpl->getDeclName();
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001228 S.Diag(IfaceVar->getLocation(), diag::note_previous_declaration)
1229 << getTypeRange(IfaceVar->getTypeSourceInfo());
1230 }
1231 else
1232 return false;
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001233 }
1234
John McCall10302c02010-10-28 02:34:38 +00001235 QualType ImplTy = ImplVar->getType();
1236 QualType IfaceTy = IfaceVar->getType();
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001237
John McCall10302c02010-10-28 02:34:38 +00001238 if (S.Context.hasSameUnqualifiedType(ImplTy, IfaceTy))
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001239 return true;
1240
1241 if (!Warn)
1242 return false;
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001243 unsigned DiagID =
1244 IsOverridingMode ? diag::warn_conflicting_overriding_param_types
1245 : diag::warn_conflicting_param_types;
John McCall10302c02010-10-28 02:34:38 +00001246
1247 // Mismatches between ObjC pointers go into a different warning
1248 // category, and sometimes they're even completely whitelisted.
1249 if (const ObjCObjectPointerType *ImplPtrTy =
1250 ImplTy->getAs<ObjCObjectPointerType>()) {
1251 if (const ObjCObjectPointerType *IfacePtrTy =
1252 IfaceTy->getAs<ObjCObjectPointerType>()) {
1253 // Allow non-matching argument types as long as they don't
1254 // violate the principle of substitutability. Specifically, the
1255 // implementation must accept any objects that the superclass
1256 // accepts, however it may also accept others.
1257 if (isObjCTypeSubstitutable(S.Context, ImplPtrTy, IfacePtrTy, true))
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001258 return false;
John McCall10302c02010-10-28 02:34:38 +00001259
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001260 DiagID =
1261 IsOverridingMode ? diag::warn_non_contravariant_overriding_param_types
1262 : diag::warn_non_contravariant_param_types;
John McCall10302c02010-10-28 02:34:38 +00001263 }
1264 }
1265
1266 S.Diag(ImplVar->getLocation(), DiagID)
1267 << getTypeRange(ImplVar->getTypeSourceInfo())
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001268 << MethodImpl->getDeclName() << IfaceTy << ImplTy;
1269 S.Diag(IfaceVar->getLocation(),
1270 (IsOverridingMode ? diag::note_previous_declaration
1271 : diag::note_previous_definition))
John McCall10302c02010-10-28 02:34:38 +00001272 << getTypeRange(IfaceVar->getTypeSourceInfo());
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001273 return false;
John McCall10302c02010-10-28 02:34:38 +00001274}
John McCallf85e1932011-06-15 23:02:42 +00001275
1276/// In ARC, check whether the conventional meanings of the two methods
1277/// match. If they don't, it's a hard error.
1278static bool checkMethodFamilyMismatch(Sema &S, ObjCMethodDecl *impl,
1279 ObjCMethodDecl *decl) {
1280 ObjCMethodFamily implFamily = impl->getMethodFamily();
1281 ObjCMethodFamily declFamily = decl->getMethodFamily();
1282 if (implFamily == declFamily) return false;
1283
1284 // Since conventions are sorted by selector, the only possibility is
1285 // that the types differ enough to cause one selector or the other
1286 // to fall out of the family.
1287 assert(implFamily == OMF_None || declFamily == OMF_None);
1288
1289 // No further diagnostics required on invalid declarations.
1290 if (impl->isInvalidDecl() || decl->isInvalidDecl()) return true;
1291
1292 const ObjCMethodDecl *unmatched = impl;
1293 ObjCMethodFamily family = declFamily;
1294 unsigned errorID = diag::err_arc_lost_method_convention;
1295 unsigned noteID = diag::note_arc_lost_method_convention;
1296 if (declFamily == OMF_None) {
1297 unmatched = decl;
1298 family = implFamily;
1299 errorID = diag::err_arc_gained_method_convention;
1300 noteID = diag::note_arc_gained_method_convention;
1301 }
1302
1303 // Indexes into a %select clause in the diagnostic.
1304 enum FamilySelector {
1305 F_alloc, F_copy, F_mutableCopy = F_copy, F_init, F_new
1306 };
1307 FamilySelector familySelector = FamilySelector();
1308
1309 switch (family) {
1310 case OMF_None: llvm_unreachable("logic error, no method convention");
1311 case OMF_retain:
1312 case OMF_release:
1313 case OMF_autorelease:
1314 case OMF_dealloc:
Nico Weber80cb6e62011-08-28 22:35:17 +00001315 case OMF_finalize:
John McCallf85e1932011-06-15 23:02:42 +00001316 case OMF_retainCount:
1317 case OMF_self:
Fariborz Jahanian9670e172011-07-05 22:38:59 +00001318 case OMF_performSelector:
John McCallf85e1932011-06-15 23:02:42 +00001319 // Mismatches for these methods don't change ownership
1320 // conventions, so we don't care.
1321 return false;
1322
1323 case OMF_init: familySelector = F_init; break;
1324 case OMF_alloc: familySelector = F_alloc; break;
1325 case OMF_copy: familySelector = F_copy; break;
1326 case OMF_mutableCopy: familySelector = F_mutableCopy; break;
1327 case OMF_new: familySelector = F_new; break;
1328 }
1329
1330 enum ReasonSelector { R_NonObjectReturn, R_UnrelatedReturn };
1331 ReasonSelector reasonSelector;
1332
1333 // The only reason these methods don't fall within their families is
1334 // due to unusual result types.
1335 if (unmatched->getResultType()->isObjCObjectPointerType()) {
1336 reasonSelector = R_UnrelatedReturn;
1337 } else {
1338 reasonSelector = R_NonObjectReturn;
1339 }
1340
1341 S.Diag(impl->getLocation(), errorID) << familySelector << reasonSelector;
1342 S.Diag(decl->getLocation(), noteID) << familySelector << reasonSelector;
1343
1344 return true;
1345}
John McCall10302c02010-10-28 02:34:38 +00001346
Fariborz Jahanian8daab972008-12-05 18:18:52 +00001347void Sema::WarnConflictingTypedMethods(ObjCMethodDecl *ImpMethodDecl,
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001348 ObjCMethodDecl *MethodDecl,
Fariborz Jahanian36bc2c62011-10-10 17:53:29 +00001349 bool IsProtocolMethodDecl) {
John McCallf85e1932011-06-15 23:02:42 +00001350 if (getLangOptions().ObjCAutoRefCount &&
1351 checkMethodFamilyMismatch(*this, ImpMethodDecl, MethodDecl))
1352 return;
1353
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001354 CheckMethodOverrideReturn(*this, ImpMethodDecl, MethodDecl,
Fariborz Jahanian36bc2c62011-10-10 17:53:29 +00001355 IsProtocolMethodDecl, false,
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001356 true);
Mike Stump1eb44332009-09-09 15:08:12 +00001357
Chris Lattner3aff9192009-04-11 19:58:42 +00001358 for (ObjCMethodDecl::param_iterator IM = ImpMethodDecl->param_begin(),
Fariborz Jahanian21761c82011-02-21 23:49:15 +00001359 IF = MethodDecl->param_begin(), EM = ImpMethodDecl->param_end();
Fariborz Jahanian21121902011-08-08 18:03:17 +00001360 IM != EM; ++IM, ++IF) {
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001361 CheckMethodOverrideParam(*this, ImpMethodDecl, MethodDecl, *IM, *IF,
Fariborz Jahanian36bc2c62011-10-10 17:53:29 +00001362 IsProtocolMethodDecl, false, true);
Fariborz Jahanian21121902011-08-08 18:03:17 +00001363 }
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00001364
Fariborz Jahanian21121902011-08-08 18:03:17 +00001365 if (ImpMethodDecl->isVariadic() != MethodDecl->isVariadic()) {
Fariborz Jahanian36bc2c62011-10-10 17:53:29 +00001366 Diag(ImpMethodDecl->getLocation(),
1367 diag::warn_conflicting_variadic);
Fariborz Jahanian21121902011-08-08 18:03:17 +00001368 Diag(MethodDecl->getLocation(), diag::note_previous_declaration);
Fariborz Jahanian21121902011-08-08 18:03:17 +00001369 }
Fariborz Jahanian21121902011-08-08 18:03:17 +00001370}
1371
Fariborz Jahanian36bc2c62011-10-10 17:53:29 +00001372void Sema::CheckConflictingOverridingMethod(ObjCMethodDecl *Method,
1373 ObjCMethodDecl *Overridden,
1374 bool IsProtocolMethodDecl) {
1375
1376 CheckMethodOverrideReturn(*this, Method, Overridden,
1377 IsProtocolMethodDecl, true,
1378 true);
1379
1380 for (ObjCMethodDecl::param_iterator IM = Method->param_begin(),
1381 IF = Overridden->param_begin(), EM = Method->param_end();
1382 IM != EM; ++IM, ++IF) {
1383 CheckMethodOverrideParam(*this, Method, Overridden, *IM, *IF,
1384 IsProtocolMethodDecl, true, true);
1385 }
1386
1387 if (Method->isVariadic() != Overridden->isVariadic()) {
1388 Diag(Method->getLocation(),
1389 diag::warn_conflicting_overriding_variadic);
1390 Diag(Overridden->getLocation(), diag::note_previous_declaration);
1391 }
1392}
1393
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001394/// WarnExactTypedMethods - This routine issues a warning if method
1395/// implementation declaration matches exactly that of its declaration.
1396void Sema::WarnExactTypedMethods(ObjCMethodDecl *ImpMethodDecl,
1397 ObjCMethodDecl *MethodDecl,
1398 bool IsProtocolMethodDecl) {
1399 // don't issue warning when protocol method is optional because primary
1400 // class is not required to implement it and it is safe for protocol
1401 // to implement it.
1402 if (MethodDecl->getImplementationControl() == ObjCMethodDecl::Optional)
1403 return;
1404 // don't issue warning when primary class's method is
1405 // depecated/unavailable.
1406 if (MethodDecl->hasAttr<UnavailableAttr>() ||
1407 MethodDecl->hasAttr<DeprecatedAttr>())
1408 return;
1409
1410 bool match = CheckMethodOverrideReturn(*this, ImpMethodDecl, MethodDecl,
1411 IsProtocolMethodDecl, false, false);
1412 if (match)
1413 for (ObjCMethodDecl::param_iterator IM = ImpMethodDecl->param_begin(),
1414 IF = MethodDecl->param_begin(), EM = ImpMethodDecl->param_end();
1415 IM != EM; ++IM, ++IF) {
1416 match = CheckMethodOverrideParam(*this, ImpMethodDecl, MethodDecl,
1417 *IM, *IF,
1418 IsProtocolMethodDecl, false, false);
1419 if (!match)
1420 break;
1421 }
1422 if (match)
1423 match = (ImpMethodDecl->isVariadic() == MethodDecl->isVariadic());
David Chisnall7ca13ef2011-08-08 17:32:19 +00001424 if (match)
1425 match = !(MethodDecl->isClassMethod() &&
1426 MethodDecl->getSelector() == GetNullarySelector("load", Context));
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001427
1428 if (match) {
1429 Diag(ImpMethodDecl->getLocation(),
1430 diag::warn_category_method_impl_match);
1431 Diag(MethodDecl->getLocation(), diag::note_method_declared_at);
1432 }
1433}
1434
Mike Stump390b4cc2009-05-16 07:39:55 +00001435/// FIXME: Type hierarchies in Objective-C can be deep. We could most likely
1436/// improve the efficiency of selector lookups and type checking by associating
1437/// with each protocol / interface / category the flattened instance tables. If
1438/// we used an immutable set to keep the table then it wouldn't add significant
1439/// memory cost and it would be handy for lookups.
Daniel Dunbarb20ef3e2008-08-27 05:40:03 +00001440
Steve Naroffefe7f362008-02-08 22:06:17 +00001441/// CheckProtocolMethodDefs - This routine checks unimplemented methods
Chris Lattner4d391482007-12-12 07:09:47 +00001442/// Declared in protocol, and those referenced by it.
Steve Naroffefe7f362008-02-08 22:06:17 +00001443void Sema::CheckProtocolMethodDefs(SourceLocation ImpLoc,
1444 ObjCProtocolDecl *PDecl,
Chris Lattner4d391482007-12-12 07:09:47 +00001445 bool& IncompleteImpl,
Steve Naroffefe7f362008-02-08 22:06:17 +00001446 const llvm::DenseSet<Selector> &InsMap,
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +00001447 const llvm::DenseSet<Selector> &ClsMap,
Fariborz Jahanianf2838592010-03-27 21:10:05 +00001448 ObjCContainerDecl *CDecl) {
1449 ObjCInterfaceDecl *IDecl;
1450 if (ObjCCategoryDecl *C = dyn_cast<ObjCCategoryDecl>(CDecl))
1451 IDecl = C->getClassInterface();
1452 else
1453 IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl);
1454 assert (IDecl && "CheckProtocolMethodDefs - IDecl is null");
1455
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +00001456 ObjCInterfaceDecl *Super = IDecl->getSuperClass();
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001457 ObjCInterfaceDecl *NSIDecl = 0;
1458 if (getLangOptions().NeXTRuntime) {
Mike Stump1eb44332009-09-09 15:08:12 +00001459 // check to see if class implements forwardInvocation method and objects
1460 // of this class are derived from 'NSProxy' so that to forward requests
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001461 // from one object to another.
Mike Stump1eb44332009-09-09 15:08:12 +00001462 // Under such conditions, which means that every method possible is
1463 // implemented in the class, we should not issue "Method definition not
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001464 // found" warnings.
1465 // FIXME: Use a general GetUnarySelector method for this.
1466 IdentifierInfo* II = &Context.Idents.get("forwardInvocation");
1467 Selector fISelector = Context.Selectors.getSelector(1, &II);
1468 if (InsMap.count(fISelector))
1469 // Is IDecl derived from 'NSProxy'? If so, no instance methods
1470 // need be implemented in the implementation.
1471 NSIDecl = IDecl->lookupInheritedClass(&Context.Idents.get("NSProxy"));
1472 }
Mike Stump1eb44332009-09-09 15:08:12 +00001473
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +00001474 // If a method lookup fails locally we still need to look and see if
1475 // the method was implemented by a base class or an inherited
1476 // protocol. This lookup is slow, but occurs rarely in correct code
1477 // and otherwise would terminate in a warning.
1478
Chris Lattner4d391482007-12-12 07:09:47 +00001479 // check unimplemented instance methods.
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001480 if (!NSIDecl)
Mike Stump1eb44332009-09-09 15:08:12 +00001481 for (ObjCProtocolDecl::instmeth_iterator I = PDecl->instmeth_begin(),
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001482 E = PDecl->instmeth_end(); I != E; ++I) {
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001483 ObjCMethodDecl *method = *I;
Mike Stump1eb44332009-09-09 15:08:12 +00001484 if (method->getImplementationControl() != ObjCMethodDecl::Optional &&
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001485 !method->isSynthesized() && !InsMap.count(method->getSelector()) &&
Mike Stump1eb44332009-09-09 15:08:12 +00001486 (!Super ||
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001487 !Super->lookupInstanceMethod(method->getSelector()))) {
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001488 // Ugly, but necessary. Method declared in protcol might have
1489 // have been synthesized due to a property declared in the class which
1490 // uses the protocol.
Mike Stump1eb44332009-09-09 15:08:12 +00001491 ObjCMethodDecl *MethodInClass =
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001492 IDecl->lookupInstanceMethod(method->getSelector());
Fariborz Jahanian8822f7c2010-03-27 19:02:17 +00001493 if (!MethodInClass || !MethodInClass->isSynthesized()) {
Fariborz Jahanian52146832010-03-31 18:23:33 +00001494 unsigned DIAG = diag::warn_unimplemented_protocol_method;
Argyrios Kyrtzidis08274082010-12-15 18:44:22 +00001495 if (Diags.getDiagnosticLevel(DIAG, ImpLoc)
David Blaikied6471f72011-09-25 23:23:43 +00001496 != DiagnosticsEngine::Ignored) {
Fariborz Jahanian52146832010-03-31 18:23:33 +00001497 WarnUndefinedMethod(ImpLoc, method, IncompleteImpl, DIAG);
Fariborz Jahanian61c8d3e2010-10-29 23:20:05 +00001498 Diag(method->getLocation(), diag::note_method_declared_at);
Fariborz Jahanian52146832010-03-31 18:23:33 +00001499 Diag(CDecl->getLocation(), diag::note_required_for_protocol_at)
1500 << PDecl->getDeclName();
1501 }
Fariborz Jahanian8822f7c2010-03-27 19:02:17 +00001502 }
Fariborz Jahaniancd187622009-05-22 17:12:32 +00001503 }
1504 }
Chris Lattner4d391482007-12-12 07:09:47 +00001505 // check unimplemented class methods
Mike Stump1eb44332009-09-09 15:08:12 +00001506 for (ObjCProtocolDecl::classmeth_iterator
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001507 I = PDecl->classmeth_begin(), E = PDecl->classmeth_end();
Douglas Gregor6ab35242009-04-09 21:40:53 +00001508 I != E; ++I) {
Ted Kremeneka526c5c2008-01-07 19:49:32 +00001509 ObjCMethodDecl *method = *I;
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +00001510 if (method->getImplementationControl() != ObjCMethodDecl::Optional &&
1511 !ClsMap.count(method->getSelector()) &&
Fariborz Jahanian8822f7c2010-03-27 19:02:17 +00001512 (!Super || !Super->lookupClassMethod(method->getSelector()))) {
Fariborz Jahanian52146832010-03-31 18:23:33 +00001513 unsigned DIAG = diag::warn_unimplemented_protocol_method;
David Blaikied6471f72011-09-25 23:23:43 +00001514 if (Diags.getDiagnosticLevel(DIAG, ImpLoc) !=
1515 DiagnosticsEngine::Ignored) {
Fariborz Jahanian52146832010-03-31 18:23:33 +00001516 WarnUndefinedMethod(ImpLoc, method, IncompleteImpl, DIAG);
Fariborz Jahanian61c8d3e2010-10-29 23:20:05 +00001517 Diag(method->getLocation(), diag::note_method_declared_at);
Fariborz Jahanian52146832010-03-31 18:23:33 +00001518 Diag(IDecl->getLocation(), diag::note_required_for_protocol_at) <<
1519 PDecl->getDeclName();
1520 }
Fariborz Jahanian8822f7c2010-03-27 19:02:17 +00001521 }
Steve Naroff58dbdeb2007-12-14 23:37:57 +00001522 }
Chris Lattner780f3292008-07-21 21:32:27 +00001523 // Check on this protocols's referenced protocols, recursively.
1524 for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(),
1525 E = PDecl->protocol_end(); PI != E; ++PI)
Daniel Dunbar7ad1b1f2008-09-04 20:01:15 +00001526 CheckProtocolMethodDefs(ImpLoc, *PI, IncompleteImpl, InsMap, ClsMap, IDecl);
Chris Lattner4d391482007-12-12 07:09:47 +00001527}
1528
Fariborz Jahanian1e159bc2011-07-16 00:08:33 +00001529/// MatchAllMethodDeclarations - Check methods declared in interface
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001530/// or protocol against those declared in their implementations.
1531///
1532void Sema::MatchAllMethodDeclarations(const llvm::DenseSet<Selector> &InsMap,
1533 const llvm::DenseSet<Selector> &ClsMap,
1534 llvm::DenseSet<Selector> &InsMapSeen,
1535 llvm::DenseSet<Selector> &ClsMapSeen,
1536 ObjCImplDecl* IMPDecl,
1537 ObjCContainerDecl* CDecl,
1538 bool &IncompleteImpl,
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001539 bool ImmediateClass,
1540 bool WarnExactMatch) {
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001541 // Check and see if instance methods in class interface have been
1542 // implemented in the implementation class. If so, their types match.
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001543 for (ObjCInterfaceDecl::instmeth_iterator I = CDecl->instmeth_begin(),
1544 E = CDecl->instmeth_end(); I != E; ++I) {
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001545 if (InsMapSeen.count((*I)->getSelector()))
1546 continue;
1547 InsMapSeen.insert((*I)->getSelector());
Mike Stump1eb44332009-09-09 15:08:12 +00001548 if (!(*I)->isSynthesized() &&
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001549 !InsMap.count((*I)->getSelector())) {
1550 if (ImmediateClass)
Fariborz Jahanian52146832010-03-31 18:23:33 +00001551 WarnUndefinedMethod(IMPDecl->getLocation(), *I, IncompleteImpl,
1552 diag::note_undef_method_impl);
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001553 continue;
Mike Stumpac5fc7c2009-08-04 21:02:39 +00001554 } else {
Mike Stump1eb44332009-09-09 15:08:12 +00001555 ObjCMethodDecl *ImpMethodDecl =
Argyrios Kyrtzidis2334f3a2011-08-30 19:43:21 +00001556 IMPDecl->getInstanceMethod((*I)->getSelector());
1557 assert(CDecl->getInstanceMethod((*I)->getSelector()) &&
1558 "Expected to find the method through lookup as well");
1559 ObjCMethodDecl *MethodDecl = *I;
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001560 // ImpMethodDecl may be null as in a @dynamic property.
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001561 if (ImpMethodDecl) {
1562 if (!WarnExactMatch)
1563 WarnConflictingTypedMethods(ImpMethodDecl, MethodDecl,
1564 isa<ObjCProtocolDecl>(CDecl));
Fariborz Jahanian8c7e67d2011-08-25 22:58:42 +00001565 else if (!MethodDecl->isSynthesized())
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001566 WarnExactTypedMethods(ImpMethodDecl, MethodDecl,
1567 isa<ObjCProtocolDecl>(CDecl));
1568 }
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001569 }
1570 }
Mike Stump1eb44332009-09-09 15:08:12 +00001571
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001572 // Check and see if class methods in class interface have been
1573 // implemented in the implementation class. If so, their types match.
Mike Stump1eb44332009-09-09 15:08:12 +00001574 for (ObjCInterfaceDecl::classmeth_iterator
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001575 I = CDecl->classmeth_begin(), E = CDecl->classmeth_end(); I != E; ++I) {
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001576 if (ClsMapSeen.count((*I)->getSelector()))
1577 continue;
1578 ClsMapSeen.insert((*I)->getSelector());
1579 if (!ClsMap.count((*I)->getSelector())) {
1580 if (ImmediateClass)
Fariborz Jahanian52146832010-03-31 18:23:33 +00001581 WarnUndefinedMethod(IMPDecl->getLocation(), *I, IncompleteImpl,
1582 diag::note_undef_method_impl);
Mike Stumpac5fc7c2009-08-04 21:02:39 +00001583 } else {
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001584 ObjCMethodDecl *ImpMethodDecl =
1585 IMPDecl->getClassMethod((*I)->getSelector());
Argyrios Kyrtzidis2334f3a2011-08-30 19:43:21 +00001586 assert(CDecl->getClassMethod((*I)->getSelector()) &&
1587 "Expected to find the method through lookup as well");
1588 ObjCMethodDecl *MethodDecl = *I;
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001589 if (!WarnExactMatch)
1590 WarnConflictingTypedMethods(ImpMethodDecl, MethodDecl,
1591 isa<ObjCProtocolDecl>(CDecl));
1592 else
1593 WarnExactTypedMethods(ImpMethodDecl, MethodDecl,
1594 isa<ObjCProtocolDecl>(CDecl));
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001595 }
1596 }
Fariborz Jahanianf54e3ae2010-10-08 22:59:25 +00001597
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001598 if (ObjCInterfaceDecl *I = dyn_cast<ObjCInterfaceDecl> (CDecl)) {
Fariborz Jahanianf54e3ae2010-10-08 22:59:25 +00001599 // Also methods in class extensions need be looked at next.
1600 for (const ObjCCategoryDecl *ClsExtDecl = I->getFirstClassExtension();
1601 ClsExtDecl; ClsExtDecl = ClsExtDecl->getNextClassExtension())
1602 MatchAllMethodDeclarations(InsMap, ClsMap, InsMapSeen, ClsMapSeen,
1603 IMPDecl,
1604 const_cast<ObjCCategoryDecl *>(ClsExtDecl),
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001605 IncompleteImpl, false, WarnExactMatch);
Fariborz Jahanianf54e3ae2010-10-08 22:59:25 +00001606
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001607 // Check for any implementation of a methods declared in protocol.
Ted Kremenek53b94412010-09-01 01:21:15 +00001608 for (ObjCInterfaceDecl::all_protocol_iterator
1609 PI = I->all_referenced_protocol_begin(),
1610 E = I->all_referenced_protocol_end(); PI != E; ++PI)
Mike Stump1eb44332009-09-09 15:08:12 +00001611 MatchAllMethodDeclarations(InsMap, ClsMap, InsMapSeen, ClsMapSeen,
1612 IMPDecl,
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001613 (*PI), IncompleteImpl, false, WarnExactMatch);
Fariborz Jahanianeee3ef12011-07-24 20:53:26 +00001614
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001615 // FIXME. For now, we are not checking for extact match of methods
1616 // in category implementation and its primary class's super class.
1617 if (!WarnExactMatch && I->getSuperClass())
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001618 MatchAllMethodDeclarations(InsMap, ClsMap, InsMapSeen, ClsMapSeen,
Mike Stump1eb44332009-09-09 15:08:12 +00001619 IMPDecl,
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001620 I->getSuperClass(), IncompleteImpl, false);
1621 }
1622}
1623
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001624/// CheckCategoryVsClassMethodMatches - Checks that methods implemented in
1625/// category matches with those implemented in its primary class and
1626/// warns each time an exact match is found.
1627void Sema::CheckCategoryVsClassMethodMatches(
1628 ObjCCategoryImplDecl *CatIMPDecl) {
1629 llvm::DenseSet<Selector> InsMap, ClsMap;
1630
1631 for (ObjCImplementationDecl::instmeth_iterator
1632 I = CatIMPDecl->instmeth_begin(),
1633 E = CatIMPDecl->instmeth_end(); I!=E; ++I)
1634 InsMap.insert((*I)->getSelector());
1635
1636 for (ObjCImplementationDecl::classmeth_iterator
1637 I = CatIMPDecl->classmeth_begin(),
1638 E = CatIMPDecl->classmeth_end(); I != E; ++I)
1639 ClsMap.insert((*I)->getSelector());
1640 if (InsMap.empty() && ClsMap.empty())
1641 return;
1642
1643 // Get category's primary class.
1644 ObjCCategoryDecl *CatDecl = CatIMPDecl->getCategoryDecl();
1645 if (!CatDecl)
1646 return;
1647 ObjCInterfaceDecl *IDecl = CatDecl->getClassInterface();
1648 if (!IDecl)
1649 return;
1650 llvm::DenseSet<Selector> InsMapSeen, ClsMapSeen;
1651 bool IncompleteImpl = false;
1652 MatchAllMethodDeclarations(InsMap, ClsMap, InsMapSeen, ClsMapSeen,
1653 CatIMPDecl, IDecl,
1654 IncompleteImpl, false, true /*WarnExactMatch*/);
1655}
Fariborz Jahanianeee3ef12011-07-24 20:53:26 +00001656
Fariborz Jahanian17cb3262010-05-05 21:52:17 +00001657void Sema::ImplMethodsVsClassMethods(Scope *S, ObjCImplDecl* IMPDecl,
Mike Stump1eb44332009-09-09 15:08:12 +00001658 ObjCContainerDecl* CDecl,
Chris Lattnercddc8882009-03-01 00:56:52 +00001659 bool IncompleteImpl) {
Chris Lattner4d391482007-12-12 07:09:47 +00001660 llvm::DenseSet<Selector> InsMap;
1661 // Check and see if instance methods in class interface have been
1662 // implemented in the implementation class.
Mike Stump1eb44332009-09-09 15:08:12 +00001663 for (ObjCImplementationDecl::instmeth_iterator
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001664 I = IMPDecl->instmeth_begin(), E = IMPDecl->instmeth_end(); I!=E; ++I)
Chris Lattner4c525092007-12-12 17:58:05 +00001665 InsMap.insert((*I)->getSelector());
Mike Stump1eb44332009-09-09 15:08:12 +00001666
Fariborz Jahanian12bac252009-04-14 23:15:21 +00001667 // Check and see if properties declared in the interface have either 1)
1668 // an implementation or 2) there is a @synthesize/@dynamic implementation
1669 // of the property in the @implementation.
Ted Kremenekc32647d2010-12-23 21:35:43 +00001670 if (isa<ObjCInterfaceDecl>(CDecl) &&
1671 !(LangOpts.ObjCDefaultSynthProperties && LangOpts.ObjCNonFragileABI2))
Fariborz Jahanian17cb3262010-05-05 21:52:17 +00001672 DiagnoseUnimplementedProperties(S, IMPDecl, CDecl, InsMap);
Fariborz Jahanian3ac1eda2010-01-20 01:51:55 +00001673
Chris Lattner4d391482007-12-12 07:09:47 +00001674 llvm::DenseSet<Selector> ClsMap;
Mike Stump1eb44332009-09-09 15:08:12 +00001675 for (ObjCImplementationDecl::classmeth_iterator
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00001676 I = IMPDecl->classmeth_begin(),
1677 E = IMPDecl->classmeth_end(); I != E; ++I)
Chris Lattner4c525092007-12-12 17:58:05 +00001678 ClsMap.insert((*I)->getSelector());
Mike Stump1eb44332009-09-09 15:08:12 +00001679
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001680 // Check for type conflict of methods declared in a class/protocol and
1681 // its implementation; if any.
1682 llvm::DenseSet<Selector> InsMapSeen, ClsMapSeen;
Mike Stump1eb44332009-09-09 15:08:12 +00001683 MatchAllMethodDeclarations(InsMap, ClsMap, InsMapSeen, ClsMapSeen,
1684 IMPDecl, CDecl,
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001685 IncompleteImpl, true);
Fariborz Jahanian74133072011-08-03 18:21:12 +00001686
Fariborz Jahanianfefe91e2011-07-28 23:19:50 +00001687 // check all methods implemented in category against those declared
1688 // in its primary class.
1689 if (ObjCCategoryImplDecl *CatDecl =
1690 dyn_cast<ObjCCategoryImplDecl>(IMPDecl))
1691 CheckCategoryVsClassMethodMatches(CatDecl);
Mike Stump1eb44332009-09-09 15:08:12 +00001692
Chris Lattner4d391482007-12-12 07:09:47 +00001693 // Check the protocol list for unimplemented methods in the @implementation
1694 // class.
Fariborz Jahanianb33f3ad2009-05-01 20:07:12 +00001695 // Check and see if class methods in class interface have been
1696 // implemented in the implementation class.
Mike Stump1eb44332009-09-09 15:08:12 +00001697
Chris Lattnercddc8882009-03-01 00:56:52 +00001698 if (ObjCInterfaceDecl *I = dyn_cast<ObjCInterfaceDecl> (CDecl)) {
Ted Kremenek53b94412010-09-01 01:21:15 +00001699 for (ObjCInterfaceDecl::all_protocol_iterator
1700 PI = I->all_referenced_protocol_begin(),
1701 E = I->all_referenced_protocol_end(); PI != E; ++PI)
Mike Stump1eb44332009-09-09 15:08:12 +00001702 CheckProtocolMethodDefs(IMPDecl->getLocation(), *PI, IncompleteImpl,
Chris Lattnercddc8882009-03-01 00:56:52 +00001703 InsMap, ClsMap, I);
1704 // Check class extensions (unnamed categories)
Fariborz Jahanian80aa1cd2010-06-22 23:20:40 +00001705 for (const ObjCCategoryDecl *Categories = I->getFirstClassExtension();
1706 Categories; Categories = Categories->getNextClassExtension())
1707 ImplMethodsVsClassMethods(S, IMPDecl,
1708 const_cast<ObjCCategoryDecl*>(Categories),
1709 IncompleteImpl);
Chris Lattnercddc8882009-03-01 00:56:52 +00001710 } else if (ObjCCategoryDecl *C = dyn_cast<ObjCCategoryDecl>(CDecl)) {
Fariborz Jahanianb106fc62009-10-05 21:32:49 +00001711 // For extended class, unimplemented methods in its protocols will
1712 // be reported in the primary class.
Fariborz Jahanian25760612010-02-15 21:55:26 +00001713 if (!C->IsClassExtension()) {
Fariborz Jahanianb106fc62009-10-05 21:32:49 +00001714 for (ObjCCategoryDecl::protocol_iterator PI = C->protocol_begin(),
1715 E = C->protocol_end(); PI != E; ++PI)
1716 CheckProtocolMethodDefs(IMPDecl->getLocation(), *PI, IncompleteImpl,
Fariborz Jahanianf2838592010-03-27 21:10:05 +00001717 InsMap, ClsMap, CDecl);
Fariborz Jahanian3ad230e2010-01-20 19:36:21 +00001718 // Report unimplemented properties in the category as well.
1719 // When reporting on missing setter/getters, do not report when
1720 // setter/getter is implemented in category's primary class
1721 // implementation.
1722 if (ObjCInterfaceDecl *ID = C->getClassInterface())
1723 if (ObjCImplDecl *IMP = ID->getImplementation()) {
1724 for (ObjCImplementationDecl::instmeth_iterator
1725 I = IMP->instmeth_begin(), E = IMP->instmeth_end(); I!=E; ++I)
1726 InsMap.insert((*I)->getSelector());
1727 }
Fariborz Jahanian17cb3262010-05-05 21:52:17 +00001728 DiagnoseUnimplementedProperties(S, IMPDecl, CDecl, InsMap);
Fariborz Jahanian3ad230e2010-01-20 19:36:21 +00001729 }
Chris Lattnercddc8882009-03-01 00:56:52 +00001730 } else
David Blaikieb219cfc2011-09-23 05:06:16 +00001731 llvm_unreachable("invalid ObjCContainerDecl type.");
Chris Lattner4d391482007-12-12 07:09:47 +00001732}
1733
Mike Stump1eb44332009-09-09 15:08:12 +00001734/// ActOnForwardClassDeclaration -
Fariborz Jahanian95ed7782011-08-27 20:50:59 +00001735Sema::DeclGroupPtrTy
Chris Lattner4d391482007-12-12 07:09:47 +00001736Sema::ActOnForwardClassDeclaration(SourceLocation AtClassLoc,
Chris Lattnerbdbde4d2009-02-16 19:25:52 +00001737 IdentifierInfo **IdentList,
Ted Kremenekc09cba62009-11-17 23:12:20 +00001738 SourceLocation *IdentLocs,
Chris Lattnerbdbde4d2009-02-16 19:25:52 +00001739 unsigned NumElts) {
Fariborz Jahanian95ed7782011-08-27 20:50:59 +00001740 SmallVector<Decl *, 8> DeclsInGroup;
Chris Lattner4d391482007-12-12 07:09:47 +00001741 for (unsigned i = 0; i != NumElts; ++i) {
1742 // Check for another declaration kind with the same name.
John McCallf36e02d2009-10-09 21:13:30 +00001743 NamedDecl *PrevDecl
Douglas Gregorc83c6872010-04-15 22:33:43 +00001744 = LookupSingleName(TUScope, IdentList[i], IdentLocs[i],
Douglas Gregorc0b39642010-04-15 23:40:53 +00001745 LookupOrdinaryName, ForRedeclaration);
Douglas Gregorf57172b2008-12-08 18:40:42 +00001746 if (PrevDecl && PrevDecl->isTemplateParameter()) {
Douglas Gregor72c3f312008-12-05 18:15:24 +00001747 // Maybe we will complain about the shadowed template parameter.
1748 DiagnoseTemplateParameterShadow(AtClassLoc, PrevDecl);
1749 // Just pretend that we didn't see the previous declaration.
1750 PrevDecl = 0;
1751 }
1752
Ted Kremeneka526c5c2008-01-07 19:49:32 +00001753 if (PrevDecl && !isa<ObjCInterfaceDecl>(PrevDecl)) {
Steve Naroffc7333882008-06-05 22:57:10 +00001754 // GCC apparently allows the following idiom:
1755 //
1756 // typedef NSObject < XCElementTogglerP > XCElementToggler;
1757 // @class XCElementToggler;
1758 //
Mike Stump1eb44332009-09-09 15:08:12 +00001759 // FIXME: Make an extension?
Richard Smith162e1c12011-04-15 14:24:37 +00001760 TypedefNameDecl *TDD = dyn_cast<TypedefNameDecl>(PrevDecl);
John McCallc12c5bb2010-05-15 11:32:37 +00001761 if (!TDD || !TDD->getUnderlyingType()->isObjCObjectType()) {
Chris Lattner3c73c412008-11-19 08:23:25 +00001762 Diag(AtClassLoc, diag::err_redefinition_different_kind) << IdentList[i];
Chris Lattner5f4a6822008-11-23 23:12:31 +00001763 Diag(PrevDecl->getLocation(), diag::note_previous_definition);
John McCallc12c5bb2010-05-15 11:32:37 +00001764 } else {
Mike Stumpac5fc7c2009-08-04 21:02:39 +00001765 // a forward class declaration matching a typedef name of a class refers
1766 // to the underlying class.
John McCallc12c5bb2010-05-15 11:32:37 +00001767 if (const ObjCObjectType *OI =
1768 TDD->getUnderlyingType()->getAs<ObjCObjectType>())
1769 PrevDecl = OI->getInterface();
Fariborz Jahaniancae27c52009-05-07 21:49:26 +00001770 }
Chris Lattner4d391482007-12-12 07:09:47 +00001771 }
Douglas Gregor7723fec2011-12-15 20:29:51 +00001772
1773 // Create a declaration to describe this forward declaration.
Douglas Gregor0af55012011-12-16 03:12:41 +00001774 ObjCInterfaceDecl *PrevIDecl
1775 = dyn_cast_or_null<ObjCInterfaceDecl>(PrevDecl);
Douglas Gregor7723fec2011-12-15 20:29:51 +00001776 ObjCInterfaceDecl *IDecl
1777 = ObjCInterfaceDecl::Create(Context, CurContext, AtClassLoc,
Douglas Gregor375bb142011-12-27 22:43:10 +00001778 IdentList[i], PrevIDecl, IdentLocs[i]);
Douglas Gregor7723fec2011-12-15 20:29:51 +00001779 IDecl->setAtEndRange(IdentLocs[i]);
Douglas Gregor7723fec2011-12-15 20:29:51 +00001780
Douglas Gregor7723fec2011-12-15 20:29:51 +00001781 PushOnScopeChains(IDecl, TUScope);
Douglas Gregor375bb142011-12-27 22:43:10 +00001782 CheckObjCDeclScope(IDecl);
1783 DeclsInGroup.push_back(IDecl);
Chris Lattner4d391482007-12-12 07:09:47 +00001784 }
Fariborz Jahanian95ed7782011-08-27 20:50:59 +00001785
1786 return BuildDeclaratorGroup(DeclsInGroup.data(), DeclsInGroup.size(), false);
Chris Lattner4d391482007-12-12 07:09:47 +00001787}
1788
John McCall0f4c4c42011-06-16 01:15:19 +00001789static bool tryMatchRecordTypes(ASTContext &Context,
1790 Sema::MethodMatchStrategy strategy,
1791 const Type *left, const Type *right);
1792
John McCallf85e1932011-06-15 23:02:42 +00001793static bool matchTypes(ASTContext &Context, Sema::MethodMatchStrategy strategy,
1794 QualType leftQT, QualType rightQT) {
1795 const Type *left =
1796 Context.getCanonicalType(leftQT).getUnqualifiedType().getTypePtr();
1797 const Type *right =
1798 Context.getCanonicalType(rightQT).getUnqualifiedType().getTypePtr();
1799
1800 if (left == right) return true;
1801
1802 // If we're doing a strict match, the types have to match exactly.
1803 if (strategy == Sema::MMS_strict) return false;
1804
1805 if (left->isIncompleteType() || right->isIncompleteType()) return false;
1806
1807 // Otherwise, use this absurdly complicated algorithm to try to
1808 // validate the basic, low-level compatibility of the two types.
1809
1810 // As a minimum, require the sizes and alignments to match.
1811 if (Context.getTypeInfo(left) != Context.getTypeInfo(right))
1812 return false;
1813
1814 // Consider all the kinds of non-dependent canonical types:
1815 // - functions and arrays aren't possible as return and parameter types
1816
1817 // - vector types of equal size can be arbitrarily mixed
1818 if (isa<VectorType>(left)) return isa<VectorType>(right);
1819 if (isa<VectorType>(right)) return false;
1820
1821 // - references should only match references of identical type
John McCall0f4c4c42011-06-16 01:15:19 +00001822 // - structs, unions, and Objective-C objects must match more-or-less
1823 // exactly
John McCallf85e1932011-06-15 23:02:42 +00001824 // - everything else should be a scalar
1825 if (!left->isScalarType() || !right->isScalarType())
John McCall0f4c4c42011-06-16 01:15:19 +00001826 return tryMatchRecordTypes(Context, strategy, left, right);
John McCallf85e1932011-06-15 23:02:42 +00001827
John McCall1d9b3b22011-09-09 05:25:32 +00001828 // Make scalars agree in kind, except count bools as chars, and group
1829 // all non-member pointers together.
John McCallf85e1932011-06-15 23:02:42 +00001830 Type::ScalarTypeKind leftSK = left->getScalarTypeKind();
1831 Type::ScalarTypeKind rightSK = right->getScalarTypeKind();
1832 if (leftSK == Type::STK_Bool) leftSK = Type::STK_Integral;
1833 if (rightSK == Type::STK_Bool) rightSK = Type::STK_Integral;
John McCall1d9b3b22011-09-09 05:25:32 +00001834 if (leftSK == Type::STK_CPointer || leftSK == Type::STK_BlockPointer)
1835 leftSK = Type::STK_ObjCObjectPointer;
1836 if (rightSK == Type::STK_CPointer || rightSK == Type::STK_BlockPointer)
1837 rightSK = Type::STK_ObjCObjectPointer;
John McCallf85e1932011-06-15 23:02:42 +00001838
1839 // Note that data member pointers and function member pointers don't
1840 // intermix because of the size differences.
1841
1842 return (leftSK == rightSK);
1843}
Chris Lattner4d391482007-12-12 07:09:47 +00001844
John McCall0f4c4c42011-06-16 01:15:19 +00001845static bool tryMatchRecordTypes(ASTContext &Context,
1846 Sema::MethodMatchStrategy strategy,
1847 const Type *lt, const Type *rt) {
1848 assert(lt && rt && lt != rt);
1849
1850 if (!isa<RecordType>(lt) || !isa<RecordType>(rt)) return false;
1851 RecordDecl *left = cast<RecordType>(lt)->getDecl();
1852 RecordDecl *right = cast<RecordType>(rt)->getDecl();
1853
1854 // Require union-hood to match.
1855 if (left->isUnion() != right->isUnion()) return false;
1856
1857 // Require an exact match if either is non-POD.
1858 if ((isa<CXXRecordDecl>(left) && !cast<CXXRecordDecl>(left)->isPOD()) ||
1859 (isa<CXXRecordDecl>(right) && !cast<CXXRecordDecl>(right)->isPOD()))
1860 return false;
1861
1862 // Require size and alignment to match.
1863 if (Context.getTypeInfo(lt) != Context.getTypeInfo(rt)) return false;
1864
1865 // Require fields to match.
1866 RecordDecl::field_iterator li = left->field_begin(), le = left->field_end();
1867 RecordDecl::field_iterator ri = right->field_begin(), re = right->field_end();
1868 for (; li != le && ri != re; ++li, ++ri) {
1869 if (!matchTypes(Context, strategy, li->getType(), ri->getType()))
1870 return false;
1871 }
1872 return (li == le && ri == re);
1873}
1874
Chris Lattner4d391482007-12-12 07:09:47 +00001875/// MatchTwoMethodDeclarations - Checks that two methods have matching type and
1876/// returns true, or false, accordingly.
1877/// TODO: Handle protocol list; such as id<p1,p2> in type comparisons
John McCallf85e1932011-06-15 23:02:42 +00001878bool Sema::MatchTwoMethodDeclarations(const ObjCMethodDecl *left,
1879 const ObjCMethodDecl *right,
1880 MethodMatchStrategy strategy) {
1881 if (!matchTypes(Context, strategy,
1882 left->getResultType(), right->getResultType()))
1883 return false;
Mike Stump1eb44332009-09-09 15:08:12 +00001884
John McCallf85e1932011-06-15 23:02:42 +00001885 if (getLangOptions().ObjCAutoRefCount &&
1886 (left->hasAttr<NSReturnsRetainedAttr>()
1887 != right->hasAttr<NSReturnsRetainedAttr>() ||
1888 left->hasAttr<NSConsumesSelfAttr>()
1889 != right->hasAttr<NSConsumesSelfAttr>()))
1890 return false;
Mike Stump1eb44332009-09-09 15:08:12 +00001891
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +00001892 ObjCMethodDecl::param_const_iterator
John McCallf85e1932011-06-15 23:02:42 +00001893 li = left->param_begin(), le = left->param_end(), ri = right->param_begin();
Mike Stump1eb44332009-09-09 15:08:12 +00001894
John McCallf85e1932011-06-15 23:02:42 +00001895 for (; li != le; ++li, ++ri) {
1896 assert(ri != right->param_end() && "Param mismatch");
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +00001897 const ParmVarDecl *lparm = *li, *rparm = *ri;
John McCallf85e1932011-06-15 23:02:42 +00001898
1899 if (!matchTypes(Context, strategy, lparm->getType(), rparm->getType()))
1900 return false;
1901
1902 if (getLangOptions().ObjCAutoRefCount &&
1903 lparm->hasAttr<NSConsumedAttr>() != rparm->hasAttr<NSConsumedAttr>())
1904 return false;
Chris Lattner4d391482007-12-12 07:09:47 +00001905 }
1906 return true;
1907}
1908
Sebastian Redldb9d2142010-08-02 23:18:59 +00001909/// \brief Read the contents of the method pool for a given selector from
1910/// external storage.
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00001911///
Sebastian Redldb9d2142010-08-02 23:18:59 +00001912/// This routine should only be called once, when the method pool has no entry
1913/// for this selector.
1914Sema::GlobalMethodPool::iterator Sema::ReadMethodPool(Selector Sel) {
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00001915 assert(ExternalSource && "We need an external AST source");
Sebastian Redldb9d2142010-08-02 23:18:59 +00001916 assert(MethodPool.find(Sel) == MethodPool.end() &&
1917 "Selector data already loaded into the method pool");
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00001918
1919 // Read the method list from the external source.
Sebastian Redldb9d2142010-08-02 23:18:59 +00001920 GlobalMethods Methods = ExternalSource->ReadMethodPool(Sel);
Mike Stump1eb44332009-09-09 15:08:12 +00001921
Sebastian Redldb9d2142010-08-02 23:18:59 +00001922 return MethodPool.insert(std::make_pair(Sel, Methods)).first;
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00001923}
1924
Sebastian Redldb9d2142010-08-02 23:18:59 +00001925void Sema::AddMethodToGlobalPool(ObjCMethodDecl *Method, bool impl,
1926 bool instance) {
1927 GlobalMethodPool::iterator Pos = MethodPool.find(Method->getSelector());
1928 if (Pos == MethodPool.end()) {
1929 if (ExternalSource)
1930 Pos = ReadMethodPool(Method->getSelector());
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00001931 else
Sebastian Redldb9d2142010-08-02 23:18:59 +00001932 Pos = MethodPool.insert(std::make_pair(Method->getSelector(),
1933 GlobalMethods())).first;
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00001934 }
Fariborz Jahanian3fe10412010-07-22 18:24:20 +00001935 Method->setDefined(impl);
Sebastian Redldb9d2142010-08-02 23:18:59 +00001936 ObjCMethodList &Entry = instance ? Pos->second.first : Pos->second.second;
Chris Lattnerb25df352009-03-04 05:16:45 +00001937 if (Entry.Method == 0) {
Chris Lattner4d391482007-12-12 07:09:47 +00001938 // Haven't seen a method with this selector name yet - add it.
Chris Lattnerb25df352009-03-04 05:16:45 +00001939 Entry.Method = Method;
1940 Entry.Next = 0;
1941 return;
Chris Lattner4d391482007-12-12 07:09:47 +00001942 }
Mike Stump1eb44332009-09-09 15:08:12 +00001943
Chris Lattnerb25df352009-03-04 05:16:45 +00001944 // We've seen a method with this name, see if we have already seen this type
1945 // signature.
John McCallf85e1932011-06-15 23:02:42 +00001946 for (ObjCMethodList *List = &Entry; List; List = List->Next) {
1947 bool match = MatchTwoMethodDeclarations(Method, List->Method);
1948
1949 if (match) {
Fariborz Jahanian8e5fc9b2010-12-21 00:44:01 +00001950 ObjCMethodDecl *PrevObjCMethod = List->Method;
1951 PrevObjCMethod->setDefined(impl);
1952 // If a method is deprecated, push it in the global pool.
1953 // This is used for better diagnostics.
Douglas Gregor0a0d2b12011-03-23 00:50:03 +00001954 if (Method->isDeprecated()) {
1955 if (!PrevObjCMethod->isDeprecated())
Fariborz Jahanian8e5fc9b2010-12-21 00:44:01 +00001956 List->Method = Method;
1957 }
1958 // If new method is unavailable, push it into global pool
1959 // unless previous one is deprecated.
Douglas Gregor0a0d2b12011-03-23 00:50:03 +00001960 if (Method->isUnavailable()) {
1961 if (PrevObjCMethod->getAvailability() < AR_Deprecated)
Fariborz Jahanian8e5fc9b2010-12-21 00:44:01 +00001962 List->Method = Method;
1963 }
Chris Lattnerb25df352009-03-04 05:16:45 +00001964 return;
Fariborz Jahanian3fe10412010-07-22 18:24:20 +00001965 }
John McCallf85e1932011-06-15 23:02:42 +00001966 }
Mike Stump1eb44332009-09-09 15:08:12 +00001967
Chris Lattnerb25df352009-03-04 05:16:45 +00001968 // We have a new signature for an existing method - add it.
1969 // This is extremely rare. Only 1% of Cocoa selectors are "overloaded".
Ted Kremenek298ed872010-02-11 00:53:01 +00001970 ObjCMethodList *Mem = BumpAlloc.Allocate<ObjCMethodList>();
1971 Entry.Next = new (Mem) ObjCMethodList(Method, Entry.Next);
Chris Lattner4d391482007-12-12 07:09:47 +00001972}
1973
John McCallf85e1932011-06-15 23:02:42 +00001974/// Determines if this is an "acceptable" loose mismatch in the global
1975/// method pool. This exists mostly as a hack to get around certain
1976/// global mismatches which we can't afford to make warnings / errors.
1977/// Really, what we want is a way to take a method out of the global
1978/// method pool.
1979static bool isAcceptableMethodMismatch(ObjCMethodDecl *chosen,
1980 ObjCMethodDecl *other) {
1981 if (!chosen->isInstanceMethod())
1982 return false;
1983
1984 Selector sel = chosen->getSelector();
1985 if (!sel.isUnarySelector() || sel.getNameForSlot(0) != "length")
1986 return false;
1987
1988 // Don't complain about mismatches for -length if the method we
1989 // chose has an integral result type.
1990 return (chosen->getResultType()->isIntegerType());
1991}
1992
Sebastian Redldb9d2142010-08-02 23:18:59 +00001993ObjCMethodDecl *Sema::LookupMethodInGlobalPool(Selector Sel, SourceRange R,
Fariborz Jahanian6b308f62010-08-09 23:27:58 +00001994 bool receiverIdOrClass,
Sebastian Redldb9d2142010-08-02 23:18:59 +00001995 bool warn, bool instance) {
1996 GlobalMethodPool::iterator Pos = MethodPool.find(Sel);
1997 if (Pos == MethodPool.end()) {
1998 if (ExternalSource)
1999 Pos = ReadMethodPool(Sel);
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00002000 else
2001 return 0;
2002 }
2003
Sebastian Redldb9d2142010-08-02 23:18:59 +00002004 ObjCMethodList &MethList = instance ? Pos->second.first : Pos->second.second;
Mike Stump1eb44332009-09-09 15:08:12 +00002005
Sebastian Redldb9d2142010-08-02 23:18:59 +00002006 if (warn && MethList.Method && MethList.Next) {
John McCallf85e1932011-06-15 23:02:42 +00002007 bool issueDiagnostic = false, issueError = false;
2008
2009 // We support a warning which complains about *any* difference in
2010 // method signature.
2011 bool strictSelectorMatch =
2012 (receiverIdOrClass && warn &&
2013 (Diags.getDiagnosticLevel(diag::warn_strict_multiple_method_decl,
2014 R.getBegin()) !=
David Blaikied6471f72011-09-25 23:23:43 +00002015 DiagnosticsEngine::Ignored));
Fariborz Jahanian6b308f62010-08-09 23:27:58 +00002016 if (strictSelectorMatch)
2017 for (ObjCMethodList *Next = MethList.Next; Next; Next = Next->Next) {
John McCallf85e1932011-06-15 23:02:42 +00002018 if (!MatchTwoMethodDeclarations(MethList.Method, Next->Method,
2019 MMS_strict)) {
2020 issueDiagnostic = true;
2021 break;
2022 }
Fariborz Jahanian6b308f62010-08-09 23:27:58 +00002023 }
2024
John McCallf85e1932011-06-15 23:02:42 +00002025 // If we didn't see any strict differences, we won't see any loose
2026 // differences. In ARC, however, we also need to check for loose
2027 // mismatches, because most of them are errors.
2028 if (!strictSelectorMatch ||
2029 (issueDiagnostic && getLangOptions().ObjCAutoRefCount))
Fariborz Jahanian6b308f62010-08-09 23:27:58 +00002030 for (ObjCMethodList *Next = MethList.Next; Next; Next = Next->Next) {
John McCallf85e1932011-06-15 23:02:42 +00002031 // This checks if the methods differ in type mismatch.
2032 if (!MatchTwoMethodDeclarations(MethList.Method, Next->Method,
2033 MMS_loose) &&
2034 !isAcceptableMethodMismatch(MethList.Method, Next->Method)) {
2035 issueDiagnostic = true;
2036 if (getLangOptions().ObjCAutoRefCount)
2037 issueError = true;
2038 break;
2039 }
Fariborz Jahanian6b308f62010-08-09 23:27:58 +00002040 }
2041
John McCallf85e1932011-06-15 23:02:42 +00002042 if (issueDiagnostic) {
2043 if (issueError)
2044 Diag(R.getBegin(), diag::err_arc_multiple_method_decl) << Sel << R;
2045 else if (strictSelectorMatch)
Fariborz Jahanian6b308f62010-08-09 23:27:58 +00002046 Diag(R.getBegin(), diag::warn_strict_multiple_method_decl) << Sel << R;
2047 else
2048 Diag(R.getBegin(), diag::warn_multiple_method_decl) << Sel << R;
John McCallf85e1932011-06-15 23:02:42 +00002049
2050 Diag(MethList.Method->getLocStart(),
2051 issueError ? diag::note_possibility : diag::note_using)
Sebastian Redldb9d2142010-08-02 23:18:59 +00002052 << MethList.Method->getSourceRange();
2053 for (ObjCMethodList *Next = MethList.Next; Next; Next = Next->Next)
2054 Diag(Next->Method->getLocStart(), diag::note_also_found)
2055 << Next->Method->getSourceRange();
2056 }
Douglas Gregorf0aaf7a2009-04-24 21:10:55 +00002057 }
2058 return MethList.Method;
2059}
2060
Fariborz Jahanian3fe10412010-07-22 18:24:20 +00002061ObjCMethodDecl *Sema::LookupImplementedMethodInGlobalPool(Selector Sel) {
Sebastian Redldb9d2142010-08-02 23:18:59 +00002062 GlobalMethodPool::iterator Pos = MethodPool.find(Sel);
2063 if (Pos == MethodPool.end())
2064 return 0;
2065
2066 GlobalMethods &Methods = Pos->second;
2067
2068 if (Methods.first.Method && Methods.first.Method->isDefined())
2069 return Methods.first.Method;
2070 if (Methods.second.Method && Methods.second.Method->isDefined())
2071 return Methods.second.Method;
Fariborz Jahanian3fe10412010-07-22 18:24:20 +00002072 return 0;
2073}
2074
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002075/// CompareMethodParamsInBaseAndSuper - This routine compares methods with
2076/// identical selector names in current and its super classes and issues
2077/// a warning if any of their argument types are incompatible.
Fariborz Jahaniandbdec8b2009-08-04 01:07:16 +00002078void Sema::CompareMethodParamsInBaseAndSuper(Decl *ClassDecl,
2079 ObjCMethodDecl *Method,
2080 bool IsInstance) {
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002081 ObjCInterfaceDecl *ID = dyn_cast<ObjCInterfaceDecl>(ClassDecl);
2082 if (ID == 0) return;
Mike Stump1eb44332009-09-09 15:08:12 +00002083
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002084 while (ObjCInterfaceDecl *SD = ID->getSuperClass()) {
Mike Stump1eb44332009-09-09 15:08:12 +00002085 ObjCMethodDecl *SuperMethodDecl =
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002086 SD->lookupMethod(Method->getSelector(), IsInstance);
2087 if (SuperMethodDecl == 0) {
Fariborz Jahaniandbdec8b2009-08-04 01:07:16 +00002088 ID = SD;
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002089 continue;
Fariborz Jahaniandbdec8b2009-08-04 01:07:16 +00002090 }
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002091 ObjCMethodDecl::param_iterator ParamI = Method->param_begin(),
2092 E = Method->param_end();
2093 ObjCMethodDecl::param_iterator PrevI = SuperMethodDecl->param_begin();
2094 for (; ParamI != E; ++ParamI, ++PrevI) {
2095 // Number of parameters are the same and is guaranteed by selector match.
2096 assert(PrevI != SuperMethodDecl->param_end() && "Param mismatch");
2097 QualType T1 = Context.getCanonicalType((*ParamI)->getType());
2098 QualType T2 = Context.getCanonicalType((*PrevI)->getType());
Chris Lattnerfc8f0e12011-04-15 05:22:18 +00002099 // If type of argument of method in this class does not match its
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002100 // respective argument type in the super class method, issue warning;
2101 if (!Context.typesAreCompatible(T1, T2)) {
Mike Stump1eb44332009-09-09 15:08:12 +00002102 Diag((*ParamI)->getLocation(), diag::ext_typecheck_base_super)
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002103 << T1 << T2;
2104 Diag(SuperMethodDecl->getLocation(), diag::note_previous_declaration);
2105 return;
2106 }
2107 }
2108 ID = SD;
2109 }
Fariborz Jahaniandbdec8b2009-08-04 01:07:16 +00002110}
2111
Fariborz Jahanianf914b972010-02-23 23:41:11 +00002112/// DiagnoseDuplicateIvars -
2113/// Check for duplicate ivars in the entire class at the start of
2114/// @implementation. This becomes necesssary because class extension can
2115/// add ivars to a class in random order which will not be known until
2116/// class's @implementation is seen.
2117void Sema::DiagnoseDuplicateIvars(ObjCInterfaceDecl *ID,
2118 ObjCInterfaceDecl *SID) {
2119 for (ObjCInterfaceDecl::ivar_iterator IVI = ID->ivar_begin(),
2120 IVE = ID->ivar_end(); IVI != IVE; ++IVI) {
2121 ObjCIvarDecl* Ivar = (*IVI);
2122 if (Ivar->isInvalidDecl())
2123 continue;
2124 if (IdentifierInfo *II = Ivar->getIdentifier()) {
2125 ObjCIvarDecl* prevIvar = SID->lookupInstanceVariable(II);
2126 if (prevIvar) {
2127 Diag(Ivar->getLocation(), diag::err_duplicate_member) << II;
2128 Diag(prevIvar->getLocation(), diag::note_previous_declaration);
2129 Ivar->setInvalidDecl();
2130 }
2131 }
2132 }
2133}
2134
Erik Verbruggend64251f2011-12-06 09:25:23 +00002135Sema::ObjCContainerKind Sema::getObjCContainerKind() const {
2136 switch (CurContext->getDeclKind()) {
2137 case Decl::ObjCInterface:
2138 return Sema::OCK_Interface;
2139 case Decl::ObjCProtocol:
2140 return Sema::OCK_Protocol;
2141 case Decl::ObjCCategory:
2142 if (dyn_cast<ObjCCategoryDecl>(CurContext)->IsClassExtension())
2143 return Sema::OCK_ClassExtension;
2144 else
2145 return Sema::OCK_Category;
2146 case Decl::ObjCImplementation:
2147 return Sema::OCK_Implementation;
2148 case Decl::ObjCCategoryImpl:
2149 return Sema::OCK_CategoryImplementation;
2150
2151 default:
2152 return Sema::OCK_None;
2153 }
2154}
2155
Steve Naroffa56f6162007-12-18 01:30:32 +00002156// Note: For class/category implemenations, allMethods/allProperties is
2157// always null.
Erik Verbruggend64251f2011-12-06 09:25:23 +00002158Decl *Sema::ActOnAtEnd(Scope *S, SourceRange AtEnd,
2159 Decl **allMethods, unsigned allNum,
2160 Decl **allProperties, unsigned pNum,
2161 DeclGroupPtrTy *allTUVars, unsigned tuvNum) {
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002162
Erik Verbruggend64251f2011-12-06 09:25:23 +00002163 if (getObjCContainerKind() == Sema::OCK_None)
2164 return 0;
2165
2166 assert(AtEnd.isValid() && "Invalid location for '@end'");
2167
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002168 ObjCContainerDecl *OCD = dyn_cast<ObjCContainerDecl>(CurContext);
2169 Decl *ClassDecl = cast<Decl>(OCD);
Fariborz Jahanian63e963c2009-11-16 18:57:01 +00002170
Mike Stump1eb44332009-09-09 15:08:12 +00002171 bool isInterfaceDeclKind =
Chris Lattnerf8d17a52008-03-16 21:17:37 +00002172 isa<ObjCInterfaceDecl>(ClassDecl) || isa<ObjCCategoryDecl>(ClassDecl)
2173 || isa<ObjCProtocolDecl>(ClassDecl);
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002174 bool checkIdenticalMethods = isa<ObjCImplementationDecl>(ClassDecl);
Steve Naroff09c47192009-01-09 15:36:25 +00002175
Steve Naroff0701bbb2009-01-08 17:28:14 +00002176 // FIXME: Remove these and use the ObjCContainerDecl/DeclContext.
2177 llvm::DenseMap<Selector, const ObjCMethodDecl*> InsMap;
2178 llvm::DenseMap<Selector, const ObjCMethodDecl*> ClsMap;
2179
Chris Lattner4d391482007-12-12 07:09:47 +00002180 for (unsigned i = 0; i < allNum; i++ ) {
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002181 ObjCMethodDecl *Method =
John McCalld226f652010-08-21 09:40:31 +00002182 cast_or_null<ObjCMethodDecl>(allMethods[i]);
Chris Lattner4d391482007-12-12 07:09:47 +00002183
2184 if (!Method) continue; // Already issued a diagnostic.
Douglas Gregorf8d49f62009-01-09 17:18:27 +00002185 if (Method->isInstanceMethod()) {
Chris Lattner4d391482007-12-12 07:09:47 +00002186 /// Check for instance method of the same name with incompatible types
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002187 const ObjCMethodDecl *&PrevMethod = InsMap[Method->getSelector()];
Mike Stump1eb44332009-09-09 15:08:12 +00002188 bool match = PrevMethod ? MatchTwoMethodDeclarations(Method, PrevMethod)
Chris Lattner4d391482007-12-12 07:09:47 +00002189 : false;
Mike Stump1eb44332009-09-09 15:08:12 +00002190 if ((isInterfaceDeclKind && PrevMethod && !match)
Eli Friedman82b4e762008-12-16 20:15:50 +00002191 || (checkIdenticalMethods && match)) {
Chris Lattner5f4a6822008-11-23 23:12:31 +00002192 Diag(Method->getLocation(), diag::err_duplicate_method_decl)
Chris Lattner077bf5e2008-11-24 03:33:13 +00002193 << Method->getDeclName();
Chris Lattner5f4a6822008-11-23 23:12:31 +00002194 Diag(PrevMethod->getLocation(), diag::note_previous_declaration);
Douglas Gregorbdb2d502010-12-21 17:34:17 +00002195 Method->setInvalidDecl();
Chris Lattner4d391482007-12-12 07:09:47 +00002196 } else {
Fariborz Jahanian72096462011-12-13 19:40:34 +00002197 if (PrevMethod) {
Argyrios Kyrtzidis3a919e72011-10-14 08:02:31 +00002198 Method->setAsRedeclaration(PrevMethod);
Fariborz Jahanian72096462011-12-13 19:40:34 +00002199 if (!Context.getSourceManager().isInSystemHeader(
2200 Method->getLocation()))
2201 Diag(Method->getLocation(), diag::warn_duplicate_method_decl)
2202 << Method->getDeclName();
2203 Diag(PrevMethod->getLocation(), diag::note_previous_declaration);
2204 }
Chris Lattner4d391482007-12-12 07:09:47 +00002205 InsMap[Method->getSelector()] = Method;
2206 /// The following allows us to typecheck messages to "id".
2207 AddInstanceMethodToGlobalPool(Method);
Mike Stump1eb44332009-09-09 15:08:12 +00002208 // verify that the instance method conforms to the same definition of
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002209 // parent methods if it shadows one.
Fariborz Jahaniandbdec8b2009-08-04 01:07:16 +00002210 CompareMethodParamsInBaseAndSuper(ClassDecl, Method, true);
Chris Lattner4d391482007-12-12 07:09:47 +00002211 }
Mike Stumpac5fc7c2009-08-04 21:02:39 +00002212 } else {
Chris Lattner4d391482007-12-12 07:09:47 +00002213 /// Check for class method of the same name with incompatible types
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002214 const ObjCMethodDecl *&PrevMethod = ClsMap[Method->getSelector()];
Mike Stump1eb44332009-09-09 15:08:12 +00002215 bool match = PrevMethod ? MatchTwoMethodDeclarations(Method, PrevMethod)
Chris Lattner4d391482007-12-12 07:09:47 +00002216 : false;
Mike Stump1eb44332009-09-09 15:08:12 +00002217 if ((isInterfaceDeclKind && PrevMethod && !match)
Eli Friedman82b4e762008-12-16 20:15:50 +00002218 || (checkIdenticalMethods && match)) {
Chris Lattner5f4a6822008-11-23 23:12:31 +00002219 Diag(Method->getLocation(), diag::err_duplicate_method_decl)
Chris Lattner077bf5e2008-11-24 03:33:13 +00002220 << Method->getDeclName();
Chris Lattner5f4a6822008-11-23 23:12:31 +00002221 Diag(PrevMethod->getLocation(), diag::note_previous_declaration);
Douglas Gregorbdb2d502010-12-21 17:34:17 +00002222 Method->setInvalidDecl();
Chris Lattner4d391482007-12-12 07:09:47 +00002223 } else {
Fariborz Jahanian72096462011-12-13 19:40:34 +00002224 if (PrevMethod) {
Argyrios Kyrtzidis3a919e72011-10-14 08:02:31 +00002225 Method->setAsRedeclaration(PrevMethod);
Fariborz Jahanian72096462011-12-13 19:40:34 +00002226 if (!Context.getSourceManager().isInSystemHeader(
2227 Method->getLocation()))
2228 Diag(Method->getLocation(), diag::warn_duplicate_method_decl)
2229 << Method->getDeclName();
2230 Diag(PrevMethod->getLocation(), diag::note_previous_declaration);
2231 }
Chris Lattner4d391482007-12-12 07:09:47 +00002232 ClsMap[Method->getSelector()] = Method;
Steve Naroffa56f6162007-12-18 01:30:32 +00002233 /// The following allows us to typecheck messages to "Class".
2234 AddFactoryMethodToGlobalPool(Method);
Mike Stump1eb44332009-09-09 15:08:12 +00002235 // verify that the class method conforms to the same definition of
Fariborz Jahaniane198f5d2009-08-04 17:01:09 +00002236 // parent methods if it shadows one.
Fariborz Jahaniandbdec8b2009-08-04 01:07:16 +00002237 CompareMethodParamsInBaseAndSuper(ClassDecl, Method, false);
Chris Lattner4d391482007-12-12 07:09:47 +00002238 }
2239 }
2240 }
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002241 if (ObjCInterfaceDecl *I = dyn_cast<ObjCInterfaceDecl>(ClassDecl)) {
Mike Stump1eb44332009-09-09 15:08:12 +00002242 // Compares properties declared in this class to those of its
Fariborz Jahanian02edb982008-05-01 00:03:38 +00002243 // super class.
Fariborz Jahanianaebf0cb2008-05-02 19:17:30 +00002244 ComparePropertiesInBaseAndSuper(I);
John McCalld226f652010-08-21 09:40:31 +00002245 CompareProperties(I, I);
Steve Naroff09c47192009-01-09 15:36:25 +00002246 } else if (ObjCCategoryDecl *C = dyn_cast<ObjCCategoryDecl>(ClassDecl)) {
Fariborz Jahanian77e14bd2008-12-06 19:59:02 +00002247 // Categories are used to extend the class by declaring new methods.
Mike Stump1eb44332009-09-09 15:08:12 +00002248 // By the same token, they are also used to add new properties. No
Fariborz Jahanian77e14bd2008-12-06 19:59:02 +00002249 // need to compare the added property to those in the class.
Daniel Dunbarb20ef3e2008-08-27 05:40:03 +00002250
Fariborz Jahanian107089f2010-01-18 18:41:16 +00002251 // Compare protocol properties with those in category
John McCalld226f652010-08-21 09:40:31 +00002252 CompareProperties(C, C);
Fariborz Jahanian88f5e9b2010-12-10 23:36:33 +00002253 if (C->IsClassExtension()) {
2254 ObjCInterfaceDecl *CCPrimary = C->getClassInterface();
2255 DiagnoseClassExtensionDupMethods(C, CCPrimary);
Fariborz Jahanian88f5e9b2010-12-10 23:36:33 +00002256 }
Chris Lattner4d391482007-12-12 07:09:47 +00002257 }
Steve Naroff09c47192009-01-09 15:36:25 +00002258 if (ObjCContainerDecl *CDecl = dyn_cast<ObjCContainerDecl>(ClassDecl)) {
Fariborz Jahanian25760612010-02-15 21:55:26 +00002259 if (CDecl->getIdentifier())
2260 // ProcessPropertyDecl is responsible for diagnosing conflicts with any
2261 // user-defined setter/getter. It also synthesizes setter/getter methods
2262 // and adds them to the DeclContext and global method pools.
2263 for (ObjCContainerDecl::prop_iterator I = CDecl->prop_begin(),
2264 E = CDecl->prop_end();
2265 I != E; ++I)
2266 ProcessPropertyDecl(*I, CDecl);
Ted Kremenek782f2f52010-01-07 01:20:12 +00002267 CDecl->setAtEndRange(AtEnd);
Steve Naroff09c47192009-01-09 15:36:25 +00002268 }
2269 if (ObjCImplementationDecl *IC=dyn_cast<ObjCImplementationDecl>(ClassDecl)) {
Ted Kremenek782f2f52010-01-07 01:20:12 +00002270 IC->setAtEndRange(AtEnd);
Fariborz Jahanian7ca8b062009-11-11 22:40:11 +00002271 if (ObjCInterfaceDecl* IDecl = IC->getClassInterface()) {
Fariborz Jahanianc78f6842010-12-11 18:39:37 +00002272 // Any property declared in a class extension might have user
2273 // declared setter or getter in current class extension or one
2274 // of the other class extensions. Mark them as synthesized as
2275 // property will be synthesized when property with same name is
2276 // seen in the @implementation.
2277 for (const ObjCCategoryDecl *ClsExtDecl =
2278 IDecl->getFirstClassExtension();
2279 ClsExtDecl; ClsExtDecl = ClsExtDecl->getNextClassExtension()) {
2280 for (ObjCContainerDecl::prop_iterator I = ClsExtDecl->prop_begin(),
2281 E = ClsExtDecl->prop_end(); I != E; ++I) {
2282 ObjCPropertyDecl *Property = (*I);
2283 // Skip over properties declared @dynamic
2284 if (const ObjCPropertyImplDecl *PIDecl
2285 = IC->FindPropertyImplDecl(Property->getIdentifier()))
2286 if (PIDecl->getPropertyImplementation()
2287 == ObjCPropertyImplDecl::Dynamic)
2288 continue;
2289
2290 for (const ObjCCategoryDecl *CExtDecl =
2291 IDecl->getFirstClassExtension();
2292 CExtDecl; CExtDecl = CExtDecl->getNextClassExtension()) {
2293 if (ObjCMethodDecl *GetterMethod =
2294 CExtDecl->getInstanceMethod(Property->getGetterName()))
2295 GetterMethod->setSynthesized(true);
2296 if (!Property->isReadOnly())
2297 if (ObjCMethodDecl *SetterMethod =
2298 CExtDecl->getInstanceMethod(Property->getSetterName()))
2299 SetterMethod->setSynthesized(true);
2300 }
2301 }
2302 }
Fariborz Jahanian17cb3262010-05-05 21:52:17 +00002303 ImplMethodsVsClassMethods(S, IC, IDecl);
Fariborz Jahanian7ca8b062009-11-11 22:40:11 +00002304 AtomicPropertySetterGetterRules(IC, IDecl);
John McCallf85e1932011-06-15 23:02:42 +00002305 DiagnoseOwningPropertyGetterSynthesis(IC);
Fariborz Jahanian2c18bb72010-08-20 21:21:08 +00002306
Fariborz Jahanianf914b972010-02-23 23:41:11 +00002307 if (LangOpts.ObjCNonFragileABI2)
2308 while (IDecl->getSuperClass()) {
2309 DiagnoseDuplicateIvars(IDecl, IDecl->getSuperClass());
2310 IDecl = IDecl->getSuperClass();
2311 }
Fariborz Jahanian7ca8b062009-11-11 22:40:11 +00002312 }
Fariborz Jahaniane4498c62010-04-28 16:11:27 +00002313 SetIvarInitializers(IC);
Mike Stump1eb44332009-09-09 15:08:12 +00002314 } else if (ObjCCategoryImplDecl* CatImplClass =
Steve Naroff09c47192009-01-09 15:36:25 +00002315 dyn_cast<ObjCCategoryImplDecl>(ClassDecl)) {
Ted Kremenek782f2f52010-01-07 01:20:12 +00002316 CatImplClass->setAtEndRange(AtEnd);
Mike Stump1eb44332009-09-09 15:08:12 +00002317
Chris Lattner4d391482007-12-12 07:09:47 +00002318 // Find category interface decl and then check that all methods declared
Daniel Dunbarb20ef3e2008-08-27 05:40:03 +00002319 // in this interface are implemented in the category @implementation.
Chris Lattner97a58872009-02-16 18:32:47 +00002320 if (ObjCInterfaceDecl* IDecl = CatImplClass->getClassInterface()) {
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002321 for (ObjCCategoryDecl *Categories = IDecl->getCategoryList();
Chris Lattner4d391482007-12-12 07:09:47 +00002322 Categories; Categories = Categories->getNextClassCategory()) {
2323 if (Categories->getIdentifier() == CatImplClass->getIdentifier()) {
Fariborz Jahanian17cb3262010-05-05 21:52:17 +00002324 ImplMethodsVsClassMethods(S, CatImplClass, Categories);
Chris Lattner4d391482007-12-12 07:09:47 +00002325 break;
2326 }
2327 }
2328 }
2329 }
Chris Lattner682bf922009-03-29 16:50:03 +00002330 if (isInterfaceDeclKind) {
2331 // Reject invalid vardecls.
2332 for (unsigned i = 0; i != tuvNum; i++) {
2333 DeclGroupRef DG = allTUVars[i].getAsVal<DeclGroupRef>();
2334 for (DeclGroupRef::iterator I = DG.begin(), E = DG.end(); I != E; ++I)
2335 if (VarDecl *VDecl = dyn_cast<VarDecl>(*I)) {
Daniel Dunbar5466c7b2009-04-14 02:25:56 +00002336 if (!VDecl->hasExternalStorage())
Steve Naroff87454162009-04-13 17:58:46 +00002337 Diag(VDecl->getLocation(), diag::err_objc_var_decl_inclass);
Fariborz Jahanianb31cb7f2009-03-21 18:06:45 +00002338 }
Chris Lattner682bf922009-03-29 16:50:03 +00002339 }
Fariborz Jahanian38e24c72009-03-18 22:33:24 +00002340 }
Fariborz Jahanian10af8792011-08-29 17:33:12 +00002341 ActOnObjCContainerFinishDefinition();
Argyrios Kyrtzidisb4a686d2011-10-17 19:48:13 +00002342
2343 for (unsigned i = 0; i != tuvNum; i++) {
2344 DeclGroupRef DG = allTUVars[i].getAsVal<DeclGroupRef>();
Argyrios Kyrtzidisc14a03d2011-11-23 20:27:36 +00002345 for (DeclGroupRef::iterator I = DG.begin(), E = DG.end(); I != E; ++I)
2346 (*I)->setTopLevelDeclInObjCContainer();
Argyrios Kyrtzidisb4a686d2011-10-17 19:48:13 +00002347 Consumer.HandleTopLevelDeclInObjCContainer(DG);
2348 }
Erik Verbruggend64251f2011-12-06 09:25:23 +00002349
2350 return ClassDecl;
Chris Lattner4d391482007-12-12 07:09:47 +00002351}
2352
2353
2354/// CvtQTToAstBitMask - utility routine to produce an AST bitmask for
2355/// objective-c's type qualifier from the parser version of the same info.
Mike Stump1eb44332009-09-09 15:08:12 +00002356static Decl::ObjCDeclQualifier
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002357CvtQTToAstBitMask(ObjCDeclSpec::ObjCDeclQualifier PQTVal) {
John McCall09e2c522011-05-01 03:04:29 +00002358 return (Decl::ObjCDeclQualifier) (unsigned) PQTVal;
Chris Lattner4d391482007-12-12 07:09:47 +00002359}
2360
Ted Kremenek422bae72010-04-18 04:59:38 +00002361static inline
Fariborz Jahanian7fda4002011-10-22 01:21:15 +00002362bool containsInvalidMethodImplAttribute(ObjCMethodDecl *IMD,
2363 const AttrVec &A) {
2364 // If method is only declared in implementation (private method),
Fariborz Jahanian7fda4002011-10-22 01:21:15 +00002365 // No need to issue any diagnostics on method definition with attributes.
Fariborz Jahanianee28a4b2011-10-22 01:56:45 +00002366 if (!IMD)
Fariborz Jahanian7fda4002011-10-22 01:21:15 +00002367 return false;
2368
Fariborz Jahanianee28a4b2011-10-22 01:56:45 +00002369 // method declared in interface has no attribute.
2370 // But implementation has attributes. This is invalid
2371 if (!IMD->hasAttrs())
2372 return true;
2373
Fariborz Jahanian7fda4002011-10-22 01:21:15 +00002374 const AttrVec &D = IMD->getAttrs();
2375 if (D.size() != A.size())
2376 return true;
2377
2378 // attributes on method declaration and definition must match exactly.
2379 // Note that we have at most a couple of attributes on methods, so this
2380 // n*n search is good enough.
2381 for (AttrVec::const_iterator i = A.begin(), e = A.end(); i != e; ++i) {
2382 bool match = false;
2383 for (AttrVec::const_iterator i1 = D.begin(), e1 = D.end(); i1 != e1; ++i1) {
2384 if ((*i)->getKind() == (*i1)->getKind()) {
2385 match = true;
2386 break;
2387 }
2388 }
2389 if (!match)
Sean Huntcf807c42010-08-18 23:23:40 +00002390 return true;
Fariborz Jahanian7fda4002011-10-22 01:21:15 +00002391 }
Sean Huntcf807c42010-08-18 23:23:40 +00002392 return false;
Ted Kremenek422bae72010-04-18 04:59:38 +00002393}
2394
Douglas Gregore97179c2011-09-08 01:46:34 +00002395namespace {
2396 /// \brief Describes the compatibility of a result type with its method.
2397 enum ResultTypeCompatibilityKind {
2398 RTC_Compatible,
2399 RTC_Incompatible,
2400 RTC_Unknown
2401 };
2402}
2403
Douglas Gregor926df6c2011-06-11 01:09:30 +00002404/// \brief Check whether the declared result type of the given Objective-C
2405/// method declaration is compatible with the method's class.
2406///
Douglas Gregore97179c2011-09-08 01:46:34 +00002407static ResultTypeCompatibilityKind
Douglas Gregor926df6c2011-06-11 01:09:30 +00002408CheckRelatedResultTypeCompatibility(Sema &S, ObjCMethodDecl *Method,
2409 ObjCInterfaceDecl *CurrentClass) {
2410 QualType ResultType = Method->getResultType();
Douglas Gregor926df6c2011-06-11 01:09:30 +00002411
2412 // If an Objective-C method inherits its related result type, then its
2413 // declared result type must be compatible with its own class type. The
2414 // declared result type is compatible if:
2415 if (const ObjCObjectPointerType *ResultObjectType
2416 = ResultType->getAs<ObjCObjectPointerType>()) {
2417 // - it is id or qualified id, or
2418 if (ResultObjectType->isObjCIdType() ||
2419 ResultObjectType->isObjCQualifiedIdType())
Douglas Gregore97179c2011-09-08 01:46:34 +00002420 return RTC_Compatible;
Douglas Gregor926df6c2011-06-11 01:09:30 +00002421
2422 if (CurrentClass) {
2423 if (ObjCInterfaceDecl *ResultClass
2424 = ResultObjectType->getInterfaceDecl()) {
2425 // - it is the same as the method's class type, or
Douglas Gregor60ef3082011-12-15 00:29:59 +00002426 if (declaresSameEntity(CurrentClass, ResultClass))
Douglas Gregore97179c2011-09-08 01:46:34 +00002427 return RTC_Compatible;
Douglas Gregor926df6c2011-06-11 01:09:30 +00002428
2429 // - it is a superclass of the method's class type
2430 if (ResultClass->isSuperClassOf(CurrentClass))
Douglas Gregore97179c2011-09-08 01:46:34 +00002431 return RTC_Compatible;
Douglas Gregor926df6c2011-06-11 01:09:30 +00002432 }
Douglas Gregore97179c2011-09-08 01:46:34 +00002433 } else {
2434 // Any Objective-C pointer type might be acceptable for a protocol
2435 // method; we just don't know.
2436 return RTC_Unknown;
Douglas Gregor926df6c2011-06-11 01:09:30 +00002437 }
2438 }
2439
Douglas Gregore97179c2011-09-08 01:46:34 +00002440 return RTC_Incompatible;
Douglas Gregor926df6c2011-06-11 01:09:30 +00002441}
2442
John McCall6c2c2502011-07-22 02:45:48 +00002443namespace {
2444/// A helper class for searching for methods which a particular method
2445/// overrides.
2446class OverrideSearch {
2447 Sema &S;
2448 ObjCMethodDecl *Method;
2449 llvm::SmallPtrSet<ObjCContainerDecl*, 8> Searched;
2450 llvm::SmallPtrSet<ObjCMethodDecl*, 8> Overridden;
2451 bool Recursive;
2452
2453public:
2454 OverrideSearch(Sema &S, ObjCMethodDecl *method) : S(S), Method(method) {
2455 Selector selector = method->getSelector();
2456
2457 // Bypass this search if we've never seen an instance/class method
2458 // with this selector before.
2459 Sema::GlobalMethodPool::iterator it = S.MethodPool.find(selector);
2460 if (it == S.MethodPool.end()) {
2461 if (!S.ExternalSource) return;
2462 it = S.ReadMethodPool(selector);
2463 }
2464 ObjCMethodList &list =
2465 method->isInstanceMethod() ? it->second.first : it->second.second;
2466 if (!list.Method) return;
2467
2468 ObjCContainerDecl *container
2469 = cast<ObjCContainerDecl>(method->getDeclContext());
2470
2471 // Prevent the search from reaching this container again. This is
2472 // important with categories, which override methods from the
2473 // interface and each other.
2474 Searched.insert(container);
2475 searchFromContainer(container);
Douglas Gregor926df6c2011-06-11 01:09:30 +00002476 }
John McCall6c2c2502011-07-22 02:45:48 +00002477
2478 typedef llvm::SmallPtrSet<ObjCMethodDecl*,8>::iterator iterator;
2479 iterator begin() const { return Overridden.begin(); }
2480 iterator end() const { return Overridden.end(); }
2481
2482private:
2483 void searchFromContainer(ObjCContainerDecl *container) {
2484 if (container->isInvalidDecl()) return;
2485
2486 switch (container->getDeclKind()) {
2487#define OBJCCONTAINER(type, base) \
2488 case Decl::type: \
2489 searchFrom(cast<type##Decl>(container)); \
2490 break;
2491#define ABSTRACT_DECL(expansion)
2492#define DECL(type, base) \
2493 case Decl::type:
2494#include "clang/AST/DeclNodes.inc"
2495 llvm_unreachable("not an ObjC container!");
2496 }
2497 }
2498
2499 void searchFrom(ObjCProtocolDecl *protocol) {
2500 // A method in a protocol declaration overrides declarations from
2501 // referenced ("parent") protocols.
2502 search(protocol->getReferencedProtocols());
2503 }
2504
2505 void searchFrom(ObjCCategoryDecl *category) {
2506 // A method in a category declaration overrides declarations from
2507 // the main class and from protocols the category references.
2508 search(category->getClassInterface());
2509 search(category->getReferencedProtocols());
2510 }
2511
2512 void searchFrom(ObjCCategoryImplDecl *impl) {
2513 // A method in a category definition that has a category
2514 // declaration overrides declarations from the category
2515 // declaration.
2516 if (ObjCCategoryDecl *category = impl->getCategoryDecl()) {
2517 search(category);
2518
2519 // Otherwise it overrides declarations from the class.
2520 } else {
2521 search(impl->getClassInterface());
2522 }
2523 }
2524
2525 void searchFrom(ObjCInterfaceDecl *iface) {
2526 // A method in a class declaration overrides declarations from
Douglas Gregor2e5c15b2011-12-15 05:27:12 +00002527 if (!iface->hasDefinition())
2528 return;
2529
John McCall6c2c2502011-07-22 02:45:48 +00002530 // - categories,
2531 for (ObjCCategoryDecl *category = iface->getCategoryList();
2532 category; category = category->getNextClassCategory())
2533 search(category);
2534
2535 // - the super class, and
2536 if (ObjCInterfaceDecl *super = iface->getSuperClass())
2537 search(super);
2538
2539 // - any referenced protocols.
2540 search(iface->getReferencedProtocols());
2541 }
2542
2543 void searchFrom(ObjCImplementationDecl *impl) {
2544 // A method in a class implementation overrides declarations from
2545 // the class interface.
2546 search(impl->getClassInterface());
2547 }
2548
2549
2550 void search(const ObjCProtocolList &protocols) {
2551 for (ObjCProtocolList::iterator i = protocols.begin(), e = protocols.end();
2552 i != e; ++i)
2553 search(*i);
2554 }
2555
2556 void search(ObjCContainerDecl *container) {
2557 // Abort if we've already searched this container.
2558 if (!Searched.insert(container)) return;
2559
2560 // Check for a method in this container which matches this selector.
2561 ObjCMethodDecl *meth = container->getMethod(Method->getSelector(),
2562 Method->isInstanceMethod());
2563
2564 // If we find one, record it and bail out.
2565 if (meth) {
2566 Overridden.insert(meth);
2567 return;
2568 }
2569
2570 // Otherwise, search for methods that a hypothetical method here
2571 // would have overridden.
2572
2573 // Note that we're now in a recursive case.
2574 Recursive = true;
2575
2576 searchFromContainer(container);
2577 }
2578};
Douglas Gregor926df6c2011-06-11 01:09:30 +00002579}
2580
John McCalld226f652010-08-21 09:40:31 +00002581Decl *Sema::ActOnMethodDeclaration(
Fariborz Jahanian7f532532011-02-09 22:20:01 +00002582 Scope *S,
Chris Lattner4d391482007-12-12 07:09:47 +00002583 SourceLocation MethodLoc, SourceLocation EndLoc,
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002584 tok::TokenKind MethodType,
John McCallb3d87482010-08-24 05:47:05 +00002585 ObjCDeclSpec &ReturnQT, ParsedType ReturnType,
Argyrios Kyrtzidis11d77162011-10-03 06:36:36 +00002586 ArrayRef<SourceLocation> SelectorLocs,
Chris Lattner4d391482007-12-12 07:09:47 +00002587 Selector Sel,
2588 // optional arguments. The number of types/arguments is obtained
2589 // from the Sel.getNumArgs().
Chris Lattnere294d3f2009-04-11 18:57:04 +00002590 ObjCArgInfo *ArgInfo,
Fariborz Jahanian4f4fd922010-04-08 00:30:06 +00002591 DeclaratorChunk::ParamInfo *CParamInfo, unsigned CNumArgs, // c-style args
Chris Lattner4d391482007-12-12 07:09:47 +00002592 AttributeList *AttrList, tok::ObjCKeywordKind MethodDeclKind,
Fariborz Jahanian90ba78c2011-03-12 18:54:30 +00002593 bool isVariadic, bool MethodDefinition) {
Steve Naroffda323ad2008-02-29 21:48:07 +00002594 // Make sure we can establish a context for the method.
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002595 if (!CurContext->isObjCContainer()) {
Steve Naroffda323ad2008-02-29 21:48:07 +00002596 Diag(MethodLoc, diag::error_missing_method_context);
John McCalld226f652010-08-21 09:40:31 +00002597 return 0;
Steve Naroffda323ad2008-02-29 21:48:07 +00002598 }
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002599 ObjCContainerDecl *OCD = dyn_cast<ObjCContainerDecl>(CurContext);
2600 Decl *ClassDecl = cast<Decl>(OCD);
Chris Lattner4d391482007-12-12 07:09:47 +00002601 QualType resultDeclType;
Mike Stump1eb44332009-09-09 15:08:12 +00002602
Douglas Gregore97179c2011-09-08 01:46:34 +00002603 bool HasRelatedResultType = false;
Douglas Gregor4bc1cb62010-03-08 14:59:44 +00002604 TypeSourceInfo *ResultTInfo = 0;
Steve Naroffccef3712009-02-20 22:59:16 +00002605 if (ReturnType) {
Douglas Gregor4bc1cb62010-03-08 14:59:44 +00002606 resultDeclType = GetTypeFromParser(ReturnType, &ResultTInfo);
Mike Stump1eb44332009-09-09 15:08:12 +00002607
Steve Naroffccef3712009-02-20 22:59:16 +00002608 // Methods cannot return interface types. All ObjC objects are
2609 // passed by reference.
John McCallc12c5bb2010-05-15 11:32:37 +00002610 if (resultDeclType->isObjCObjectType()) {
Chris Lattner2dd979f2009-04-11 19:08:56 +00002611 Diag(MethodLoc, diag::err_object_cannot_be_passed_returned_by_value)
2612 << 0 << resultDeclType;
John McCalld226f652010-08-21 09:40:31 +00002613 return 0;
Douglas Gregor926df6c2011-06-11 01:09:30 +00002614 }
Douglas Gregore97179c2011-09-08 01:46:34 +00002615
2616 HasRelatedResultType = (resultDeclType == Context.getObjCInstanceType());
Fariborz Jahanianaab24a62011-07-21 17:00:47 +00002617 } else { // get the type for "id".
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002618 resultDeclType = Context.getObjCIdType();
Fariborz Jahanianfeb4fa12011-07-21 17:38:14 +00002619 Diag(MethodLoc, diag::warn_missing_method_return_type)
Argyrios Kyrtzidis11d77162011-10-03 06:36:36 +00002620 << FixItHint::CreateInsertion(SelectorLocs.front(), "(id)");
Fariborz Jahanianaab24a62011-07-21 17:00:47 +00002621 }
Mike Stump1eb44332009-09-09 15:08:12 +00002622
2623 ObjCMethodDecl* ObjCMethod =
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +00002624 ObjCMethodDecl::Create(Context, MethodLoc, EndLoc, Sel,
Argyrios Kyrtzidis11d77162011-10-03 06:36:36 +00002625 resultDeclType,
Douglas Gregor4bc1cb62010-03-08 14:59:44 +00002626 ResultTInfo,
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002627 CurContext,
Chris Lattner6c4ae5d2008-03-16 00:49:28 +00002628 MethodType == tok::minus, isVariadic,
Argyrios Kyrtzidis75cf3e82011-08-17 19:25:08 +00002629 /*isSynthesized=*/false,
2630 /*isImplicitlyDeclared=*/false, /*isDefined=*/false,
Douglas Gregor926df6c2011-06-11 01:09:30 +00002631 MethodDeclKind == tok::objc_optional
2632 ? ObjCMethodDecl::Optional
2633 : ObjCMethodDecl::Required,
Douglas Gregore97179c2011-09-08 01:46:34 +00002634 HasRelatedResultType);
Mike Stump1eb44332009-09-09 15:08:12 +00002635
Chris Lattner5f9e2722011-07-23 10:55:15 +00002636 SmallVector<ParmVarDecl*, 16> Params;
Mike Stump1eb44332009-09-09 15:08:12 +00002637
Chris Lattner7db638d2009-04-11 19:42:43 +00002638 for (unsigned i = 0, e = Sel.getNumArgs(); i != e; ++i) {
John McCall58e46772009-10-23 21:48:59 +00002639 QualType ArgType;
John McCalla93c9342009-12-07 02:54:59 +00002640 TypeSourceInfo *DI;
Mike Stump1eb44332009-09-09 15:08:12 +00002641
Chris Lattnere294d3f2009-04-11 18:57:04 +00002642 if (ArgInfo[i].Type == 0) {
John McCall58e46772009-10-23 21:48:59 +00002643 ArgType = Context.getObjCIdType();
2644 DI = 0;
Chris Lattnere294d3f2009-04-11 18:57:04 +00002645 } else {
John McCall58e46772009-10-23 21:48:59 +00002646 ArgType = GetTypeFromParser(ArgInfo[i].Type, &DI);
Steve Naroff6082c622008-12-09 19:36:17 +00002647 // Perform the default array/function conversions (C99 6.7.5.3p[7,8]).
Douglas Gregor79e6bd32011-07-12 04:42:08 +00002648 ArgType = Context.getAdjustedParameterType(ArgType);
Chris Lattnere294d3f2009-04-11 18:57:04 +00002649 }
Mike Stump1eb44332009-09-09 15:08:12 +00002650
Fariborz Jahanian7f532532011-02-09 22:20:01 +00002651 LookupResult R(*this, ArgInfo[i].Name, ArgInfo[i].NameLoc,
2652 LookupOrdinaryName, ForRedeclaration);
2653 LookupName(R, S);
2654 if (R.isSingleResult()) {
2655 NamedDecl *PrevDecl = R.getFoundDecl();
2656 if (S->isDeclScope(PrevDecl)) {
Fariborz Jahanian90ba78c2011-03-12 18:54:30 +00002657 Diag(ArgInfo[i].NameLoc,
2658 (MethodDefinition ? diag::warn_method_param_redefinition
2659 : diag::warn_method_param_declaration))
Fariborz Jahanian7f532532011-02-09 22:20:01 +00002660 << ArgInfo[i].Name;
2661 Diag(PrevDecl->getLocation(),
2662 diag::note_previous_declaration);
2663 }
2664 }
2665
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002666 SourceLocation StartLoc = DI
2667 ? DI->getTypeLoc().getBeginLoc()
2668 : ArgInfo[i].NameLoc;
2669
John McCall81ef3e62011-04-23 02:46:06 +00002670 ParmVarDecl* Param = CheckParameter(ObjCMethod, StartLoc,
2671 ArgInfo[i].NameLoc, ArgInfo[i].Name,
2672 ArgType, DI, SC_None, SC_None);
Mike Stump1eb44332009-09-09 15:08:12 +00002673
John McCall70798862011-05-02 00:30:12 +00002674 Param->setObjCMethodScopeInfo(i);
2675
Chris Lattner0ed844b2008-04-04 06:12:32 +00002676 Param->setObjCDeclQualifier(
Chris Lattnere294d3f2009-04-11 18:57:04 +00002677 CvtQTToAstBitMask(ArgInfo[i].DeclSpec.getObjCDeclQualifier()));
Mike Stump1eb44332009-09-09 15:08:12 +00002678
Chris Lattnerf97e8fa2009-04-11 19:34:56 +00002679 // Apply the attributes to the parameter.
Douglas Gregor9cdda0c2009-06-17 21:51:59 +00002680 ProcessDeclAttributeList(TUScope, Param, ArgInfo[i].ArgAttrs);
Mike Stump1eb44332009-09-09 15:08:12 +00002681
Fariborz Jahanian7f532532011-02-09 22:20:01 +00002682 S->AddDecl(Param);
2683 IdResolver.AddDecl(Param);
2684
Chris Lattner0ed844b2008-04-04 06:12:32 +00002685 Params.push_back(Param);
2686 }
Fariborz Jahanian7f532532011-02-09 22:20:01 +00002687
Fariborz Jahanian4f4fd922010-04-08 00:30:06 +00002688 for (unsigned i = 0, e = CNumArgs; i != e; ++i) {
John McCalld226f652010-08-21 09:40:31 +00002689 ParmVarDecl *Param = cast<ParmVarDecl>(CParamInfo[i].Param);
Fariborz Jahanian4f4fd922010-04-08 00:30:06 +00002690 QualType ArgType = Param->getType();
2691 if (ArgType.isNull())
2692 ArgType = Context.getObjCIdType();
2693 else
2694 // Perform the default array/function conversions (C99 6.7.5.3p[7,8]).
Douglas Gregor79e6bd32011-07-12 04:42:08 +00002695 ArgType = Context.getAdjustedParameterType(ArgType);
John McCallc12c5bb2010-05-15 11:32:37 +00002696 if (ArgType->isObjCObjectType()) {
Fariborz Jahanian4f4fd922010-04-08 00:30:06 +00002697 Diag(Param->getLocation(),
2698 diag::err_object_cannot_be_passed_returned_by_value)
2699 << 1 << ArgType;
2700 Param->setInvalidDecl();
2701 }
2702 Param->setDeclContext(ObjCMethod);
Fariborz Jahanian7f532532011-02-09 22:20:01 +00002703
Fariborz Jahanian4f4fd922010-04-08 00:30:06 +00002704 Params.push_back(Param);
2705 }
2706
Argyrios Kyrtzidis491306a2011-10-03 06:37:04 +00002707 ObjCMethod->setMethodParams(Context, Params, SelectorLocs);
Ted Kremeneka526c5c2008-01-07 19:49:32 +00002708 ObjCMethod->setObjCDeclQualifier(
2709 CvtQTToAstBitMask(ReturnQT.getObjCDeclQualifier()));
Daniel Dunbar35682492008-09-26 04:12:28 +00002710
2711 if (AttrList)
Douglas Gregor9cdda0c2009-06-17 21:51:59 +00002712 ProcessDeclAttributeList(TUScope, ObjCMethod, AttrList);
Mike Stump1eb44332009-09-09 15:08:12 +00002713
Douglas Gregorbdb2d502010-12-21 17:34:17 +00002714 // Add the method now.
John McCall6c2c2502011-07-22 02:45:48 +00002715 const ObjCMethodDecl *PrevMethod = 0;
2716 if (ObjCImplDecl *ImpDecl = dyn_cast<ObjCImplDecl>(ClassDecl)) {
Chris Lattner4d391482007-12-12 07:09:47 +00002717 if (MethodType == tok::minus) {
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00002718 PrevMethod = ImpDecl->getInstanceMethod(Sel);
2719 ImpDecl->addInstanceMethod(ObjCMethod);
Chris Lattner4d391482007-12-12 07:09:47 +00002720 } else {
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00002721 PrevMethod = ImpDecl->getClassMethod(Sel);
2722 ImpDecl->addClassMethod(ObjCMethod);
Chris Lattner4d391482007-12-12 07:09:47 +00002723 }
Douglas Gregor926df6c2011-06-11 01:09:30 +00002724
Fariborz Jahanian7fda4002011-10-22 01:21:15 +00002725 ObjCMethodDecl *IMD = 0;
2726 if (ObjCInterfaceDecl *IDecl = ImpDecl->getClassInterface())
2727 IMD = IDecl->lookupMethod(ObjCMethod->getSelector(),
2728 ObjCMethod->isInstanceMethod());
Sean Huntcf807c42010-08-18 23:23:40 +00002729 if (ObjCMethod->hasAttrs() &&
Fariborz Jahanianec236782011-12-06 00:02:41 +00002730 containsInvalidMethodImplAttribute(IMD, ObjCMethod->getAttrs())) {
Fariborz Jahanian28441e62011-12-21 00:09:11 +00002731 SourceLocation MethodLoc = IMD->getLocation();
2732 if (!getSourceManager().isInSystemHeader(MethodLoc)) {
2733 Diag(EndLoc, diag::warn_attribute_method_def);
2734 Diag(MethodLoc, diag::note_method_declared_at);
2735 }
Fariborz Jahanianec236782011-12-06 00:02:41 +00002736 }
Douglas Gregorbdb2d502010-12-21 17:34:17 +00002737 } else {
2738 cast<DeclContext>(ClassDecl)->addDecl(ObjCMethod);
Chris Lattner4d391482007-12-12 07:09:47 +00002739 }
John McCall6c2c2502011-07-22 02:45:48 +00002740
Chris Lattner4d391482007-12-12 07:09:47 +00002741 if (PrevMethod) {
2742 // You can never have two method definitions with the same name.
Chris Lattner5f4a6822008-11-23 23:12:31 +00002743 Diag(ObjCMethod->getLocation(), diag::err_duplicate_method_decl)
Chris Lattner077bf5e2008-11-24 03:33:13 +00002744 << ObjCMethod->getDeclName();
Chris Lattner5f4a6822008-11-23 23:12:31 +00002745 Diag(PrevMethod->getLocation(), diag::note_previous_declaration);
Mike Stump1eb44332009-09-09 15:08:12 +00002746 }
John McCall54abf7d2009-11-04 02:18:39 +00002747
Douglas Gregor926df6c2011-06-11 01:09:30 +00002748 // If this Objective-C method does not have a related result type, but we
2749 // are allowed to infer related result types, try to do so based on the
2750 // method family.
2751 ObjCInterfaceDecl *CurrentClass = dyn_cast<ObjCInterfaceDecl>(ClassDecl);
2752 if (!CurrentClass) {
2753 if (ObjCCategoryDecl *Cat = dyn_cast<ObjCCategoryDecl>(ClassDecl))
2754 CurrentClass = Cat->getClassInterface();
2755 else if (ObjCImplDecl *Impl = dyn_cast<ObjCImplDecl>(ClassDecl))
2756 CurrentClass = Impl->getClassInterface();
2757 else if (ObjCCategoryImplDecl *CatImpl
2758 = dyn_cast<ObjCCategoryImplDecl>(ClassDecl))
2759 CurrentClass = CatImpl->getClassInterface();
2760 }
John McCall6c2c2502011-07-22 02:45:48 +00002761
Douglas Gregore97179c2011-09-08 01:46:34 +00002762 ResultTypeCompatibilityKind RTC
2763 = CheckRelatedResultTypeCompatibility(*this, ObjCMethod, CurrentClass);
John McCall6c2c2502011-07-22 02:45:48 +00002764
2765 // Search for overridden methods and merge information down from them.
2766 OverrideSearch overrides(*this, ObjCMethod);
2767 for (OverrideSearch::iterator
2768 i = overrides.begin(), e = overrides.end(); i != e; ++i) {
2769 ObjCMethodDecl *overridden = *i;
2770
2771 // Propagate down the 'related result type' bit from overridden methods.
Douglas Gregore97179c2011-09-08 01:46:34 +00002772 if (RTC != RTC_Incompatible && overridden->hasRelatedResultType())
Douglas Gregor926df6c2011-06-11 01:09:30 +00002773 ObjCMethod->SetRelatedResultType();
John McCall6c2c2502011-07-22 02:45:48 +00002774
2775 // Then merge the declarations.
2776 mergeObjCMethodDecls(ObjCMethod, overridden);
Fariborz Jahanian730cfb12011-08-10 17:16:30 +00002777
2778 // Check for overriding methods
2779 if (isa<ObjCInterfaceDecl>(ObjCMethod->getDeclContext()) ||
Fariborz Jahanian36bc2c62011-10-10 17:53:29 +00002780 isa<ObjCImplementationDecl>(ObjCMethod->getDeclContext()))
2781 CheckConflictingOverridingMethod(ObjCMethod, overridden,
2782 isa<ObjCProtocolDecl>(overridden->getDeclContext()));
Douglas Gregor926df6c2011-06-11 01:09:30 +00002783 }
2784
John McCallf85e1932011-06-15 23:02:42 +00002785 bool ARCError = false;
2786 if (getLangOptions().ObjCAutoRefCount)
2787 ARCError = CheckARCMethodDecl(*this, ObjCMethod);
2788
Douglas Gregore97179c2011-09-08 01:46:34 +00002789 // Infer the related result type when possible.
2790 if (!ARCError && RTC == RTC_Compatible &&
2791 !ObjCMethod->hasRelatedResultType() &&
2792 LangOpts.ObjCInferRelatedResultType) {
Douglas Gregor926df6c2011-06-11 01:09:30 +00002793 bool InferRelatedResultType = false;
2794 switch (ObjCMethod->getMethodFamily()) {
2795 case OMF_None:
2796 case OMF_copy:
2797 case OMF_dealloc:
Nico Weber80cb6e62011-08-28 22:35:17 +00002798 case OMF_finalize:
Douglas Gregor926df6c2011-06-11 01:09:30 +00002799 case OMF_mutableCopy:
2800 case OMF_release:
2801 case OMF_retainCount:
Fariborz Jahanian9670e172011-07-05 22:38:59 +00002802 case OMF_performSelector:
Douglas Gregor926df6c2011-06-11 01:09:30 +00002803 break;
2804
2805 case OMF_alloc:
2806 case OMF_new:
2807 InferRelatedResultType = ObjCMethod->isClassMethod();
2808 break;
2809
2810 case OMF_init:
2811 case OMF_autorelease:
2812 case OMF_retain:
2813 case OMF_self:
2814 InferRelatedResultType = ObjCMethod->isInstanceMethod();
2815 break;
2816 }
2817
John McCall6c2c2502011-07-22 02:45:48 +00002818 if (InferRelatedResultType)
Douglas Gregor926df6c2011-06-11 01:09:30 +00002819 ObjCMethod->SetRelatedResultType();
Douglas Gregor926df6c2011-06-11 01:09:30 +00002820 }
2821
John McCalld226f652010-08-21 09:40:31 +00002822 return ObjCMethod;
Chris Lattner4d391482007-12-12 07:09:47 +00002823}
2824
Chris Lattnercc98eac2008-12-17 07:13:27 +00002825bool Sema::CheckObjCDeclScope(Decl *D) {
Sebastian Redl7a126a42010-08-31 00:36:30 +00002826 if (isa<TranslationUnitDecl>(CurContext->getRedeclContext()))
Anders Carlsson15281452008-11-04 16:57:32 +00002827 return false;
Fariborz Jahanian58a76492011-08-22 18:34:22 +00002828 // Following is also an error. But it is caused by a missing @end
2829 // and diagnostic is issued elsewhere.
Fariborz Jahaniana28948f2011-08-22 15:54:49 +00002830 if (isa<ObjCContainerDecl>(CurContext->getRedeclContext())) {
2831 return false;
2832 }
2833
Anders Carlsson15281452008-11-04 16:57:32 +00002834 Diag(D->getLocation(), diag::err_objc_decls_may_only_appear_in_global_scope);
2835 D->setInvalidDecl();
Mike Stump1eb44332009-09-09 15:08:12 +00002836
Anders Carlsson15281452008-11-04 16:57:32 +00002837 return true;
2838}
Chris Lattnercc98eac2008-12-17 07:13:27 +00002839
Chris Lattnercc98eac2008-12-17 07:13:27 +00002840/// Called whenever @defs(ClassName) is encountered in the source. Inserts the
2841/// instance variables of ClassName into Decls.
John McCalld226f652010-08-21 09:40:31 +00002842void Sema::ActOnDefs(Scope *S, Decl *TagD, SourceLocation DeclStart,
Chris Lattnercc98eac2008-12-17 07:13:27 +00002843 IdentifierInfo *ClassName,
Chris Lattner5f9e2722011-07-23 10:55:15 +00002844 SmallVectorImpl<Decl*> &Decls) {
Chris Lattnercc98eac2008-12-17 07:13:27 +00002845 // Check that ClassName is a valid class
Douglas Gregorc83c6872010-04-15 22:33:43 +00002846 ObjCInterfaceDecl *Class = getObjCInterfaceDecl(ClassName, DeclStart);
Chris Lattnercc98eac2008-12-17 07:13:27 +00002847 if (!Class) {
2848 Diag(DeclStart, diag::err_undef_interface) << ClassName;
2849 return;
2850 }
Fariborz Jahanian0468fb92009-04-21 20:28:41 +00002851 if (LangOpts.ObjCNonFragileABI) {
2852 Diag(DeclStart, diag::err_atdef_nonfragile_interface);
2853 return;
2854 }
Mike Stump1eb44332009-09-09 15:08:12 +00002855
Chris Lattnercc98eac2008-12-17 07:13:27 +00002856 // Collect the instance variables
Jordy Rosedb8264e2011-07-22 02:08:32 +00002857 SmallVector<const ObjCIvarDecl*, 32> Ivars;
Fariborz Jahanian2c18bb72010-08-20 21:21:08 +00002858 Context.DeepCollectObjCIvars(Class, true, Ivars);
Fariborz Jahanian41833352009-06-04 17:08:55 +00002859 // For each ivar, create a fresh ObjCAtDefsFieldDecl.
Fariborz Jahanian2c18bb72010-08-20 21:21:08 +00002860 for (unsigned i = 0; i < Ivars.size(); i++) {
Jordy Rosedb8264e2011-07-22 02:08:32 +00002861 const FieldDecl* ID = cast<FieldDecl>(Ivars[i]);
John McCalld226f652010-08-21 09:40:31 +00002862 RecordDecl *Record = dyn_cast<RecordDecl>(TagD);
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002863 Decl *FD = ObjCAtDefsFieldDecl::Create(Context, Record,
2864 /*FIXME: StartL=*/ID->getLocation(),
2865 ID->getLocation(),
Fariborz Jahanian41833352009-06-04 17:08:55 +00002866 ID->getIdentifier(), ID->getType(),
2867 ID->getBitWidth());
John McCalld226f652010-08-21 09:40:31 +00002868 Decls.push_back(FD);
Fariborz Jahanian41833352009-06-04 17:08:55 +00002869 }
Mike Stump1eb44332009-09-09 15:08:12 +00002870
Chris Lattnercc98eac2008-12-17 07:13:27 +00002871 // Introduce all of these fields into the appropriate scope.
Chris Lattner5f9e2722011-07-23 10:55:15 +00002872 for (SmallVectorImpl<Decl*>::iterator D = Decls.begin();
Chris Lattnercc98eac2008-12-17 07:13:27 +00002873 D != Decls.end(); ++D) {
John McCalld226f652010-08-21 09:40:31 +00002874 FieldDecl *FD = cast<FieldDecl>(*D);
Chris Lattnercc98eac2008-12-17 07:13:27 +00002875 if (getLangOptions().CPlusPlus)
2876 PushOnScopeChains(cast<FieldDecl>(FD), S);
John McCalld226f652010-08-21 09:40:31 +00002877 else if (RecordDecl *Record = dyn_cast<RecordDecl>(TagD))
Argyrios Kyrtzidis17945a02009-06-30 02:36:12 +00002878 Record->addDecl(FD);
Chris Lattnercc98eac2008-12-17 07:13:27 +00002879 }
2880}
2881
Douglas Gregor160b5632010-04-26 17:32:49 +00002882/// \brief Build a type-check a new Objective-C exception variable declaration.
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002883VarDecl *Sema::BuildObjCExceptionDecl(TypeSourceInfo *TInfo, QualType T,
2884 SourceLocation StartLoc,
2885 SourceLocation IdLoc,
2886 IdentifierInfo *Id,
Douglas Gregor160b5632010-04-26 17:32:49 +00002887 bool Invalid) {
2888 // ISO/IEC TR 18037 S6.7.3: "The type of an object with automatic storage
2889 // duration shall not be qualified by an address-space qualifier."
2890 // Since all parameters have automatic store duration, they can not have
2891 // an address space.
2892 if (T.getAddressSpace() != 0) {
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002893 Diag(IdLoc, diag::err_arg_with_address_space);
Douglas Gregor160b5632010-04-26 17:32:49 +00002894 Invalid = true;
2895 }
2896
2897 // An @catch parameter must be an unqualified object pointer type;
2898 // FIXME: Recover from "NSObject foo" by inserting the * in "NSObject *foo"?
2899 if (Invalid) {
2900 // Don't do any further checking.
Douglas Gregorbe270a02010-04-26 17:57:08 +00002901 } else if (T->isDependentType()) {
2902 // Okay: we don't know what this type will instantiate to.
Douglas Gregor160b5632010-04-26 17:32:49 +00002903 } else if (!T->isObjCObjectPointerType()) {
2904 Invalid = true;
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002905 Diag(IdLoc ,diag::err_catch_param_not_objc_type);
Douglas Gregor160b5632010-04-26 17:32:49 +00002906 } else if (T->isObjCQualifiedIdType()) {
2907 Invalid = true;
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002908 Diag(IdLoc, diag::err_illegal_qualifiers_on_catch_parm);
Douglas Gregor160b5632010-04-26 17:32:49 +00002909 }
2910
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002911 VarDecl *New = VarDecl::Create(Context, CurContext, StartLoc, IdLoc, Id,
2912 T, TInfo, SC_None, SC_None);
Douglas Gregor324b54d2010-05-03 18:51:14 +00002913 New->setExceptionVariable(true);
2914
Douglas Gregor9aab9c42011-12-10 01:22:52 +00002915 // In ARC, infer 'retaining' for variables of retainable type.
2916 if (getLangOptions().ObjCAutoRefCount && inferObjCARCLifetime(New))
2917 Invalid = true;
2918
Douglas Gregor160b5632010-04-26 17:32:49 +00002919 if (Invalid)
2920 New->setInvalidDecl();
2921 return New;
2922}
2923
John McCalld226f652010-08-21 09:40:31 +00002924Decl *Sema::ActOnObjCExceptionDecl(Scope *S, Declarator &D) {
Douglas Gregor160b5632010-04-26 17:32:49 +00002925 const DeclSpec &DS = D.getDeclSpec();
2926
2927 // We allow the "register" storage class on exception variables because
2928 // GCC did, but we drop it completely. Any other storage class is an error.
2929 if (DS.getStorageClassSpec() == DeclSpec::SCS_register) {
2930 Diag(DS.getStorageClassSpecLoc(), diag::warn_register_objc_catch_parm)
2931 << FixItHint::CreateRemoval(SourceRange(DS.getStorageClassSpecLoc()));
2932 } else if (DS.getStorageClassSpec() != DeclSpec::SCS_unspecified) {
2933 Diag(DS.getStorageClassSpecLoc(), diag::err_storage_spec_on_catch_parm)
2934 << DS.getStorageClassSpec();
2935 }
2936 if (D.getDeclSpec().isThreadSpecified())
2937 Diag(D.getDeclSpec().getThreadSpecLoc(), diag::err_invalid_thread);
2938 D.getMutableDeclSpec().ClearStorageClassSpecs();
2939
2940 DiagnoseFunctionSpecifiers(D);
2941
2942 // Check that there are no default arguments inside the type of this
2943 // exception object (C++ only).
2944 if (getLangOptions().CPlusPlus)
2945 CheckExtraCXXDefaultArguments(D);
2946
Argyrios Kyrtzidis32153982011-06-28 03:01:15 +00002947 TypeSourceInfo *TInfo = GetTypeForDeclarator(D, S);
John McCallbf1a0282010-06-04 23:28:52 +00002948 QualType ExceptionType = TInfo->getType();
Douglas Gregor160b5632010-04-26 17:32:49 +00002949
Abramo Bagnaraff676cb2011-03-08 08:55:46 +00002950 VarDecl *New = BuildObjCExceptionDecl(TInfo, ExceptionType,
2951 D.getSourceRange().getBegin(),
2952 D.getIdentifierLoc(),
2953 D.getIdentifier(),
Douglas Gregor160b5632010-04-26 17:32:49 +00002954 D.isInvalidType());
2955
2956 // Parameter declarators cannot be qualified (C++ [dcl.meaning]p1).
2957 if (D.getCXXScopeSpec().isSet()) {
2958 Diag(D.getIdentifierLoc(), diag::err_qualified_objc_catch_parm)
2959 << D.getCXXScopeSpec().getRange();
2960 New->setInvalidDecl();
2961 }
2962
2963 // Add the parameter declaration into this scope.
John McCalld226f652010-08-21 09:40:31 +00002964 S->AddDecl(New);
Douglas Gregor160b5632010-04-26 17:32:49 +00002965 if (D.getIdentifier())
2966 IdResolver.AddDecl(New);
2967
2968 ProcessDeclAttributes(S, New, D);
2969
2970 if (New->hasAttr<BlocksAttr>())
2971 Diag(New->getLocation(), diag::err_block_on_nonlocal);
John McCalld226f652010-08-21 09:40:31 +00002972 return New;
Douglas Gregor4e6c0d12010-04-23 23:01:43 +00002973}
Fariborz Jahanian786cd152010-04-27 17:18:58 +00002974
2975/// CollectIvarsToConstructOrDestruct - Collect those ivars which require
Fariborz Jahaniane4498c62010-04-28 16:11:27 +00002976/// initialization.
Fariborz Jahanian2c18bb72010-08-20 21:21:08 +00002977void Sema::CollectIvarsToConstructOrDestruct(ObjCInterfaceDecl *OI,
Chris Lattner5f9e2722011-07-23 10:55:15 +00002978 SmallVectorImpl<ObjCIvarDecl*> &Ivars) {
Fariborz Jahanian2c18bb72010-08-20 21:21:08 +00002979 for (ObjCIvarDecl *Iv = OI->all_declared_ivar_begin(); Iv;
2980 Iv= Iv->getNextIvar()) {
Fariborz Jahanian786cd152010-04-27 17:18:58 +00002981 QualType QT = Context.getBaseElementType(Iv->getType());
Douglas Gregor68dd3ee2010-05-20 02:24:22 +00002982 if (QT->isRecordType())
Fariborz Jahanian2c18bb72010-08-20 21:21:08 +00002983 Ivars.push_back(Iv);
Fariborz Jahanian786cd152010-04-27 17:18:58 +00002984 }
2985}
Fariborz Jahaniane4498c62010-04-28 16:11:27 +00002986
Fariborz Jahanian3fe10412010-07-22 18:24:20 +00002987void Sema::DiagnoseUseOfUnimplementedSelectors() {
Douglas Gregor5b9dc7c2011-07-28 14:54:22 +00002988 // Load referenced selectors from the external source.
2989 if (ExternalSource) {
2990 SmallVector<std::pair<Selector, SourceLocation>, 4> Sels;
2991 ExternalSource->ReadReferencedSelectors(Sels);
2992 for (unsigned I = 0, N = Sels.size(); I != N; ++I)
2993 ReferencedSelectors[Sels[I].first] = Sels[I].second;
2994 }
2995
Fariborz Jahanian8b789132011-02-04 23:19:27 +00002996 // Warning will be issued only when selector table is
2997 // generated (which means there is at lease one implementation
2998 // in the TU). This is to match gcc's behavior.
2999 if (ReferencedSelectors.empty() ||
3000 !Context.AnyObjCImplementation())
Fariborz Jahanian3fe10412010-07-22 18:24:20 +00003001 return;
3002 for (llvm::DenseMap<Selector, SourceLocation>::iterator S =
3003 ReferencedSelectors.begin(),
3004 E = ReferencedSelectors.end(); S != E; ++S) {
3005 Selector Sel = (*S).first;
3006 if (!LookupImplementedMethodInGlobalPool(Sel))
3007 Diag((*S).second, diag::warn_unimplemented_selector) << Sel;
3008 }
3009 return;
3010}