[PATCH] D58386: [DebugInfo] Pre-RA MachineSink: sink DBG_VALUEs that don't immediately follow the sunk instruction too

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 12 02:50:35 PDT 2019


jmorse updated this revision to Diff 209442.
jmorse added a comment.

Refresh this patch:

- Rebase it onto the current open stack
- Remove a bunch of needless attributes and metadata

There's a minor test change too: one of the parent patches now no longer marks DBG_VALUEs undef where it can recover them by copy-propagating. This isn't a behaviour that this patch tests for, so I've updated the checks for the first DBG_VALUE in the test to not be marked "undef".


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58386/new/

https://reviews.llvm.org/D58386

Files:
  lib/CodeGen/MachineSink.cpp
  test/DebugInfo/MIR/X86/machinesink.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58386.209442.patch
Type: text/x-patch
Size: 11027 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190712/3a0de57b/attachment.bin>


More information about the llvm-commits mailing list