On Thu, Nov 8, 2012 at 10:35 PM, David Blaikie <[email protected]> wrote:
> On Thu, Nov 8, 2012 at 10:33 PM, Brian Brooks <[email protected]> wrote:
>> The assert fails if method is static, but the Decl should also be marked
>> invalid.  I didn't try this.
>
> Yeah, that's what I had in mind:
>
> http://clang.llvm.org/doxygen/DeclBase_8h_source.html#l00465
>
> (on the off chance there are other kinds of invalidity that could
> cause problems with printing out a candidate, for example)

Done in r167609, thanks.

>
>>
>> I went down a bad path of not returning the Decl if it was marked invalid
>> due to static function member.  That caused a handful of issues with
>> existing test cases.
>>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to