<div dir="ltr">I'll look into this.  I think we should just print the unadjusted type if we have an adjusted function type under a member pointer.  This should be an easy type printer fix.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Dec 10, 2013 at 11:58 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Mon, Dec 9, 2013 at 5:18 PM, Rafael Espíndola<br>
<<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> Mingw switched abis with the release of gcc 4.7<br>
> (<a href="http://gcc.gnu.org/gcc-4.7/changes.html" target="_blank">http://gcc.gnu.org/gcc-4.7/changes.html</a>). The main change is that now<br>
> mingw (like msvc) given thiscall calling convention to methods by<br>
> default.<br>
><br>
> I think the last bug blocking us to support the new abi has just been<br>
> fixed. The question now is how to switch.<br>
<br>
</div>I'm still seeing some test failures. For example,<br>
CXX/class.access/p6.cpp fails if I apply your patch and run with<br>
-triple i686-pc-mingw32:<br>
<br>
$ "D:/src/llvm/build.release/bin/./clang.EXE" "-cc1" -triple<br>
i686-pc-mingw32 "-internal-isystem"<br>
"D:\src\llvm\build.release\bin\..\lib\clang\3.4\include"<br>
"-fsyntax-only" "-verify"<br>
"D:\src\llvm\tools\clang\test\CXX\class.access\p6.cpp"<br>
error: 'error' diagnostics expected but not seen:<br>
  File D:\src\llvm\tools\clang\test\CXX\class.access\p6.cpp Line 180:<br>
'operator void *(class test8::A::*)(void) const' is a private member<br>
of 'test8::A'<br>
error: 'error' diagnostics seen but not expected:<br>
  File D:\src\llvm\tools\clang\test\CXX\class.access\p6.cpp Line 180:<br>
'operator void *(class test8::A::*)(void) __attribute__((thiscall))<br>
const' is a private member of 'test8::A'<br>
2 errors generated.<br>
<br>
There are a handful other tests failing in the same way.<br>
<br>
It would be nice if we could make the TypePrinter not print the<br>
calling convention if it's the default one for the ABI, but<br>
TypePrinter doesn't have a lot of context.. no Sema, no ASTContext :/<br>
<span class="HOEnZb"><font color="#888888"><br>
 - Hans<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br></div>