[cfe-commits] r44171 - in /cfe/trunk: Driver/RewriteTest.cpp Parse/ParseObjc.cpp Sema/Sema.h Sema/SemaDecl.cpp include/clang/AST/DeclObjC.h include/clang/Parse/Action.h

[Steve Naroff]

On Nov 15, 2007, at 9:24 AM, Chris Lattner wrote:

>> URL: http://llvm.org/viewvc/llvm-project?rev=44171&view=rev
>> Log:
>>
>> Implement support for variadic methods (work in progress).
>
> hey Steve,
>
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =====================================================================
>> --- cfe/trunk/include/clang/AST/DeclObjC.h (original)
>> +++ cfe/trunk/include/clang/AST/DeclObjC.h Thu Nov 15 06:35:21 2007
>> @@ -626,6 +626,8 @@
>>   /// declared in class Decl.
>>   /// instance (true) or class (false) method.
>>   bool IsInstance : 1;
>> +  bool IsVariadic : 1;
>> +
>
> Should "isvariadic" be a property of the methoddecl or should it be  
> a property of the selector?  Doesn't the message expression also  
> need this?
>

Traditionally, it has been associated with the method decl.

I added a "numExprs" slot to message expression. I was previously  
using the selector for this info...

snaroff

> -Chris
>