<div dir="ltr">Ok, I'll do that.<div><br></div><div style>Thanks,</div><div>Amara</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 May 2013 00:25, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, May 3, 2013 at 4:09 PM, Amara Emerson <<a href="mailto:amara.emerson@gmail.com">amara.emerson@gmail.com</a>> wrote:<br>


> Hi Bill,<br>
><br>
> This seems to be due to some compiler dependent failures which I'm currently<br>
> investigating. I just removed it to prevent polluting the buildbots further,<br>
> it'll be re-committed with the fix, otherwise I'll revert the original<br>
> feature commit.<br>
<br>
</div>Usually we do this by XFAILing the test rather than deleting it.<br>
<br>
(also, even that's a questionable practice - if you just committed<br>
something that broke the bots, the strategy should be to revert your<br>
patch until you understand why it's broken (be it a test bug or a<br>
production bug))<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Regards,<br>
> Amara<br>
><br>
><br>
> On 3 May 2013 23:58, Bill Wendling <<a href="mailto:wendling@apple.com">wendling@apple.com</a>> wrote:<br>
>><br>
>> On May 3, 2013, at 3:39 PM, Amara Emerson <<a href="mailto:amara.emerson@arm.com">amara.emerson@arm.com</a>> wrote:<br>
>><br>
>> > Author: aemerson<br>
>> > Date: Fri May  3 17:39:03 2013<br>
>> > New Revision: 181066<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=181066&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=181066&view=rev</a><br>
>> > Log:<br>
>> > Delete test instead.<br>
>> ><br>
>> Hi Amara,<br>
>><br>
>> Does this test point to a real problem? or is it just not a good test? If<br>
>> the former, you should keep this test around and fix the problem. :-)<br>
>><br>
>> -bw<br>
>><br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
</div></div></blockquote></div><br></div>