Rewrite method definition bodies. Also renamed a method to distinguish between method declarations and definitions.

llvm-svn: 44080

llvm-svn: 44025

Add an error diagnostic to Parse::ParseObjCMessageExpression().

This now exposes the following bug...

******************** TEST 'Sema/message.m' FAILED! ********************
Command: 
 clang -fsyntax-only -verify Sema/message.m
Output:
Errors seen but not expected:
  Line 9: invalid receiver to message expression
******************** TEST 'Sema/message.m' FAILED! ********************

As far as I can tell, all messages to method agruments fail.

The method arguments are built by Sema::ObjcActOnStartOfMethodDef().

llvm-svn: 44022

Replace 2 method definition actions (ActOnFunctionDefBody, ActOnMethodDefBody) with 1 method definition action (ActOnFinishFunctionBody). I can't think of any reason that we would need two action hooks.

llvm-svn: 44000

Replace 3 method definition functions (ObjcParseMethodDefinition, ParseObjCInstanceMethodDefinition, ParseObjCClassMethodDefinition) with 1 method definition function (ParseObjCMethodDefinition).

Less code/confusion.

llvm-svn: 43999

This is the last 5% of the solution to teaching Sema::ActOnInstanceMessage() about private methods (r43989).

While the diff is large, the idea is very simple. When we parse method definitions (in an @implementation), we need to add them incrementally (rather than wait until the @end).

Other details...

- Renamed Sema::ActOnAddMethodsToObjcDecl() to Sema::ActOnAtEnd(). The methods are now optional arguments.
- Removed Parser::AllImplMethods (a nice cleanup).
- Added location info to ObjcImplementationDecl (since we will need it very soon:-)
- Modified message.m test to no longer allow the bogus diagnostic.

llvm-svn: 43995

llvm-svn: 43986

llvm-svn: 43979

process method definitions.

llvm-svn: 43967

to 'self' method of instance methods.

llvm-svn: 43957

llvm-svn: 43915

llvm-svn: 43802

llvm-svn: 43778

Implement rewrite rules for ObjC string constants.

llvm-svn: 43665

llvm-svn: 43649

llvm-svn: 43640

llvm-svn: 43629

llvm-svn: 43628

llvm-svn: 43623

2) Fixed a test failure (which should have failed all along!).

llvm-svn: 43589

Fix two rewriter bugs:

- For @class, don't generate multiple typedefs.
- Handle the following edge case interface...

@interface NSMiddleSpecifier : NSObject {}

@end

...which was incorrectly being rewritten to...

struct _interface_NSMiddleSpecifier {
        struct _interface_NSObject _NSObject;
};
{}

llvm-svn: 43582

llvm-svn: 43580

- Add location info to category/protocol AST's
- Rewrite categories.

llvm-svn: 43501

Remove a couple FIXME's for rewriting ObjC interfaces (which are now being rewritten properly).

llvm-svn: 43494

More support for rewriting ObjC intefaces. Still some edge cases to handle...

llvm-svn: 43493

Remove a bunch of TODO's that have been done for quite some time...

llvm-svn: 43476

This commit contains lot's of small tweaks to how we pass around and store SourceLocation's for interfaces/protocols/categories/implementations.

llvm-svn: 43475

Start rewriting ObjC interfaces. As a start, we comment out all the methods. This involved refining how the parser/AST passes/manages SourceLocations for ObjcMethodDecl's. 

llvm-svn: 43404

llvm-svn: 43178

llvm-svn: 43075

llvm-svn: 43051

@selector probably gets this wrong also.

llvm-svn: 43048

llvm-svn: 43039

llvm-svn: 43038

llvm-svn: 43022

Fix a parser bug with message expressions - need to call ParsePostfixExpressionSuffix(). 

Now were correctly allow the following...

    i = [str rangeOfString:@"]"].length;

llvm-svn: 43012

- Added Sema::AddFactoryMethodToGlobalPool and Sema::AddInstanceMethodToGlobalPool and DenseMaps. This will allow us to efficiently lookup a method from a selector given no type information (for the "id" data type).

- Fixed some funky "}
                    else {" indentation in Sema::ActOnAddMethodsToObjcDecl(). I'd prefer we stay away from this style...it wastes space and isn't any easier to read (from my perspective, at least:-)

- Changed Parser::ParseObjCInterfaceDeclList() to only call Action::ActOnAddMethodsToObjcDecl() when it actually has methods to add (since most interface have methods, this is a very minor cleanup).

llvm-svn: 42957

llvm-svn: 42883

llvm-svn: 42856

Remove Scope argument from ObjC actions that either don't need it or can now use TUScope.
Also improve a recently added comment.

llvm-svn: 42826