<div dir="ltr">For what it's worth, I'm very much in favor of fixing the ASAN bug asap, rather than reverting this patch.<div>I need this patch to fix a lot of regressions in the code base I'm working on. That is besides the fact that it actually lifts the requirement for users to upgrade their libraries. </div><div><br></div><div><br></div><div>Thanks,</div><div>-- Larisse.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 23, 2015 at 3:16 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 23 February 2015 at 11:07, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
> Green bots are no doubt better than red bots or no bots, but I'm not sure<br>
> what level of ARM support our team has committed to.<br>
<br>
</span>I appreciate that, but you're not the only sanitizer developers out<br>
there. If we found a bug in ASAN for ARM, we have to fix it. If that<br>
means involving other groups / companies (like Linaro), than so be it.<br>
<br>
You may not have committed to any ARM sanitizer work, but we have. So,<br>
if there is an issue with one of your patches, we must help you solve<br>
them. And, in the same way as any other project in the LLVM sphere,<br>
you can't just XFAIL tests on another platform just because you<br>
haven't committed to working on that platform.<br>
<br>
Two years ago I have fixed all test-suite failures for ARM, and<br>
coincidentally, I have also fixed it for PowerPC and Intel. On one<br>
occasion, I have reverted my change and worked with other developers<br>
to make it work on those platforms, too. You can imagine that I also<br>
haven't committed to work on Intel or PPC, but that's what it takes to<br>
get the product stable. Overall, I don't trust in my solution if it<br>
only works for my own architecture. In the long run, it's better for<br>
you if it continues working for ARM.<br>
<br>
However, despite all of that, it's our community's policy to not hide<br>
failures under the rug, especially if it concerns a first-class<br>
architecture, like ARM. The sanitizers are not a local project any<br>
more, they're part of our community, and we have to care for it like<br>
we care for all others.<br>
<div class="HOEnZb"><div class="h5"><br>
cheers,<br>
--renato<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>