Regressions are not OK. Unless the fix is imminent, we should revert and work on a correct patch.<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 28, 2012 at 4:08 PM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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>
<div class="HOEnZb"><div class="h5"><br>
On Sat, Jun 23, 2012 at 4:50 PM, Charles Davis <<a href="mailto:cdavis@mymail.mines.edu">cdavis@mymail.mines.edu</a>> 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 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>
</div></div></blockquote></div><br></div>