Ideally without copying code, I think we should implement the strategy Timur, John and I discussed where there is a flag to enable Itanium-fallback in the MS ABI to cover cases where that is sufficient for forward progress. <div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Jun 28, 2012 at 5:07 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">
The fix is easy: Copy the code from the Itanium case to the Microsoft<br>
one. While it's not correct, it matches the old behaviour while<br>
keeping the abstraction of this patch.<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Jun 29, 2012 at 12:24 AM, Timur Iskhodzhanov<br>
<<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
> FYI, by "regression" I meant there was a functionality which was not<br>
> broken but rather disabled as a whole :)<br>
><br>
> On Thu, Jun 28, 2012 at 4:21 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
>> Regressions are not OK. Unless the fix is imminent, we should revert and<br>
>> work on a correct patch.<br>
>><br>
>><br>
>> On Thu, Jun 28, 2012 at 4:08 PM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>><br>
>> wrote:<br>
>>><br>
>>> FTR, this patch has regressed:<br>
>>> <a href="http://llvm.org/bugs/show_bug.cgi?id=13231" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=13231</a><br>
>>><br>
>>> Now I can't build non-trivial C++ code anymore.<br>
>>> [I had local patches to work OK on some files and was planning to send<br>
>>> a few things for review but now I'm literally stuck at ~r158985]<br>
>>><br>
>>> Please fix this ASAP or I'll have to revert it (locally, maybe; but<br>
>>> this is still not cool).<br>
>>><br>
>>> On Sat, Jun 23, 2012 at 4:50 PM, Charles Davis <<a href="mailto:cdavis@mymail.mines.edu">cdavis@mymail.mines.edu</a>><br>
>>> wrote:<br>
>>> ><br>
>>> > On Jun 23, 2012, at 1:11 PM, John McCall wrote:<br>
>>> >> The remaining trivial patch, which just declares, defines, and calls<br>
>>> >> EmitVTables, is approved.<br>
>>> > Thanks, r159091.<br>
>>> ><br>
>>> > Chip<br>
>>> ><br>
>>> ><br>
>>> > _______________________________________________<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>
>>> 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>
>><br>
> _______________________________________________<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>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
João Matos<br>
</font></span></blockquote></div><br></div>