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

[David Blaikie]

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)

>
> 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.
>