[cfe-commits] r51888 - in /cfe/trunk: include/clang/AST/ExprObjC.h include/clang/AST/StmtNodes.def lib/AST/Expr.cpp lib/AST/StmtPrinter.cpp lib/AST/StmtSerialization.cpp lib/Sema/SemaExpr.cpp test/Sema/objc-property-7.m

[Chris Lattner]

On Jun 2, 2008, at 10:42 PM, Steve Naroff wrote:

>
> On Jun 2, 2008, at 10:10 PM, Chris Lattner wrote:
>
>> On Jun 2, 2008, at 4:03 PM, Steve Naroff wrote:
>>> URL: http://llvm.org/viewvc/llvm-project?rev=51888&view=rev
>>> Log:
>>> Fix <rdar://problem/5979875> clang on xcode: error: use of  
>>> undeclared identifier 'super'
>>
>> Hey Steve,
>>
>> Instead of a whole new AST node for this, how about adding a new  
>> enum to PreDefinedExpr?  Does the rewriter handle this properly?
>
> That sounds simpler - thanks for pointing it out.
>
> I don't believe the rewriter currently deals with PreDefinedExpr  
> (but that shouldn't be a problem).

Right, I'm just saying whatever old logic you had for handling  
'super' (if any!) should be moved over to handle the new AST  
representation.

-Chris