[patch] Further issues when thiscall is the default for methods.

[Rafael Espíndola]

The previous patches tried to deduce the correct function type. I now
don't think that is possible in general. Consider

class foo {
    template <typename T> static void bar(T v);
};
extern template void foo::bar(const void *);

We will only now that bar is static after a lookup, so it looks like
we have to handle this in the template instantiation code. This also
makes me suspicious of the previous users of FreeFunction always being
correct.

The attached patch reverts my previous two (but not the tests) and
instead handles the issue in DeduceTemplateArguments.

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: application/octet-stream
Size: 12362 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131119/66bb780d/attachment.obj>