<div style="font-family: arial, helvetica, sans-serif"><font size="2">I'm sorry, but for now I would ask you to not submit patches for unpublished or undocumented ABI features.<div><br></div><div>There are some very limited places where we can take patches for such features, but it's not yet clear to me that this is one. I'll talk to some others about this particular instance.</div>
<div><br></div><div>We're still working out some aspects of the Windows support strategy for Clang, but it would help a lot to focus efforts on fixing correctness issues with the published (on MSDN) ABI specifications.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 26, 2012 at 11:25 PM, João Matos <span dir="ltr"><<a href="mailto:ripzonetriton@gmail.com" target="_blank">ripzonetriton@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Chandler, most of the details of the MS ABI are not publicly<br>
documented, as far as I know there is no spec.<br>
<br>
The only reference I can find is this:<br>
<a href="http://msdn.microsoft.com/en-us/library/ff798096.aspx" target="_blank">http://msdn.microsoft.com/en-us/library/ff798096.aspx</a><br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jun 27, 2012 at 7:13 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
> Can you include a citation of the spec you're basing the contents of the<br>
> patch off of please?<br>
><br>
><br>
> On Mon, Jun 25, 2012 at 2:36 PM, João Matos <<a href="mailto:ripzonetriton@gmail.com">ripzonetriton@gmail.com</a>> wrote:<br>
>><br>
>> AFAIK this is independent of bitness. It's just the equivalent of<br>
>> "__cxa_pure_virtual" in the Itanium ABI and is used also in the 64bit<br>
>> MS ABI.<br>
>><br>
>> On Mon, Jun 25, 2012 at 10:19 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>><br>
>> wrote:<br>
>> > On Sun, Jun 24, 2012 at 5:43 PM, João Matos <<a href="mailto:ripzonetriton@gmail.com">ripzonetriton@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Attached is a patch that abstract the virtual pure call C++ ABI. It<br>
>> >> also implements the correct one for the MS ABI.<br>
>> ><br>
>> ><br>
>> > What ABI are you basing this on?<br>
>> ><br>
>> > Currently, Clang is focused on Win64 support based on the published<br>
>> > win64<br>
>> > ABI specs, so it may take some time for this patch to get reviewed<br>
>> > otherwise...<br>
>><br>
>><br>
>><br>
>> --<br>
>> João Matos<br>
><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
João Matos<br>
</font></span></blockquote></div><br></div></font></div>