[PATCH] D47369: [DebugInfo][ScheduleDAGInstrs] Prevent scheduler from reordering DBG_VALUE instructions through their clobbering MIs

Fri May 25 08:03:56 PDT 2018

andrew.zhogin added a comment.

In https://reviews.llvm.org/D47369#1112421, @dberris wrote:

> In https://reviews.llvm.org/D47369#1112419, @andrew.zhogin wrote:
>
> > In https://reviews.llvm.org/D47369#1112341, @dberris wrote:
> >
> > > Is this only an issue with AArch64, or is this more generic? Have you seen how this applies to other targets (say X86)?
> >
> >
> > I have reproduced it for AArch64, but it is generic problem and may occur for any target. And cut debug-loc liveranges for some variables (with >=https://reviews.llvm.org/owners/package/1/ optimizations).
>
>
> Okay, so will this break any of the other target tests? Can the test be made more generic alongside any existing debug info tests in codegen?

For sure, I did check-all runs. Existing tests are fine with the patch (most use -O0 compilation).
I will try to make the test more generic.

Repository:
  rL LLVM

https://reviews.llvm.org/D47369