<div dir="ltr">And we can disable the test on ARM for now to make the bot green</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 25, 2015 at 8:17 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Can we investigate offline and fix, instead of reverting? <div>Can you run the test manually and show what does it print? </div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 25, 2015 at 5:05 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">Hi Kostya,<br>
<br>
The fix didn't work, shall we revert the patch and investigate offline?<br>
<br>
cheers,<br>
--renato<br>
<div><div><br>
On 24 February 2015 at 11:38, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br>
> On 23 February 2015 at 20:40, Kostya Serebryany <<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>> wrote:<br>
>> Author: kcc<br>
>> Date: Mon Feb 23 14:40:53 2015<br>
>> New Revision: 230256<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=230256&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=230256&view=rev</a><br>
>> Log:<br>
>> [asan] when registering globals, use the same unwinder as we use for malloc, instead of the one used for FATAL crash (which may be too slow)<br>
><br>
> Hi Kostya,<br>
><br>
> This patch broke this test:<br>
><br>
> <a href="http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/3888/steps/ninja%20check%201/logs/AddressSanitizer-arm-linux%3A%3Aodr-violation.cc" target="_blank">http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/3888/steps/ninja%20check%201/logs/AddressSanitizer-arm-linux%3A%3Aodr-violation.cc</a><br>
><br>
> I reverted your patch locally and the test then passes. Can you have a<br>
> look, please?<br>
><br>
> cheers,<br>
> -renato<br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>