<div dir="ltr">On Tue, Jun 18, 2013 at 6:25 AM, Benno Rice <span dir="ltr"><<a href="mailto:benno@freebsd.org" target="_blank">benno@freebsd.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div>
<div>On 12/06/2013, at 10:40 PM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:</div><br></div><blockquote type="cite"><div dir="ltr"><div>On Wed, Jun 12, 2013 at 7:13 AM, Charles Davis <span dir="ltr"><<a href="mailto:cdavis5x@gmail.com" target="_blank">cdavis5x@gmail.com</a>></span> wrote:<br>

</div><div><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div><div>
<div>You mean like in my patch here:</div></div></div></div><div><br></div><div><a href="http://llvm-reviews.chandlerc.com/D729" target="_blank">http://llvm-reviews.chandlerc.com/D729</a></div><div><div>
<br></div><div>?</div></div></div></blockquote><div><br></div><div>Yeah, that looks good to me.  :)</div><div><br></div><div>From the docs, it seems attr(ms_abi) just maps to x86_64_win64cc and x86_stdcallcc depending on the architecture:</div>


<div><a href="http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html#index-g_t_0040code_007bsysv_005fabi_007d-attribute-2670" target="_blank">http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html#index-g_t_0040code_007bsysv_005fabi_007d-attribute-2670</a></div>


<div>gcc also appears to support a sysv_abi attribute, which lets you use the sysv CCs when targeting Windows.</div></div></div></div></div>
</blockquote></div><br><div>So what's in the way of this being committed? Is there anything I can do that would accelerate things?</div></div></blockquote><div><br></div><div>Charles, can you commit <a href="http://llvm-reviews.chandlerc.com/D729" target="_blank">http://llvm-reviews.chandlerc.com/D729</a>, or is something blocking that?<br>

</div><div><br></div><div>For the LLVM side, Charles's patch looks preferable to me.  I'd apply his patch to LLVM and base your clang patch on that, and resend.</div><div><br></div><div>You should also answer Richard's question about whether the types are canonically different.</div>

<div><br></div><div>For i686, does ms_abi map to stdcall?  Does gcc consider them canonically different (can you form different overloads)?</div><div><br></div><div>For x86_64, can I form an overload between ms_abi and normal function pointers?</div>

<div><br></div></div></div></div>