[cfe-commits] PATCH: Add support for resolving overloaded template functions in Sema::DiagnoseEmptyLookup

Kaelyn Uhrain rikka at google.com
Thu Aug 4 16:07:33 PDT 2011


On Thu, Aug 4, 2011 at 4:03 PM, Chandler Carruth <chandlerc at google.com>wrote:

> FYI
>
>
> On Thu, Aug 4, 2011 at 3:04 PM, Kaelyn Uhrain <rikka at google.com> wrote:
>
>> it also fixes tiny bug in my previous patch that lead to no error message
>> being printed out if the call to BestViableFunction did not succeed
>
>
> Do you have a test case for this?
>

That would be the additions to function-overload-typo-crash.cpp without the
rest of my patch. ;)


>
> Also, can you split this into its own patch, and go ahead and commit it?
>

Yeah, I can do that. It's a one-line change (in the current patch, "return
true;" -> "break;")
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110804/2913d0f5/attachment.html>


More information about the cfe-commits mailing list