[cfe-commits] r167604 - in /cfe/trunk: lib/Sema/SemaOverload.cpp test/SemaCXX/overloaded-operator-decl.cpp

[Nico Weber]

On Thu, Nov 8, 2012 at 10:35 PM, David Blaikie <dblaikie at gmail.com> wrote:
> On Thu, Nov 8, 2012 at 10:33 PM, Brian Brooks <brooks.brian at gmail.com> 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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits