summaryrefslogtreecommitdiffstats
path: root/clang/lib/Sema/SemaDeclObjC.cpp
Commit message (Collapse)AuthorAgeFilesLines
...
* Mostly renaming some methods and updating comments toFariborz Jahanian2010-01-181-27/+34
| | | | | | | reflect what these methods are actually doing. One method template for future work. No change in functionality. llvm-svn: 93742
* Improve location information for Objective-C category declarations. WeDouglas Gregor2010-01-161-2/+3
| | | | | | | | | | | | | previously only had a single location (the @ in @interface); now we know where the @ is (for the start of the declaration), where the class name is (that's the normal "location" now for diagnostics), and where the category name is. Also, eliminated the redundant "end" location, since ObjCContainerDecl already has better @end information. The only XFAIL'd test is temporary; will un-XFAIL-it once I've taught CIndex how to use the new locations. llvm-svn: 93639
* Keep track of the source locations for each protocol reference inDouglas Gregor2010-01-161-5/+13
| | | | | | | | Objective-C classes, protocol definitions, forward protocol declarations, and categories. This information isn't actually used yet; that's coming next. llvm-svn: 93636
* Change ObjCContainerDecl to contain the entire range for the '@end'Ted Kremenek2010-01-071-7/+12
| | | | | | | | | | | piece of the declaration. The '@' and the 'end' are separate tokens, and require two SourceLocations to accurately track. This change was motivated because ObjCContainerDecl::getSourceRange() would previously not return the entire range of the declaration (the 'end' would be left off). llvm-svn: 92891
* Whenever we emit a typo-correction diagnostic, also emit a noteDouglas Gregor2010-01-071-0/+4
| | | | | | | pointing to the declaration that we found that has that name (if it is unique). llvm-svn: 92877
* When suggesting a typo correction for an @implementation without aDouglas Gregor2010-01-061-1/+7
| | | | | | | | | | corresponding @interface, provide a note showing which interface we're referring to. This note has the fix-it hint on it. Also, don't automatically apply fix-it hints for notes. They're meant to express fix-its that would change semantics. llvm-svn: 92870
* Fix a bug when property is redeclared in multipleFariborz Jahanian2010-01-061-0/+26
| | | | | | | continuation classes and its original declaration is imported from a protocol. This fixes radar 7509234. llvm-svn: 92856
* Do not diagnose method disguised as property setterFariborz Jahanian2010-01-061-2/+5
| | | | | | for a 'readonly' property. Fixes radar 7427072. llvm-svn: 92808
* When declaring an Objective-C implementation without a correspondingDouglas Gregor2010-01-041-4/+18
| | | | | | | | | | | | | | | | | | | | | | interface, suggest correction of typos. For example, given: @interface NSString @end @implementation NSstring @end we'll warn with: t.m:4:19: warning: cannot find interface declaration for 'NSstring'; did you mean 'NSString'? @implementation NSstring ^ However, since this is just a warning, we don't provide a fix-it hint. Good idea, Ted! llvm-svn: 92488
* Implement typo correction for a variety of Objective-C-specificDouglas Gregor2010-01-031-2/+24
| | | | | | | | | | | | | | constructs: - Instance variable lookup ("foo->ivar" and, in instance methods, "ivar") - Property name lookup ("foo.prop") - Superclasses - Various places where a class name is required - Protocol names (e.g., id<proto>) This seems to cover many of the common places where typos could occur. llvm-svn: 92449
* Diagnose duplicate declaration of a property. FixesFariborz Jahanian2009-12-171-1/+8
| | | | | | PR5809 llvm-svn: 91575
* Diagnose property of reference type as unsupportedFariborz Jahanian2009-12-161-0/+4
| | | | | | instead of crashing for now. llvm-svn: 91546
* Improve the diagnostic when a new declaration conflicts with a using shadowJohn McCall2009-12-101-4/+4
| | | | | | | | declaration. Rename note_using_decl to note_using, which is possibly less confusing. Add a test for non-class-scope using decl collisions and be sure to note the case we can't diagnose yet. llvm-svn: 91057
* DeclaratorInfo -> TypeSourceInfo. Makes an effort to rename associated ↵John McCall2009-12-071-2/+2
| | | | | | | | | | | | | | | | | | | | | variables, but the results are imperfect. For posterity, I did: cat <<EOF > $cmdfile s/DeclaratorInfo/TypeSourceInfo/g s/DInfo/TInfo/g s/TypeTypeSourceInfo/TypeSourceInfo/g s/SourceTypeSourceInfo/TypeSourceInfo/g EOF find lib -name '*.cpp' -not -path 'lib/Parse/*' -exec sed -i '' -f $cmdfile '{}' \; find lib -name '*.h' -exec sed -i '' -f $cmdfile '{}' \; find include -name '*.h' -not -path 'include/clang/Parse/*' -not -path 'include/clang/Basic/*' -exec sed -i '' -f $cmdfile '{}' \; llvm-svn: 90743
* Don't warn if objc method param types in declaration andFariborz Jahanian2009-11-181-3/+4
| | | | | | | | implementation mismatch in their qualifiers only. This will match similar behavior in c/c++ and fixes radar 7211653. llvm-svn: 89220
* Add SourceLocations to ObjCClassDecl for the class identifiers referenced by ↵Ted Kremenek2009-11-181-5/+3
| | | | | | @class. llvm-svn: 89170
* Pass source locations of identifiers referenced by @class through ↵Ted Kremenek2009-11-171-0/+1
| | | | | | Action::ActOnForwardClassDeclaration(). llvm-svn: 89162
* Do not register ObjCInterfaceDecls implicitly created by @class in theTed Kremenek2009-11-171-1/+14
| | | | | | | | | | | | | | | | | | current DeclContext. These "imaginary" declarations pose issues for clients searching DeclContext for actual declarations. Instead, register them for name lookup, and add the ObjCInterfaceDecl later to the DeclContext when we hit an actual @interface declaration. This also fixes a bug where the invariant that the Decls in a DeclContext are sorted in order of their appearance is no longer violated. What could happen is that an @class causes an ObjCInterfaceDecl to get added first to the DeclContext, then the ObjCClassDecl itself is added, and then later the SourceLocation of the ObjCInterfaceDecl is updated with the correct location (which is later in the file). This breaks an assumed invariant in ResolveLocation.cpp (and possibly other clients). llvm-svn: 89160
* Start support of weak_import objective-c classes.Fariborz Jahanian2009-11-171-0/+2
| | | | | | (Radar 6815425). llvm-svn: 89124
* Handle case of missing '@end' in implementation contextFariborz Jahanian2009-11-161-1/+6
| | | | | | | | gracefully, on par with gcc, by: Issuing a warning, doing final sematinc check of its definitions and generating its meta-data. llvm-svn: 88934
* writable atomic property's setter/getter must be in 'lock' step ofFariborz Jahanian2009-11-111-1/+38
| | | | | | | either both synthesized or bith user defined. Implements radar 6557233. llvm-svn: 86887
* Changed a variable name to match what it representsFariborz Jahanian2009-11-101-3/+3
| | | | | | (Ted's feedback). llvm-svn: 86702
* Since default writable attribute is 'assign', allowFariborz Jahanian2009-11-061-2/+1
| | | | | | | | | | specification of 'assign' (no warning to be issued), when a continuation class makes a 'readonly' attribute 'readwrite' but also specifies the 'assign' attribute. (this matches gcc's behavior and prevents exessive warnings)/ llvm-svn: 86297
* Change our basic strategy for avoiding deprecation warnings when the decl useJohn McCall2009-11-041-0/+10
| | | | | | | | | | | | appears in a deprecated context. In the new strategy, we emit the warnings as usual unless we're currently parsing a declaration, where "declaration" is restricted to mean a decl group or a few special cases in Objective C. If we *are* parsing a declaration, we queue up the deprecation warnings until the declaration has been completely parsed, and then emit them only if the decl is not deprecated. We also standardize the bookkeeping for deprecation so as to avoid special cases. llvm-svn: 85998
* Assortment of property attributes declared in continuationFariborz Jahanian2009-11-031-2/+9
| | | | | | | class must match those of same property declared in its primary class. (Fixes radar 7352425) llvm-svn: 85843
* Property declared in continuation class can only be used toFariborz Jahanian2009-11-021-12/+6
| | | | | | | change a readonly property declared in the class (and its inherited protocols) to writable property. (Fixes radar 7350645). llvm-svn: 85836
* Diagnose implementation of a property declared in a categoryFariborz Jahanian2009-11-021-0/+8
| | | | | | in its class implementation instead of crashing. Fixes radar 7350345. llvm-svn: 85813
* Remove OriginalTypeParmDecl; the original type is the one specifiedJohn McCall2009-10-231-16/+9
| | | | | | | | | | | | | in the DeclaratorInfo, if one is present. Preserve source information through template instantiation. This is made more complicated by the possibility that ParmVarDecls don't have DIs, which is possibly worth fixing in the future. Also preserve source information for function parameters in ObjC method declarations. llvm-svn: 84971
* Refactor the LookupResult API to simplify most common operations. Require ↵John McCall2009-10-091-8/+10
| | | | | | | | | users to pass a LookupResult reference to lookup routines. Call out uses which assume a single result. llvm-svn: 83674
* tweaked my last patch to 1) preserve the protocol inFariborz Jahanian2009-10-051-9/+11
| | | | | | | | | extension class's protocol list so its AST is complete. 2) Because of this no need to issue warning on unimplemeted methods coming from the extended class protocols because warning is issued when class definition is seen. llvm-svn: 83326
* Patch to implement Protocols on class extensions.Fariborz Jahanian2009-10-051-2/+9
| | | | | | <rdar://problem/7269631> Protocols on class extensions don't work llvm-svn: 83322
* Change all the Type::getAsFoo() methods to specializations of Type::getAs().John McCall2009-09-211-3/+3
| | | | | | | | | | | Several of the existing methods were identical to their respective specializations, and so have been removed entirely. Several more 'leaf' optimizations were introduced. The getAsFoo() methods which imposed extra conditions, like getAsObjCInterfacePointerType(), have been left in place. llvm-svn: 82501
* Make sure ObjCInterfaceDecl's that are forward declared always contain a ↵Steve Naroff2009-09-111-1/+5
| | | | | | valid location. llvm-svn: 81487
* Remove tabs, and whitespace cleanups.Mike Stump2009-09-091-288/+286
| | | | llvm-svn: 81346
* patch to prevent crash in hopelessly incorrectFariborz Jahanian2009-08-281-0/+1
| | | | | | method definition with labels. llvm-svn: 80381
* Don't issue warning on multiple selector found when Fariborz Jahanian2009-08-221-2/+3
| | | | | | selector name is for a @selector expression. llvm-svn: 79776
* Use Sema's LocInfoType to pass and preserve type source info through the Parser.Argyrios Kyrtzidis2009-08-191-2/+2
| | | | llvm-svn: 79395
* Introduce DeclaratorDecl and pass DeclaratorInfo through the Decl/Sema ↵Argyrios Kyrtzidis2009-08-191-1/+4
| | | | | | | | | | | | interfaces. DeclaratorDecl contains a DeclaratorInfo* to keep type source info. Subclasses of DeclaratorDecl are FieldDecl, FunctionDecl, and VarDecl. EnumConstantDecl still inherits from ValueDecl since it has no need for DeclaratorInfo. Decl/Sema interfaces accept a DeclaratorInfo as parameter but no DeclaratorInfo is created yet. llvm-svn: 79392
* Fixed a regression in deciding when to issue warning on properties whichFariborz Jahanian2009-08-141-2/+4
| | | | | | implement NSCopying protocol in GC mode. llvm-svn: 79008
* error on property of objc interface type instead of crashingFariborz Jahanian2009-08-121-0/+2
| | | | llvm-svn: 78826
* Patch to warn if a property which is 'assign' by defaultFariborz Jahanian2009-08-111-1/+17
| | | | | | may not implement NSCopying protocol in -fobjc-gc[-only] mode. llvm-svn: 78726
* Canonicalize else.Mike Stump2009-08-041-27/+16
| | | | llvm-svn: 78102
* Some code refactoring. Be more generous in issuance of warningFariborz Jahanian2009-08-041-24/+34
| | | | | | on method type mismatches per Chris's comment. llvm-svn: 78075
* Compare matching selectors in current andFariborz Jahanian2009-08-041-0/+33
| | | | | | | super class(s) and warn on any parameter type mismatch if potentially unsafe. llvm-svn: 78029
* Make ObjCImplDecl inherit from ObjCContainerDecl.Argyrios Kyrtzidis2009-07-271-2/+2
| | | | | | ObjCContainerDecl now is the root class for objc decls that contain methods. llvm-svn: 77235
* Remove a bunch of FIXME's related to ObjC type checking.Steve Naroff2009-07-231-4/+5
| | | | | | | | - Move Sema::ObjCQualifiedIdTypesAreCompatible(), Sema::QualifiedIdConformsQualifiedId(), and a couple helper functions to ASTContext. - Change ASTContext::canAssignObjCInterfaces() to use ASTContext:: ObjCQualifiedIdTypesAreCompatible(). - Tweak several test cases to accommodate the new/improved type checking. llvm-svn: 76830
* Remove the ObjCCategoryImpls vector from Sema class.Argyrios Kyrtzidis2009-07-211-2/+0
| | | | | | Use ObjCInterfaceDecl::getCategoryClassMethod() and ObjCInterfaceDecl::getCategoryInstanceMethod() for the same functionality. llvm-svn: 76510
* Remove Sema::LookupObjCImplementation and replace it with just calling ↵Argyrios Kyrtzidis2009-07-211-4/+5
| | | | | | ObjCInterfaceDecl::getImplementation(). llvm-svn: 76509
* - Introduce ASTContext::getObjCImplementation() and ↵Argyrios Kyrtzidis2009-07-211-3/+27
| | | | | | | | | | | | ASTContext::setObjCImplementation() which use a DenseMap to associate an interface/category with its implementation (if one exists). - Introduce ObjCInterfaceDecl::get/setImplementation() and ObjCCategoryDecl::get/setImplementation() that use the above methods. - Add a compiler error for when a category is reimplemented. llvm-svn: 76508
* Remove ASTContext::isObjCObjectPointerType().Steve Naroff2009-07-161-3/+5
| | | | | | Convert all clients to use the new predicate on Type. llvm-svn: 76076
OpenPOWER on IntegriCloud