SemaObjCProperty.cpp revision c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44e
19d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//===--- SemaObjCProperty.cpp - Semantic Analysis for ObjC @property ------===// 29d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// 39d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// The LLVM Compiler Infrastructure 49d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// 59d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// This file is distributed under the University of Illinois Open Source 69d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// License. See LICENSE.TXT for details. 79d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// 89d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//===----------------------------------------------------------------------===// 99d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// 109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// This file implements semantic analysis for Objective C @property and 119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// @synthesize declarations. 129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek// 139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek//===----------------------------------------------------------------------===// 149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek#include "Sema.h" 1617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian#include "SemaInit.h" 1717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian#include "clang/AST/ExprObjC.h" 189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekusing namespace clang; 209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 2128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===// 2228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek// Grammar actions. 2328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===// 2428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 2528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted KremenekSema::DeclPtrTy Sema::ActOnProperty(Scope *S, SourceLocation AtLoc, 2628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek FieldDeclarator &FD, 2728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCDeclSpec &ODS, 2828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Selector GetterSel, 2928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Selector SetterSel, 3028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek DeclPtrTy ClassCategory, 3128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek bool *isOverridingProperty, 3228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek tok::ObjCKeywordKind MethodImplKind) { 3328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek unsigned Attributes = ODS.getPropertyAttributes(); 3428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek bool isReadWrite = ((Attributes & ObjCDeclSpec::DQ_PR_readwrite) || 3528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // default is readwrite! 3628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek !(Attributes & ObjCDeclSpec::DQ_PR_readonly)); 3728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // property is defaulted to 'assign' if it is readwrite and is 3828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // not retain or copy 3928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek bool isAssign = ((Attributes & ObjCDeclSpec::DQ_PR_assign) || 4028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek (isReadWrite && 4128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek !(Attributes & ObjCDeclSpec::DQ_PR_retain) && 4228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek !(Attributes & ObjCDeclSpec::DQ_PR_copy))); 43e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 4428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek QualType T = GetTypeForDeclarator(FD.D, S); 4528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (T->isReferenceType()) { 4628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(AtLoc, diag::error_reference_property); 4728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 4828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 49e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Proceed with constructing the ObjCPropertDecls. 50e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCContainerDecl *ClassDecl = 51e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek cast<ObjCContainerDecl>(ClassCategory.getAs<Decl>()); 52e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 5328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (ObjCCategoryDecl *CDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl)) 54e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if (CDecl->IsClassExtension()) 55e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek return HandlePropertyInClassExtension(S, CDecl, AtLoc, 56e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek FD, GetterSel, SetterSel, 57e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek isAssign, isReadWrite, 58e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Attributes, 59e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek isOverridingProperty, T, 60e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek MethodImplKind); 61e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 62842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian DeclPtrTy Res = DeclPtrTy::make(CreatePropertyDecl(S, ClassDecl, AtLoc, FD, 63e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek GetterSel, SetterSel, 64e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek isAssign, isReadWrite, 65e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Attributes, T, MethodImplKind)); 66842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian // Validate the attributes on the @property. 67842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian CheckObjCPropertyAttributes(Res, AtLoc, Attributes); 68842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian return Res; 69e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek} 70e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 71e3d67bcc7b89b335dbcd097a299658057ea31097Ted KremenekSema::DeclPtrTy 72e3d67bcc7b89b335dbcd097a299658057ea31097Ted KremenekSema::HandlePropertyInClassExtension(Scope *S, ObjCCategoryDecl *CDecl, 73e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek SourceLocation AtLoc, FieldDeclarator &FD, 74e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Selector GetterSel, Selector SetterSel, 75e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek const bool isAssign, 76e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek const bool isReadWrite, 77e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek const unsigned Attributes, 78e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek bool *isOverridingProperty, 79e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek QualType T, 80e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek tok::ObjCKeywordKind MethodImplKind) { 81e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 82e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Diagnose if this property is already in continuation class. 83e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek DeclContext *DC = cast<DeclContext>(CDecl); 84e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek IdentifierInfo *PropertyId = FD.D.getIdentifier(); 85894ae6ad081465a3b168db7e60ab3f94e3730b57Ted Kremenek 869f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek if (ObjCPropertyDecl *prevDecl = 879f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek ObjCPropertyDecl::findPropertyDecl(DC, PropertyId)) { 88e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Diag(AtLoc, diag::err_duplicate_property); 89894ae6ad081465a3b168db7e60ab3f94e3730b57Ted Kremenek Diag(prevDecl->getLocation(), diag::note_property_declare); 90e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek return DeclPtrTy(); 91e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek } 92e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 93e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Create a new ObjCPropertyDecl with the DeclContext being 94e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // the class extension. 95e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCPropertyDecl *PDecl = 96e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCPropertyDecl::Create(Context, DC, FD.D.getIdentifierLoc(), 97e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PropertyId, AtLoc, T); 9822f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian if (Attributes & ObjCDeclSpec::DQ_PR_readonly) 9922f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly); 10022f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian if (Attributes & ObjCDeclSpec::DQ_PR_readwrite) 10122f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite); 10222f757b38da3fc9f17ea9e99524064fdfbca3456Fariborz Jahanian 103e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek DC->addDecl(PDecl); 104e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 105e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // We need to look in the @interface to see if the @property was 106e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // already declared. 107e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCInterfaceDecl *CCPrimary = CDecl->getClassInterface(); 108e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if (!CCPrimary) { 109e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Diag(CDecl->getLocation(), diag::err_continuation_class); 110e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek *isOverridingProperty = true; 111e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek return DeclPtrTy(); 112e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek } 113e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 114e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Find the property in continuation class's primary class only. 115e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCPropertyDecl *PIDecl = 116e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek CCPrimary->FindPropertyVisibleInPrimaryClass(PropertyId); 117e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 118e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if (!PIDecl) { 119e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // No matching property found in the primary class. Just fall thru 120e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // and add property to continuation class's primary class. 121e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCPropertyDecl *PDecl = 122e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek CreatePropertyDecl(S, CCPrimary, AtLoc, 123e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek FD, GetterSel, SetterSel, isAssign, isReadWrite, 124e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Attributes, T, MethodImplKind); 125e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 126e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // A case of continuation class adding a new property in the class. This 127e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // is not what it was meant for. However, gcc supports it and so should we. 128e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Make sure setter/getters are declared here. 129e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ProcessPropertyDecl(PDecl, CCPrimary); 130e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek return DeclPtrTy::make(PDecl); 131e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 132e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek } 133e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 134e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // The property 'PIDecl's readonly attribute will be over-ridden 135e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // with continuation class's readwrite property attribute! 136e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek unsigned PIkind = PIDecl->getPropertyAttributes(); 137e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if (isReadWrite && (PIkind & ObjCPropertyDecl::OBJC_PR_readonly)) { 138e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek unsigned retainCopyNonatomic = 139e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek (ObjCPropertyDecl::OBJC_PR_retain | 140e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCPropertyDecl::OBJC_PR_copy | 141e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCPropertyDecl::OBJC_PR_nonatomic); 142e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if ((Attributes & retainCopyNonatomic) != 143e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek (PIkind & retainCopyNonatomic)) { 144e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Diag(AtLoc, diag::warn_property_attr_mismatch); 145e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Diag(PIDecl->getLocation(), diag::note_property_declare); 146e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek } 1479944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek DeclContext *DC = cast<DeclContext>(CCPrimary); 1489944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek if (!ObjCPropertyDecl::findPropertyDecl(DC, 1499944c769b69b1904a7b16d3ce10fbdc9c67c764fTed Kremenek PIDecl->getDeclName().getAsIdentifierInfo())) { 150e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Protocol is not in the primary class. Must build one for it. 151e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCDeclSpec ProtocolPropertyODS; 152e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // FIXME. Assuming that ObjCDeclSpec::ObjCPropertyAttributeKind 153e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // and ObjCPropertyDecl::PropertyAttributeKind have identical 154e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // values. Should consolidate both into one enum type. 155e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ProtocolPropertyODS. 156e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek setPropertyAttributes((ObjCDeclSpec::ObjCPropertyAttributeKind) 157e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIkind); 158e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 159e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek DeclPtrTy ProtocolPtrTy = 160e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ActOnProperty(S, AtLoc, FD, ProtocolPropertyODS, 161e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIDecl->getGetterName(), 162e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIDecl->getSetterName(), 163e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek DeclPtrTy::make(CCPrimary), isOverridingProperty, 164e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek MethodImplKind); 165e9686572f17968048a5e097254f85c202ea44aecTed Kremenek PIDecl = cast<ObjCPropertyDecl>(ProtocolPtrTy.getAs<Decl>()); 16628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 167e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIDecl->makeitReadWriteAttribute(); 168e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_retain) 169e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain); 170e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_copy) 171e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy); 172e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PIDecl->setSetterName(SetterSel); 173e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek } else { 174e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Diag(AtLoc, diag::err_use_continuation_class) 175e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek << CCPrimary->getDeclName(); 176e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Diag(PIDecl->getLocation(), diag::note_property_declare); 177e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek } 178e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek *isOverridingProperty = true; 179e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek // Make sure setter decl is synthesized, and added to primary class's list. 180e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ProcessPropertyDecl(PIDecl, CCPrimary); 181e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek return DeclPtrTy(); 182e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek} 183e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 184e3d67bcc7b89b335dbcd097a299658057ea31097Ted KremenekObjCPropertyDecl *Sema::CreatePropertyDecl(Scope *S, 185e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek ObjCContainerDecl *CDecl, 186e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek SourceLocation AtLoc, 187e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek FieldDeclarator &FD, 188e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Selector GetterSel, 189e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek Selector SetterSel, 190e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek const bool isAssign, 191e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek const bool isReadWrite, 192e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek const unsigned Attributes, 193e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek QualType T, 194e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek tok::ObjCKeywordKind MethodImplKind){ 195e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 196e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek IdentifierInfo *PropertyId = FD.D.getIdentifier(); 19728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 19828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Issue a warning if property is 'assign' as default and its object, which is 19928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // gc'able conforms to NSCopying protocol 20028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (getLangOptions().getGCMode() != LangOptions::NonGC && 20128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek isAssign && !(Attributes & ObjCDeclSpec::DQ_PR_assign)) 202c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall if (const ObjCObjectPointerType *ObjPtrTy = 203c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall T->getAs<ObjCObjectPointerType>()) { 204c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall ObjCInterfaceDecl *IDecl = ObjPtrTy->getObjectType()->getInterface(); 205c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall if (IDecl) 206c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall if (ObjCProtocolDecl* PNSCopying = 207c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall LookupProtocol(&Context.Idents.get("NSCopying"), AtLoc)) 208c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall if (IDecl->ClassImplementsProtocol(PNSCopying, true)) 209c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall Diag(AtLoc, diag::warn_implements_nscopying) << PropertyId; 21028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 211c12c5bba6ceb6acd4e51e7a0fc03257da9cfd44eJohn McCall if (T->isObjCObjectType()) 21228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(FD.D.getIdentifierLoc(), diag::err_statically_allocated_object); 21328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 214e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek DeclContext *DC = cast<DeclContext>(CDecl); 21528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCPropertyDecl *PDecl = ObjCPropertyDecl::Create(Context, DC, 21628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek FD.D.getIdentifierLoc(), 217e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek PropertyId, AtLoc, T); 218e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek 2199f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek if (ObjCPropertyDecl *prevDecl = 2209f550ff05d496e6b9480e5619a21d9da0c9e27c1Ted Kremenek ObjCPropertyDecl::findPropertyDecl(DC, PropertyId)) { 22128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PDecl->getLocation(), diag::err_duplicate_property); 222894ae6ad081465a3b168db7e60ab3f94e3730b57Ted Kremenek Diag(prevDecl->getLocation(), diag::note_property_declare); 22328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setInvalidDecl(); 22428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 22528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek else 22628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek DC->addDecl(PDecl); 22728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 22828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (T->isArrayType() || T->isFunctionType()) { 22928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(AtLoc, diag::err_property_type) << T; 23028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setInvalidDecl(); 23128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 23228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 23328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ProcessDeclAttributes(S, PDecl, FD.D); 23428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 23528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Regardless of setter/getter attribute, we save the default getter/setter 23628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // selector names in anticipation of declaration of setter/getter methods. 23728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setGetterName(GetterSel); 23828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setSetterName(SetterSel); 23928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 24028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_readonly) 24128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readonly); 24228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 24328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_getter) 24428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_getter); 24528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 24628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_setter) 24728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_setter); 24828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 24928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (isReadWrite) 25028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_readwrite); 25128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 25228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_retain) 25328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_retain); 25428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 25528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_copy) 25628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_copy); 25728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 25828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (isAssign) 25928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_assign); 26028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 26128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_nonatomic) 26228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyAttributes(ObjCPropertyDecl::OBJC_PR_nonatomic); 26328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 26428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (MethodImplKind == tok::objc_required) 26528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyImplementation(ObjCPropertyDecl::Required); 26628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek else if (MethodImplKind == tok::objc_optional) 26728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PDecl->setPropertyImplementation(ObjCPropertyDecl::Optional); 26828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 269e3d67bcc7b89b335dbcd097a299658057ea31097Ted Kremenek return PDecl; 27028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek} 27128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 27228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 27328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// ActOnPropertyImplDecl - This routine performs semantic checks and 27428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// builds the AST node for a property implementation declaration; declared 27528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// as @synthesize or @dynamic. 27628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek/// 27717cb326cb62a59f53d92236394af40eaae4eddbdFariborz JahanianSema::DeclPtrTy Sema::ActOnPropertyImplDecl(Scope *S, 27817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SourceLocation AtLoc, 27928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek SourceLocation PropertyLoc, 28028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek bool Synthesize, 28128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek DeclPtrTy ClassCatImpDecl, 28228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IdentifierInfo *PropertyId, 28328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IdentifierInfo *PropertyIvar) { 284e9686572f17968048a5e097254f85c202ea44aecTed Kremenek ObjCContainerDecl *ClassImpDecl = 285e9686572f17968048a5e097254f85c202ea44aecTed Kremenek cast_or_null<ObjCContainerDecl>(ClassCatImpDecl.getAs<Decl>()); 28628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Make sure we have a context for the property implementation declaration. 28728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!ClassImpDecl) { 28828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(AtLoc, diag::error_missing_property_context); 28928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 29028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 29128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCPropertyDecl *property = 0; 29228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCInterfaceDecl* IDecl = 0; 29328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Find the class or category class where this property must have 29428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // a declaration. 29528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCImplementationDecl *IC = 0; 29628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCCategoryImplDecl* CatImplClass = 0; 29728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if ((IC = dyn_cast<ObjCImplementationDecl>(ClassImpDecl))) { 29828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IDecl = IC->getClassInterface(); 29928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // We always synthesize an interface for an implementation 30028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // without an interface decl. So, IDecl is always non-zero. 30128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek assert(IDecl && 30228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek "ActOnPropertyImplDecl - @implementation without @interface"); 30328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 30428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Look for this property declaration in the @implementation's @interface 30528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek property = IDecl->FindPropertyDeclaration(PropertyId); 30628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!property) { 30728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_bad_property_decl) << IDecl->getDeclName(); 30828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 30928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 31028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (const ObjCCategoryDecl *CD = 31128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek dyn_cast<ObjCCategoryDecl>(property->getDeclContext())) { 31228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!CD->IsClassExtension()) { 31328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_category_property) << CD->getDeclName(); 31428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(property->getLocation(), diag::note_property_declare); 31528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 31628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 31728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 31828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } else if ((CatImplClass = dyn_cast<ObjCCategoryImplDecl>(ClassImpDecl))) { 31928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Synthesize) { 32028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(AtLoc, diag::error_synthesize_category_decl); 32128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 32228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 32328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IDecl = CatImplClass->getClassInterface(); 32428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!IDecl) { 32528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(AtLoc, diag::error_missing_property_interface); 32628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 32728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 32828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCCategoryDecl *Category = 32928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IDecl->FindCategoryDeclaration(CatImplClass->getIdentifier()); 33028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 33128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // If category for this implementation not found, it is an error which 33228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // has already been reported eralier. 33328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!Category) 33428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 33528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Look for this property declaration in @implementation's category 33628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek property = Category->FindPropertyDeclaration(PropertyId); 33728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!property) { 33828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_bad_category_property_decl) 33928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << Category->getDeclName(); 34028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 34128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 34228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } else { 34328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(AtLoc, diag::error_bad_property_context); 34428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 34528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 34628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCIvarDecl *Ivar = 0; 34728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Check that we have a valid, previously declared ivar for @synthesize 34828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Synthesize) { 34928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // @synthesize 35028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!PropertyIvar) 35128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PropertyIvar = PropertyId; 35228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek QualType PropType = Context.getCanonicalType(property->getType()); 35328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Check that this is a previously declared 'ivar' in 'IDecl' interface 35428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCInterfaceDecl *ClassDeclared; 35528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Ivar = IDecl->lookupInstanceVariable(PropertyIvar, ClassDeclared); 35628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!Ivar) { 35729fa69addf8b19871e7866c32c92b5d142c4bfbcDaniel Dunbar Ivar = ObjCIvarDecl::Create(Context, ClassImpDecl, PropertyLoc, 35828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PropertyIvar, PropType, /*Dinfo=*/0, 3592846b2b47029c877357438d16c73911c1510be6cFariborz Jahanian ObjCIvarDecl::Protected, 36028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek (Expr *)0); 36129fa69addf8b19871e7866c32c92b5d142c4bfbcDaniel Dunbar ClassImpDecl->addDecl(Ivar); 36228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IDecl->makeDeclVisibleInContext(Ivar, false); 36328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek property->setPropertyIvarDecl(Ivar); 36428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 36528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!getLangOptions().ObjCNonFragileABI) 36628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_missing_property_ivar_decl) << PropertyId; 36728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Note! I deliberately want it to fall thru so, we have a 36828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // a property implementation and to avoid future warnings. 36928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } else if (getLangOptions().ObjCNonFragileABI && 37028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ClassDeclared != IDecl) { 37128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_ivar_in_superclass_use) 37228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << property->getDeclName() << Ivar->getDeclName() 37328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << ClassDeclared->getDeclName(); 37428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Ivar->getLocation(), diag::note_previous_access_declaration) 37528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << Ivar << Ivar->getNameAsCString(); 37628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Note! I deliberately want it to fall thru so more errors are caught. 37728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 37828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek QualType IvarType = Context.getCanonicalType(Ivar->getType()); 37928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 38028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Check that type of property and its ivar are type compatible. 38128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (PropType != IvarType) { 38228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (CheckAssignmentConstraints(PropType, IvarType) != Compatible) { 38328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_property_ivar_type) 384f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek << property->getDeclName() << PropType 385f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek << Ivar->getDeclName() << IvarType; 386f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek Diag(Ivar->getLocation(), diag::note_ivar_decl); 38728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Note! I deliberately want it to fall thru so, we have a 38828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // a property implementation and to avoid future warnings. 38928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 39028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 39128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // FIXME! Rules for properties are somewhat different that those 39228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // for assignments. Use a new routine to consolidate all cases; 39328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // specifically for property redeclarations as well as for ivars. 39428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek QualType lhsType =Context.getCanonicalType(PropType).getUnqualifiedType(); 39528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek QualType rhsType =Context.getCanonicalType(IvarType).getUnqualifiedType(); 39628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (lhsType != rhsType && 39728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek lhsType->isArithmeticType()) { 39828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_property_ivar_type) 399f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek << property->getDeclName() << PropType 400f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek << Ivar->getDeclName() << IvarType; 401f921a4868cf1876636f6684e7f68697b18c0cb47Ted Kremenek Diag(Ivar->getLocation(), diag::note_ivar_decl); 40228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Fall thru - see previous comment 40328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 40428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // __weak is explicit. So it works on Canonical type. 40528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (PropType.isObjCGCWeak() && !IvarType.isObjCGCWeak() && 40628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek getLangOptions().getGCMode() != LangOptions::NonGC) { 40728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_weak_property) 40828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << property->getDeclName() << Ivar->getDeclName(); 40928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Fall thru - see previous comment 41028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 41128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if ((property->getType()->isObjCObjectPointerType() || 41228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PropType.isObjCGCStrong()) && IvarType.isObjCGCWeak() && 41328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek getLangOptions().getGCMode() != LangOptions::NonGC) { 41428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_strong_property) 41528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << property->getDeclName() << Ivar->getDeclName(); 41628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Fall thru - see previous comment 41728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 41828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 41928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } else if (PropertyIvar) 42028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // @dynamic 42128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_dynamic_property_ivar_decl); 42228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek assert (property && "ActOnPropertyImplDecl - property declaration missing"); 42328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCPropertyImplDecl *PIDecl = 42428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCPropertyImplDecl::Create(Context, CurContext, AtLoc, PropertyLoc, 42528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek property, 42628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek (Synthesize ? 42728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCPropertyImplDecl::Synthesize 42828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek : ObjCPropertyImplDecl::Dynamic), 42928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Ivar); 43017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian if (ObjCMethodDecl *getterMethod = property->getGetterMethodDecl()) { 43117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian getterMethod->createImplicitParams(Context, IDecl); 43217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian if (getLangOptions().CPlusPlus && Synthesize) { 43317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian // For Objective-C++, need to synthesize the AST for the IVAR object to be 43417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian // returned by the getter as it must conform to C++'s copy-return rules. 43517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian // FIXME. Eventually we want to do this for Objective-C as well. 43617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian ImplicitParamDecl *SelfDecl = getterMethod->getSelfDecl(); 43717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian DeclRefExpr *SelfExpr = 43817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian new (Context) DeclRefExpr(SelfDecl,SelfDecl->getType(), 43917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SourceLocation()); 44017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian Expr *IvarRefExpr = 44117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc, 44217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SelfExpr, true, true); 44317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian OwningExprResult Res = 44417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian PerformCopyInitialization(InitializedEntity::InitializeResult( 4453c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor SourceLocation(), 4463c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor getterMethod->getResultType(), 4473c9034cb7ff1d6c1e4ecd1b44c98f553df013c7cDouglas Gregor /*NRVO=*/false), 44817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SourceLocation(), 44917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian Owned(IvarRefExpr)); 45017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian if (!Res.isInvalid()) { 45117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian Expr *ResExpr = Res.takeAs<Expr>(); 45217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian if (ResExpr) 45317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian ResExpr = MaybeCreateCXXExprWithTemporaries(ResExpr); 45417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian PIDecl->setGetterCXXConstructor(ResExpr); 45517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian } 45617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian } 45717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian } 45817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian if (ObjCMethodDecl *setterMethod = property->getSetterMethodDecl()) { 45917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian setterMethod->createImplicitParams(Context, IDecl); 46017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian if (getLangOptions().CPlusPlus && Synthesize) { 46117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian // FIXME. Eventually we want to do this for Objective-C as well. 46217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian ImplicitParamDecl *SelfDecl = setterMethod->getSelfDecl(); 46317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian DeclRefExpr *SelfExpr = 46417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian new (Context) DeclRefExpr(SelfDecl,SelfDecl->getType(), 46517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SourceLocation()); 46617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian Expr *lhs = 46717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian new (Context) ObjCIvarRefExpr(Ivar, Ivar->getType(), AtLoc, 46817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SelfExpr, true, true); 46917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian ObjCMethodDecl::param_iterator P = setterMethod->param_begin(); 47017cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian ParmVarDecl *Param = (*P); 47117cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian Expr *rhs = new (Context) DeclRefExpr(Param,Param->getType(), 47217cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian SourceLocation()); 47317cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian OwningExprResult Res = BuildBinOp(S, SourceLocation(), 47417cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian BinaryOperator::Assign, lhs, rhs); 47517cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian PIDecl->setSetterCXXAssignment(Res.takeAs<Expr>()); 47617cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian } 47717cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian } 47817cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanian 47928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (IC) { 48028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Synthesize) 48128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (ObjCPropertyImplDecl *PPIDecl = 48228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IC->FindPropertyImplIvarDecl(PropertyIvar)) { 48328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_duplicate_ivar_use) 48428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier() 48528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << PropertyIvar; 48628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PPIDecl->getLocation(), diag::note_previous_use); 48728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 48828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 48928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (ObjCPropertyImplDecl *PPIDecl 49028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek = IC->FindPropertyImplDecl(PropertyId)) { 49128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_property_implemented) << PropertyId; 49228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PPIDecl->getLocation(), diag::note_previous_declaration); 49328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 49428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 49528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek IC->addPropertyImplementation(PIDecl); 49628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } else { 49728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Synthesize) 49828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (ObjCPropertyImplDecl *PPIDecl = 49928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek CatImplClass->FindPropertyImplIvarDecl(PropertyIvar)) { 50028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_duplicate_ivar_use) 50128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << PropertyId << PPIDecl->getPropertyDecl()->getIdentifier() 50228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << PropertyIvar; 50328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PPIDecl->getLocation(), diag::note_previous_use); 50428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 50528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 50628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (ObjCPropertyImplDecl *PPIDecl = 50728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek CatImplClass->FindPropertyImplDecl(PropertyId)) { 50828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PropertyLoc, diag::error_property_implemented) << PropertyId; 50928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(PPIDecl->getLocation(), diag::note_previous_declaration); 51028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy(); 51128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 51228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek CatImplClass->addPropertyImplementation(PIDecl); 51328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 51428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 51528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek return DeclPtrTy::make(PIDecl); 51628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek} 51728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 51828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===// 51928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek// Helper methods. 52028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek//===----------------------------------------------------------------------===// 52128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek 5229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// DiagnosePropertyMismatch - Compares two properties for their 5239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// attributes and types and warns on a variety of inconsistencies. 5249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// 5259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid 5269d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::DiagnosePropertyMismatch(ObjCPropertyDecl *Property, 5279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *SuperProperty, 5289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek const IdentifierInfo *inheritedName) { 5299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl::PropertyAttributeKind CAttr = 5309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Property->getPropertyAttributes(); 5319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl::PropertyAttributeKind SAttr = 5329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SuperProperty->getPropertyAttributes(); 5339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((CAttr & ObjCPropertyDecl::OBJC_PR_readonly) 5349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek && (SAttr & ObjCPropertyDecl::OBJC_PR_readwrite)) 5359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_readonly_property) 5369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getDeclName() << inheritedName; 5379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((CAttr & ObjCPropertyDecl::OBJC_PR_copy) 5389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek != (SAttr & ObjCPropertyDecl::OBJC_PR_copy)) 5399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_property_attribute) 5409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getDeclName() << "copy" << inheritedName; 5419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek else if ((CAttr & ObjCPropertyDecl::OBJC_PR_retain) 5429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek != (SAttr & ObjCPropertyDecl::OBJC_PR_retain)) 5439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_property_attribute) 5449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getDeclName() << "retain" << inheritedName; 5459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 5469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((CAttr & ObjCPropertyDecl::OBJC_PR_nonatomic) 5479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek != (SAttr & ObjCPropertyDecl::OBJC_PR_nonatomic)) 5489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_property_attribute) 5499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getDeclName() << "atomic" << inheritedName; 5509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Property->getSetterName() != SuperProperty->getSetterName()) 5519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_property_attribute) 5529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getDeclName() << "setter" << inheritedName; 5539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Property->getGetterName() != SuperProperty->getGetterName()) 5549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_property_attribute) 5559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getDeclName() << "getter" << inheritedName; 5569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 5579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek QualType LHSType = 5589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Context.getCanonicalType(SuperProperty->getType()); 5599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek QualType RHSType = 5609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Context.getCanonicalType(Property->getType()); 5619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 5629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!Context.typesAreCompatible(LHSType, RHSType)) { 5639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // FIXME: Incorporate this test with typesAreCompatible. 5649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (LHSType->isObjCQualifiedIdType() && RHSType->isObjCQualifiedIdType()) 5659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Context.ObjCQualifiedIdTypesAreCompatible(LHSType, RHSType, false)) 5669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return; 5679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::warn_property_types_are_incompatible) 5689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getType() << SuperProperty->getType() << inheritedName; 5699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 5709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 5719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 5729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekbool Sema::DiagnosePropertyAccessorMismatch(ObjCPropertyDecl *property, 5739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl *GetterMethod, 5749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SourceLocation Loc) { 5759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (GetterMethod && 5769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek GetterMethod->getResultType() != property->getType()) { 5779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek AssignConvertType result = Incompatible; 5789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (property->getType()->isObjCObjectPointerType()) 5799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek result = CheckAssignmentConstraints(GetterMethod->getResultType(), 5809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getType()); 5819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (result != Compatible) { 5829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Loc, diag::warn_accessor_property_type_mismatch) 5839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << property->getDeclName() 5849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << GetterMethod->getSelector(); 5859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(GetterMethod->getLocation(), diag::note_declared_at); 5869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return true; 5879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 5889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 5899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 5909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 5919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 5929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// ComparePropertiesInBaseAndSuper - This routine compares property 5939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// declarations in base and its super class, if any, and issues 5949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// diagnostics in a variety of inconsistant situations. 5959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// 5969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::ComparePropertiesInBaseAndSuper(ObjCInterfaceDecl *IDecl) { 5979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCInterfaceDecl *SDecl = IDecl->getSuperClass(); 5989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!SDecl) 5999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return; 6009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // FIXME: O(N^2) 6019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::prop_iterator S = SDecl->prop_begin(), 6029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = SDecl->prop_end(); S != E; ++S) { 6039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *SuperPDecl = (*S); 6049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Does property in super class has declaration in current class? 6059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::prop_iterator I = IDecl->prop_begin(), 6069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->prop_end(); I != E; ++I) { 6079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *PDecl = (*I); 6089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (SuperPDecl->getIdentifier() == PDecl->getIdentifier()) 6099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek DiagnosePropertyMismatch(PDecl, SuperPDecl, 6109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SDecl->getIdentifier()); 6119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 6149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 6159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// MatchOneProtocolPropertiesInClass - This routine goes thru the list 6169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// of properties declared in a protocol and compares their attribute against 6179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// the same property declared in the class or category. 6189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid 6199d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::MatchOneProtocolPropertiesInClass(Decl *CDecl, 6209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCProtocolDecl *PDecl) { 6219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl); 6229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!IDecl) { 6239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Category 6249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl); 6259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek assert (CatDecl && "MatchOneProtocolPropertiesInClass"); 6269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!CatDecl->IsClassExtension()) 6279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(), 6289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = PDecl->prop_end(); P != E; ++P) { 6299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Pr = (*P); 6309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCCategoryDecl::prop_iterator CP, CE; 6319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Is this property already in category's list of properties? 6322d2f9368d35b3628c7e3b4563f74849a0f901a00Ted Kremenek for (CP = CatDecl->prop_begin(), CE = CatDecl->prop_end(); CP!=CE; ++CP) 6339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((*CP)->getIdentifier() == Pr->getIdentifier()) 6349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek break; 6359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (CP != CE) 6369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Property protocol already exist in class. Diagnose any mismatch. 6379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek DiagnosePropertyMismatch((*CP), Pr, PDecl->getIdentifier()); 6389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return; 6409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(), 6429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = PDecl->prop_end(); P != E; ++P) { 6439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Pr = (*P); 6449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCInterfaceDecl::prop_iterator CP, CE; 6459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Is this property already in class's list of properties? 6469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (CP = IDecl->prop_begin(), CE = IDecl->prop_end(); CP != CE; ++CP) 6479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((*CP)->getIdentifier() == Pr->getIdentifier()) 6489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek break; 6499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (CP != CE) 6509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Property protocol already exist in class. Diagnose any mismatch. 6519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek DiagnosePropertyMismatch((*CP), Pr, PDecl->getIdentifier()); 6529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 6549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 6559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// CompareProperties - This routine compares properties 6569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// declared in 'ClassOrProtocol' objects (which can be a class or an 6579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// inherited protocol with the list of properties for class/category 'CDecl' 6589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// 6599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::CompareProperties(Decl *CDecl, 6609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek DeclPtrTy ClassOrProtocol) { 6619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Decl *ClassDecl = ClassOrProtocol.getAs<Decl>(); 6629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCInterfaceDecl *IDecl = dyn_cast_or_null<ObjCInterfaceDecl>(CDecl); 6639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 6649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!IDecl) { 6659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Category 6669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCCategoryDecl *CatDecl = static_cast<ObjCCategoryDecl*>(CDecl); 6679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek assert (CatDecl && "CompareProperties"); 6689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCCategoryDecl *MDecl = dyn_cast<ObjCCategoryDecl>(ClassDecl)) { 6699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCCategoryDecl::protocol_iterator P = MDecl->protocol_begin(), 6709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = MDecl->protocol_end(); P != E; ++P) 6719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Match properties of category with those of protocol (*P) 6729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek MatchOneProtocolPropertiesInClass(CatDecl, *P); 6739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 6749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Go thru the list of protocols for this category and recursively match 6759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // their properties with those in the category. 6769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCCategoryDecl::protocol_iterator P = CatDecl->protocol_begin(), 6779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = CatDecl->protocol_end(); P != E; ++P) 6789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CompareProperties(CatDecl, DeclPtrTy::make(*P)); 6799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } else { 6809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl); 6819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(), 6829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = MD->protocol_end(); P != E; ++P) 6839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek MatchOneProtocolPropertiesInClass(CatDecl, *P); 6849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return; 6869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 6879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 6889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCInterfaceDecl *MDecl = dyn_cast<ObjCInterfaceDecl>(ClassDecl)) { 6899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::protocol_iterator P = MDecl->protocol_begin(), 6909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = MDecl->protocol_end(); P != E; ++P) 6919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Match properties of class IDecl with those of protocol (*P). 6929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek MatchOneProtocolPropertiesInClass(IDecl, *P); 6939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 6949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Go thru the list of protocols for this class and recursively match 6959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // their properties with those declared in the class. 6969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::protocol_iterator P = IDecl->protocol_begin(), 6979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->protocol_end(); P != E; ++P) 6989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CompareProperties(IDecl, DeclPtrTy::make(*P)); 6999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } else { 7009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCProtocolDecl *MD = cast<ObjCProtocolDecl>(ClassDecl); 7019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::protocol_iterator P = MD->protocol_begin(), 7029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = MD->protocol_end(); P != E; ++P) 7039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek MatchOneProtocolPropertiesInClass(IDecl, *P); 7049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 7069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 7079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// isPropertyReadonly - Return true if property is readonly, by searching 7089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// for the property in the class and in its categories and implementations 7099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// 7109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekbool Sema::isPropertyReadonly(ObjCPropertyDecl *PDecl, 7119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCInterfaceDecl *IDecl) { 7129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // by far the most common case. 7139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!PDecl->isReadOnly()) 7149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Even if property is ready only, if interface has a user defined setter, 7169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // it is not considered read only. 7179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (IDecl->getInstanceMethod(PDecl->getSetterName())) 7189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 7209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Main class has the property as 'readonly'. Must search 7219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // through the category list to see if the property's 7229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // attribute has been over-ridden to 'readwrite'. 7239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCCategoryDecl *Category = IDecl->getCategoryList(); 7249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Category; Category = Category->getNextClassCategory()) { 7259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Even if property is ready only, if a category has a user defined setter, 7269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // it is not considered read only. 7279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Category->getInstanceMethod(PDecl->getSetterName())) 7289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *P = 7309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Category->FindPropertyDeclaration(PDecl->getIdentifier()); 7319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (P && !P->isReadOnly()) 7329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 7359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Also, check for definition of a setter method in the implementation if 7369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // all else failed. 7379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCMethodDecl *OMD = dyn_cast<ObjCMethodDecl>(CurContext)) { 7389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCImplementationDecl *IMD = 7399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek dyn_cast<ObjCImplementationDecl>(OMD->getDeclContext())) { 7409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (IMD->getInstanceMethod(PDecl->getSetterName())) 7419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } else if (ObjCCategoryImplDecl *CIMD = 7439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek dyn_cast<ObjCCategoryImplDecl>(OMD->getDeclContext())) { 7449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (CIMD->getInstanceMethod(PDecl->getSetterName())) 7459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Lastly, look through the implementation (if one is in scope). 7499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCImplementationDecl *ImpDecl = IDecl->getImplementation()) 7509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ImpDecl->getInstanceMethod(PDecl->getSetterName())) 7519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return false; 7529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // If all fails, look at the super class. 7539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCInterfaceDecl *SIDecl = IDecl->getSuperClass()) 7549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return isPropertyReadonly(PDecl, SIDecl); 7559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return true; 7569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 7579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 7589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// CollectImmediateProperties - This routine collects all properties in 7599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// the class and its conforming protocols; but not those it its super class. 7609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::CollectImmediateProperties(ObjCContainerDecl *CDecl, 7619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) { 7629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) { 7639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(), 7649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->prop_end(); P != E; ++P) { 7659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = (*P); 7669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek PropMap[Prop->getIdentifier()] = Prop; 7679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // scan through class's protocols. 7699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::protocol_iterator PI = IDecl->protocol_begin(), 7709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->protocol_end(); PI != E; ++PI) 771738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian // Exclude property for protocols which conform to class's super-class, 772738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian // as super-class has to implement the property. 773738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian if (!ProtocolConformsToSuperClass(IDecl, (*PI))) 774738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian CollectImmediateProperties((*PI), PropMap); 7759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (ObjCCategoryDecl *CATDecl = dyn_cast<ObjCCategoryDecl>(CDecl)) { 7779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!CATDecl->IsClassExtension()) 7789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCContainerDecl::prop_iterator P = CATDecl->prop_begin(), 7799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = CATDecl->prop_end(); P != E; ++P) { 7809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = (*P); 7819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek PropMap[Prop->getIdentifier()] = Prop; 7829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // scan through class's protocols. 7849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::protocol_iterator PI = CATDecl->protocol_begin(), 7859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = CATDecl->protocol_end(); PI != E; ++PI) 7869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CollectImmediateProperties((*PI), PropMap); 7879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) { 7899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(), 7909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = PDecl->prop_end(); P != E; ++P) { 7919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = (*P); 7929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *&PropEntry = PropMap[Prop->getIdentifier()]; 7939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!PropEntry) 7949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek PropEntry = Prop; 7959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 7969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // scan through protocol's protocols. 7979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(), 7989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = PDecl->protocol_end(); PI != E; ++PI) 7999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CollectImmediateProperties((*PI), PropMap); 8009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 8019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 8029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 803509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// CollectClassPropertyImplementations - This routine collects list of 804509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties to be implemented in the class. This includes, class's 805509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// and its conforming protocols' properties. 806509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianstatic void CollectClassPropertyImplementations(ObjCContainerDecl *CDecl, 807509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) { 808509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian if (ObjCInterfaceDecl *IDecl = dyn_cast<ObjCInterfaceDecl>(CDecl)) { 809509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(), 810509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian E = IDecl->prop_end(); P != E; ++P) { 811509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian ObjCPropertyDecl *Prop = (*P); 812509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian PropMap[Prop->getIdentifier()] = Prop; 813509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 814509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian for (ObjCInterfaceDecl::protocol_iterator PI = IDecl->protocol_begin(), 815509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian E = IDecl->protocol_end(); PI != E; ++PI) 816509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian CollectClassPropertyImplementations((*PI), PropMap); 817509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 818509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian else if (ObjCProtocolDecl *PDecl = dyn_cast<ObjCProtocolDecl>(CDecl)) { 819509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(), 820509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian E = PDecl->prop_end(); P != E; ++P) { 821509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian ObjCPropertyDecl *Prop = (*P); 822509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian PropMap[Prop->getIdentifier()] = Prop; 823509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 824509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian // scan through protocol's protocols. 825509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(), 826509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian E = PDecl->protocol_end(); PI != E; ++PI) 827509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian CollectClassPropertyImplementations((*PI), PropMap); 828509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 829509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian} 830509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian 831509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// CollectSuperClassPropertyImplementations - This routine collects list of 832509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties to be implemented in super class(s) and also coming from their 833509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// conforming protocols. 834509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianstatic void CollectSuperClassPropertyImplementations(ObjCInterfaceDecl *CDecl, 835509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>& PropMap) { 836509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian if (ObjCInterfaceDecl *SDecl = CDecl->getSuperClass()) { 837509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian while (SDecl) { 838509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian CollectClassPropertyImplementations(SDecl, PropMap); 839509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian SDecl = SDecl->getSuperClass(); 840509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 841509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 842509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian} 843509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian 844738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian/// ProtocolConformsToSuperClass - Returns true if class's given protocol 845738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian/// conforms to one of its super class's protocols. 846738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanianbool Sema::ProtocolConformsToSuperClass(const ObjCInterfaceDecl *IDecl, 847738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian const ObjCProtocolDecl *PDecl) { 848738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian if (const ObjCInterfaceDecl *CDecl = IDecl->getSuperClass()) { 849738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian for (ObjCInterfaceDecl::protocol_iterator PI = CDecl->protocol_begin(), 850738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian E = CDecl->protocol_end(); PI != E; ++PI) { 851738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian if (ProtocolConformsToProtocol((*PI), PDecl)) 852738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian return true; 853738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian return ProtocolConformsToSuperClass(CDecl, PDecl); 854738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian } 855738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian } 856738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian return false; 857738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian} 858738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian 859738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanianbool Sema::ProtocolConformsToProtocol(const ObjCProtocolDecl *NestedProtocol, 860738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian const ObjCProtocolDecl *PDecl) { 861738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian if (PDecl->getIdentifier() == NestedProtocol->getIdentifier()) 862738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian return true; 863738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian // scan through protocol's protocols. 864738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(), 865738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian E = PDecl->protocol_end(); PI != E; ++PI) 866738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian if (ProtocolConformsToProtocol(NestedProtocol, (*PI))) 867738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian return true; 868738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian return false; 869738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian} 870738698d9d2e1ab0b1ae6a04874d521814907b8a5Fariborz Jahanian 8719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// LookupPropertyDecl - Looks up a property in the current class and all 8729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// its protocols. 8739d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekObjCPropertyDecl *Sema::LookupPropertyDecl(const ObjCContainerDecl *CDecl, 8749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek IdentifierInfo *II) { 8759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (const ObjCInterfaceDecl *IDecl = 8769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek dyn_cast<ObjCInterfaceDecl>(CDecl)) { 8779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCContainerDecl::prop_iterator P = IDecl->prop_begin(), 8789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->prop_end(); P != E; ++P) { 8799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = (*P); 8809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Prop->getIdentifier() == II) 8819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return Prop; 8829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 8839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // scan through class's protocols. 8849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCInterfaceDecl::protocol_iterator PI = IDecl->protocol_begin(), 8859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->protocol_end(); PI != E; ++PI) { 8869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II); 8879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Prop) 8889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return Prop; 8899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 8909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 8919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek else if (const ObjCProtocolDecl *PDecl = 8929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek dyn_cast<ObjCProtocolDecl>(CDecl)) { 8939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::prop_iterator P = PDecl->prop_begin(), 8949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = PDecl->prop_end(); P != E; ++P) { 8959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = (*P); 8969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Prop->getIdentifier() == II) 8979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return Prop; 8989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 8999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // scan through protocol's protocols. 9009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCProtocolDecl::protocol_iterator PI = PDecl->protocol_begin(), 9019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = PDecl->protocol_end(); PI != E; ++PI) { 9029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = LookupPropertyDecl((*PI), II); 9039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Prop) 9049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return Prop; 9059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 9069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 9079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return 0; 9089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 9099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 910509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// DefaultSynthesizeProperties - This routine default synthesizes all 911509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian/// properties which must be synthesized in class's @implementation. 912509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanianvoid Sema::DefaultSynthesizeProperties (Scope *S, ObjCImplDecl* IMPDecl, 913509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian ObjCInterfaceDecl *IDecl) { 914509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian 915509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap; 916509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian CollectClassPropertyImplementations(IDecl, PropMap); 917509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian if (PropMap.empty()) 918509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian return; 919509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> SuperPropMap; 920509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian CollectSuperClassPropertyImplementations(IDecl, SuperPropMap); 921509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian 922509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator 923509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian P = PropMap.begin(), E = PropMap.end(); P != E; ++P) { 924509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian ObjCPropertyDecl *Prop = P->second; 925509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian // If property to be implemented in the super class, ignore. 926509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian if (SuperPropMap[Prop->getIdentifier()]) 927509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian continue; 928509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian // Is there a matching propery synthesize/dynamic? 929509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian if (Prop->isInvalidDecl() || 930509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional || 931509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian IMPDecl->FindPropertyImplIvarDecl(Prop->getIdentifier())) 932509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian continue; 933509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian ActOnPropertyImplDecl(S, IMPDecl->getLocation(), IMPDecl->getLocation(), 934509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian true, DeclPtrTy::make(IMPDecl), 935509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian Prop->getIdentifier(), Prop->getIdentifier()); 936509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian } 937509d477ae5a24282719320fe92038ccf57358840Fariborz Jahanian} 9389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 93917cb326cb62a59f53d92236394af40eaae4eddbdFariborz Jahanianvoid Sema::DiagnoseUnimplementedProperties(Scope *S, ObjCImplDecl* IMPDecl, 9409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCContainerDecl *CDecl, 9419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek const llvm::DenseSet<Selector>& InsMap) { 9429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*> PropMap; 9439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CollectImmediateProperties(CDecl, PropMap); 9449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (PropMap.empty()) 9459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return; 9469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 9479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek llvm::DenseSet<ObjCPropertyDecl *> PropImplMap; 9489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCImplDecl::propimpl_iterator 9499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek I = IMPDecl->propimpl_begin(), 9509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek EI = IMPDecl->propimpl_end(); I != EI; ++I) 9519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek PropImplMap.insert((*I)->getPropertyDecl()); 9529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 9539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (llvm::DenseMap<IdentifierInfo *, ObjCPropertyDecl*>::iterator 9549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek P = PropMap.begin(), E = PropMap.end(); P != E; ++P) { 9559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Prop = P->second; 9569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Is there a matching propery synthesize/dynamic? 9579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (Prop->isInvalidDecl() || 9589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Prop->getPropertyImplementation() == ObjCPropertyDecl::Optional || 9599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek PropImplMap.count(Prop)) 9609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek continue; 9619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!InsMap.count(Prop->getGetterName())) { 9629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Prop->getLocation(), 9639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek isa<ObjCCategoryDecl>(CDecl) ? 9649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::warn_setter_getter_impl_required_in_category : 9659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::warn_setter_getter_impl_required) 9669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Prop->getDeclName() << Prop->getGetterName(); 9679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(IMPDecl->getLocation(), 9689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::note_property_impl_required); 9699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 9709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 9719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!Prop->isReadOnly() && !InsMap.count(Prop->getSetterName())) { 9729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Prop->getLocation(), 9739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek isa<ObjCCategoryDecl>(CDecl) ? 9749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::warn_setter_getter_impl_required_in_category : 9759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::warn_setter_getter_impl_required) 9769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Prop->getDeclName() << Prop->getSetterName(); 9779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(IMPDecl->getLocation(), 9789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::note_property_impl_required); 9799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 9809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 9819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 9829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 9839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid 9849d64c15223497f31899cc47fbe1531873dc06f2eTed KremenekSema::AtomicPropertySetterGetterRules (ObjCImplDecl* IMPDecl, 9859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCContainerDecl* IDecl) { 9869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Rules apply in non-GC mode only 9879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (getLangOptions().getGCMode() != LangOptions::NonGC) 9889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek return; 9899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek for (ObjCContainerDecl::prop_iterator I = IDecl->prop_begin(), 9909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek E = IDecl->prop_end(); 9919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek I != E; ++I) { 9929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl *Property = (*I); 9939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek unsigned Attributes = Property->getPropertyAttributes(); 9949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // We only care about readwrite atomic property. 9959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((Attributes & ObjCPropertyDecl::OBJC_PR_nonatomic) || 9969d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek !(Attributes & ObjCPropertyDecl::OBJC_PR_readwrite)) 9979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek continue; 9989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (const ObjCPropertyImplDecl *PIDecl 9999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek = IMPDecl->FindPropertyImplDecl(Property->getIdentifier())) { 10009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (PIDecl->getPropertyImplementation() == ObjCPropertyImplDecl::Dynamic) 10019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek continue; 10029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl *GetterMethod = 10039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek IMPDecl->getInstanceMethod(Property->getGetterName()); 10049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl *SetterMethod = 10059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek IMPDecl->getInstanceMethod(Property->getSetterName()); 10069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((GetterMethod && !SetterMethod) || (!GetterMethod && SetterMethod)) { 10079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SourceLocation MethodLoc = 10089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek (GetterMethod ? GetterMethod->getLocation() 10099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek : SetterMethod->getLocation()); 10109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(MethodLoc, diag::warn_atomic_property_rule) 10119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << Property->getIdentifier(); 10129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(Property->getLocation(), diag::note_property_declare); 10139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 10149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 10159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 10169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 10179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 10189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// ProcessPropertyDecl - Make sure that any user-defined setter/getter methods 10199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// have the property type and issue diagnostics if they don't. 10209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// Also synthesize a getter/setter method if none exist (and update the 10219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// appropriate lookup tables. FIXME: Should reconsider if adding synthesized 10229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek/// methods is the "right" thing to do. 10239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenekvoid Sema::ProcessPropertyDecl(ObjCPropertyDecl *property, 10249d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCContainerDecl *CD) { 10259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl *GetterMethod, *SetterMethod; 10269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 10279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek GetterMethod = CD->getInstanceMethod(property->getGetterName()); 10289d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SetterMethod = CD->getInstanceMethod(property->getSetterName()); 10299d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek DiagnosePropertyAccessorMismatch(property, GetterMethod, 10309d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getLocation()); 10319d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 10329d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (SetterMethod) { 10339d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl::PropertyAttributeKind CAttr = 10349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getPropertyAttributes(); 10359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((!(CAttr & ObjCPropertyDecl::OBJC_PR_readonly)) && 10369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Context.getCanonicalType(SetterMethod->getResultType()) != 10379d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Context.VoidTy) 10389d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(SetterMethod->getLocation(), diag::err_setter_type_void); 10399d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (SetterMethod->param_size() != 1 || 10409d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ((*SetterMethod->param_begin())->getType() != property->getType())) { 10419d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(property->getLocation(), 10429d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek diag::warn_accessor_property_type_mismatch) 10439d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << property->getDeclName() 10449d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek << SetterMethod->getSelector(); 10459d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Diag(SetterMethod->getLocation(), diag::note_declared_at); 10469d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 10479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 10489d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 10499d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Synthesize getter/setter methods if none exist. 10509d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Find the default getter and if one not found, add one. 10519d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // FIXME: The synthesized property we set here is misleading. We almost always 10529d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // synthesize these methods unless the user explicitly provided prototypes 10539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // (which is odd, but allowed). Sema should be typechecking that the 10549d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // declarations jive in that situation (which it is not currently). 10559d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!GetterMethod) { 10569d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // No instance method of same name as property getter name was found. 10579d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Declare a getter method and add it to the list of methods 10589d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // for this class. 10599d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek GetterMethod = ObjCMethodDecl::Create(Context, property->getLocation(), 10609d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getLocation(), property->getGetterName(), 10619d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getType(), 0, CD, true, false, true, 10629d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek (property->getPropertyImplementation() == 10639d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl::Optional) ? 10649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl::Optional : 10659d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl::Required); 10669d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CD->addDecl(GetterMethod); 10679d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } else 10689d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // A user declared getter will be synthesize when @synthesize of 10699d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // the property with the same name is seen in the @implementation 10709d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek GetterMethod->setSynthesized(true); 10719d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->setGetterMethodDecl(GetterMethod); 10729d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 10739d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Skip setter if property is read-only. 10749d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!property->isReadOnly()) { 10759d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Find the default setter and if one not found, add one. 10769d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (!SetterMethod) { 10779d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // No instance method of same name as property setter name was found. 10789d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Declare a setter method and add it to the list of methods 10799d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // for this class. 10809d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SetterMethod = ObjCMethodDecl::Create(Context, property->getLocation(), 10819d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getLocation(), 10829d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getSetterName(), 10839d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek Context.VoidTy, 0, CD, true, false, true, 10849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek (property->getPropertyImplementation() == 10859d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCPropertyDecl::Optional) ? 10869d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl::Optional : 10879d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ObjCMethodDecl::Required); 10889d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Invent the arguments for the setter. We don't bother making a 10899d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // nice name for the argument. 10909d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek ParmVarDecl *Argument = ParmVarDecl::Create(Context, SetterMethod, 10919d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getLocation(), 10929d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getIdentifier(), 10939d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->getType(), 10949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek /*TInfo=*/0, 10959d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek VarDecl::None, 109616573fa9705b546b7597c273b25b85d6321e2b33Douglas Gregor VarDecl::None, 10979d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 0); 10984ecb25fa94897b2c03510292acace710e5262ba5Fariborz Jahanian SetterMethod->setMethodParams(Context, &Argument, 1, 1); 10999d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek CD->addDecl(SetterMethod); 11009d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } else 11019d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // A user declared setter will be synthesize when @synthesize of 11029d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // the property with the same name is seen in the @implementation 11039d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SetterMethod->setSynthesized(true); 11049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek property->setSetterMethodDecl(SetterMethod); 11059d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek } 11069d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // Add any synthesized methods to the global pool. This allows us to 11079d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // handle the following, which is supported by GCC (and part of the design). 11089d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // 11099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // @interface Foo 11109d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // @property double bar; 11119d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // @end 11129d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // 11139d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // void thisIsUnfortunate() { 11149d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // id foo; 11159d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // double bar = [foo bar]; 11169d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // } 11179d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // 11189d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (GetterMethod) 11199d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek AddInstanceMethodToGlobalPool(GetterMethod); 11209d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if (SetterMethod) 11219d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek AddInstanceMethodToGlobalPool(SetterMethod); 11229d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 11239d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 1124842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanianvoid Sema::CheckObjCPropertyAttributes(DeclPtrTy PropertyPtrTy, 11259d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek SourceLocation Loc, 11269d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek unsigned &Attributes) { 11279d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // FIXME: Improve the reported location. 1128842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian Decl *PDecl = PropertyPtrTy.getAs<Decl>(); 11295fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek if (!PDecl) 11305fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek return; 11315fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek 11325fcd52a6c94c7f93bd80062248353c45330da23aTed Kremenek ObjCPropertyDecl *PropertyDecl = cast<ObjCPropertyDecl>(PDecl); 1133842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian QualType PropertyTy = PropertyDecl->getType(); 11349d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 11359d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek // readonly and readwrite/assign/retain/copy conflict. 11369d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek if ((Attributes & ObjCDeclSpec::DQ_PR_readonly) && 113728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek (Attributes & (ObjCDeclSpec::DQ_PR_readwrite | 113828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCDeclSpec::DQ_PR_assign | 113928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCDeclSpec::DQ_PR_copy | 114028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCDeclSpec::DQ_PR_retain))) { 114128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek const char * which = (Attributes & ObjCDeclSpec::DQ_PR_readwrite) ? 114228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek "readwrite" : 114328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek (Attributes & ObjCDeclSpec::DQ_PR_assign) ? 114428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek "assign" : 114528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek (Attributes & ObjCDeclSpec::DQ_PR_copy) ? 114628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek "copy" : "retain"; 11479d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 114828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, (Attributes & (ObjCDeclSpec::DQ_PR_readwrite)) ? 114928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek diag::err_objc_property_attr_mutually_exclusive : 115028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek diag::warn_objc_property_attr_mutually_exclusive) 115128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << "readonly" << which; 115228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 11539d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 115428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Check for copy or retain on non-object types. 115528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if ((Attributes & (ObjCDeclSpec::DQ_PR_copy | ObjCDeclSpec::DQ_PR_retain)) && 115628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek !PropertyTy->isObjCObjectPointerType() && 115728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek !PropertyTy->isBlockPointerType() && 1158842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian !Context.isObjCNSObjectType(PropertyTy) && 1159842f07b065ac481fce9d82d55cb62e36bac6c921Fariborz Jahanian !PropertyDecl->getAttr<ObjCNSObjectAttr>()) { 116028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::err_objc_property_requires_object) 116128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << (Attributes & ObjCDeclSpec::DQ_PR_copy ? "copy" : "retain"); 116228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Attributes &= ~(ObjCDeclSpec::DQ_PR_copy | ObjCDeclSpec::DQ_PR_retain); 116328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 11649d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 116528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Check for more than one of { assign, copy, retain }. 116628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_assign) { 116728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_copy) { 116828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::err_objc_property_attr_mutually_exclusive) 116928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << "assign" << "copy"; 117028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Attributes &= ~ObjCDeclSpec::DQ_PR_copy; 117128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 117228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_retain) { 117328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::err_objc_property_attr_mutually_exclusive) 117428685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << "assign" << "retain"; 117528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Attributes &= ~ObjCDeclSpec::DQ_PR_retain; 117628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 117728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } else if (Attributes & ObjCDeclSpec::DQ_PR_copy) { 117828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (Attributes & ObjCDeclSpec::DQ_PR_retain) { 117928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::err_objc_property_attr_mutually_exclusive) 118028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek << "copy" << "retain"; 118128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Attributes &= ~ObjCDeclSpec::DQ_PR_retain; 118228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 118328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 11849d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 118528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Warn if user supplied no assignment attribute, property is 118628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // readwrite, and this is an object type. 118728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!(Attributes & (ObjCDeclSpec::DQ_PR_assign | ObjCDeclSpec::DQ_PR_copy | 118828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek ObjCDeclSpec::DQ_PR_retain)) && 118928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek !(Attributes & ObjCDeclSpec::DQ_PR_readonly) && 119028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek PropertyTy->isObjCObjectPointerType()) { 119128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // Skip this warning in gc-only mode. 119228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (getLangOptions().getGCMode() != LangOptions::GCOnly) 119328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::warn_objc_property_no_assignment_attribute); 11949d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 119528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // If non-gc code warn that this is likely inappropriate. 119628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (getLangOptions().getGCMode() == LangOptions::NonGC) 119728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::warn_objc_property_default_assign_on_object); 11989d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 119928685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // FIXME: Implement warning dependent on NSCopying being 120028685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // implemented. See also: 120128685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // <rdar://5168496&4855821&5607453&5096644&4947311&5698469&4947014&5168496> 120228685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek // (please trim this list while you are at it). 120328685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek } 12049d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek 120528685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek if (!(Attributes & ObjCDeclSpec::DQ_PR_copy) 120628685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek && getLangOptions().getGCMode() == LangOptions::GCOnly 120728685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek && PropertyTy->isBlockPointerType()) 120828685ab2ddeea146841bf2e277e25c7b32dfc9f6Ted Kremenek Diag(Loc, diag::warn_objc_property_copy_missing_on_block); 12099d64c15223497f31899cc47fbe1531873dc06f2eTed Kremenek} 1210