<div dir="ltr">Could you, please, commit it?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 21 February 2013 01:33, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM, Thanks!<br>
<div class="im"><br>
On 20 February 2013 12:43, Alexander Zinenko <<a href="mailto:ftynse@gmail.com">ftynse@gmail.com</a>> wrote:<br>
</div><div><div class="h5">> clang -cc1 -cxx-abi microsoft -triple i686-pc-win32 as of now behaves as<br>
> follows<br>
><br>
> class A {<br>
>  public:<br>
>   static void foo();<br>
> };<br>
><br>
> void __cdecl A::foo() {}  // Error<br>
> void __thiscall A::foo() {}  // OK<br>
><br>
> __thiscall is invalid for non-instance methods, though. And __cdecl is an<br>
> implicit default.<br>
><br>
> The problem is in MergeFunctionDecl: New function decl doesn't have<br>
> information about static storage duration yet, only the Old one, therefore<br>
> wrong default CC is checked.<br>
><br>
> The patch also adds a test for implicit calling convention compatibility in<br>
> declarations.<br>
><br>
> Please review!<br>
><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div>