[PATCH] D52125: X86, AArch64, ARM: Do not attach debug location to spill/reload instructions

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 2 11:36:05 PDT 2018


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.

> On 2 Oct 2018, at 11:19, Matthias Braun <matze at braunis.de> wrote:
> 
> +CC dsanders
> 
>> On Oct 2, 2018, at 11:17 AM, Matthias Braun <matze at braunis.de> wrote:
>> 
>> Feel free to revert for now, but are you sure it is actually this commit?
>> 
>> 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...
>> 
>> 
>>> On Oct 2, 2018, at 11:06 AM, Matt Morehouse via Phabricator via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>>> 
>>> morehouse added a comment.
>>> 
>>> This change is breaking some symbolization tests on the Android bot:  http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/15630
>>> 
>>> Please fix or revert.
>>> 
>>> 
>>> Repository:
>>> rL LLVM
>>> 
>>> https://reviews.llvm.org/D52125
>>> 
>>> 
>>> 
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>> 
> 



More information about the llvm-commits mailing list