discussion of this change.

llvm-svn: 45410

   The later fails if the token is not an identifier.
 - Replace tabs with spaces.
 - Various other cleanups.

Patch by Nico Weber!

llvm-svn: 45368

llvm-svn: 44978

llvm-svn: 44964

llvm-svn: 44923

this is passed to sema and ignored there, so the second part of the
string will not make it into the AST.  Passing to Fariborz to finish
Sema + AST construction.

llvm-svn: 44898

a method declaration.

llvm-svn: 44870

Make sure Parser::ParseObjCSelectorExpression() handles unary selectors (with no arguments) properly.

llvm-svn: 44636

Finish up variadic methods/messages.

llvm-svn: 44172

Implement support for variadic methods (work in progress).

llvm-svn: 44171

This fixes code like "if((bool)x) {}" for example.

Patch by Nate Begeman.

llvm-svn: 44159

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