[cfe-commits] r49699 - in /cfe/trunk: Driver/ASTConsumers.cpp include/clang/AST/DeclObjC.h include/clang/Parse/Action.h include/clang/Parse/Parser.h lib/AST/DeclObjC.cpp lib/Parse/ParseObjc.cpp lib/Sema/Sema.h lib/Sema/SemaDeclObjC.cpp

[Fariborz Jahanian]

On Apr 15, 2008, at 4:34 PM, Chris Lattner wrote:

> On Apr 14, 2008, at 4:36 PM, Fariborz Jahanian wrote:
>> Author: fjahanian
>> Date: Mon Apr 14 18:36:35 2008
>> New Revision: 49699
>> URL: http://llvm.org/viewvc/llvm-project?rev=49699&view=rev
>> Log:
>> New AST representation for each objc2's property declaration.
>
> Nice, thanks Fariborz!
>
> I think that the propertydecl enum in DeclBase.h needs to be moved  
> to reflect its new place in the class hierarchy.

I am not sure what you mean. ObjCPropertyDecl is derived from  
NamedDecl and it seems to be in the correct position.

>
>
> Also, given the new approach, should ObjCPropertyDecl inherit from  
> FieldDecl, what do you think?

Think of a property declaration as macro for declaring setters and  
getters. It really has nothing to do with fields.

- Fariborz

>
>
> -Chris
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20080415/840fde8b/attachment.html>