<div dir="ltr"><div dir="ltr">Bot went green after revert:  <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/15666" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/15666</a></div><div dir="ltr"><br></div><div><a class="gmail_plusreply" id="m_5271213106249862186plusReplyChip-2" href="mailto:eugenis@google.com" target="_blank">+Evgenii Stepanov</a> for debugging advice.<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 2, 2018 at 11:55 AM Matthias Braun via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yes, it seems only some of the failures where GlobalISel related and some are still around.<br>
<br>
Trying to understand what's going on here, since I suspect I am not able to run the hwasan tests myself (or am I when I am in an apple universe?):<br>
<br>
It seems that the tool prints every memory read and write. I suspect some of the reads and writes that get picked up are actually spills and reloads which after my change have slightly different debug locations attached (they really shouldn't have any debug information attached, but now happen to pick up the location of whatever is before the spill/reload instead of whatever is after). If those tests are indeed accidentally picking up spills/reloads instead of the load/stores they want to be testing for, then how can we fix this?<br>
<br>
> On Oct 2, 2018, at 11:36 AM, Daniel Sanders <<a href="mailto:daniel_l_sanders@apple.com" target="_blank">daniel_l_sanders@apple.com</a>> wrote:<br>
> <br>
> I reverted the GlobalISel change (r343521) yesterday so it shouldn't be causing any further failures. I only see hwasan failures on the latest build.<br>
> <br>
>> On 2 Oct 2018, at 11:19, Matthias Braun <<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>> wrote:<br>
>> <br>
>> +CC dsanders<br>
>> <br>
>>> On Oct 2, 2018, at 11:17 AM, Matthias Braun <<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>> wrote:<br>
>>> <br>
>>> Feel free to revert for now, but are you sure it is actually this commit?<br>
>>> <br>
>>> It's hard to say for sure what happenes with the earlier hwasan tests, but the later tests that failed are crashing in GlobalISel related code, which should all run before the code that I changed...<br>
>>> <br>
>>> <br>
>>>> On Oct 2, 2018, at 11:06 AM, Matt Morehouse via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>>>> <br>
>>>> morehouse added a comment.<br>
>>>> <br>
>>>> This change is breaking some symbolization tests on the Android bot:  <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/15630" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/15630</a><br>
>>>> <br>
>>>> Please fix or revert.<br>
>>>> <br>
>>>> <br>
>>>> Repository:<br>
>>>> rL LLVM<br>
>>>> <br>
>>>> <a href="https://reviews.llvm.org/D52125" rel="noreferrer" target="_blank">https://reviews.llvm.org/D52125</a><br>
>>>> <br>
>>>> <br>
>>>> <br>
>>>> _______________________________________________<br>
>>>> llvm-commits mailing list<br>
>>>> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
>>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
>>> <br>
>> <br>
> <br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>