[PATCH] D45637: [DebugInfo] Ignore DBG_VALUE instructions in PostRA Machine Sink

Matt Davis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 11 10:46:40 PDT 2018


mattd updated this revision to Diff 150789.
mattd edited the summary of this revision.
mattd added a comment.

Thanks for the review @bjope!

My apologies, you are correct, the registers in the DBG_VALUE were getting
captured in `LiveRegUnits::accumulateUsedDefed`  I fixed up the summary
and a few comments in the code/test as well.

- Updated the patch to use isDebugInstr().
- Cor rected some comments and renamed the test to reflect that the functionality is trying to avoid processing debug instructions.


https://reviews.llvm.org/D45637

Files:
  lib/CodeGen/MachineSink.cpp
  test/CodeGen/X86/postra-ignore-dbg-instrs.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45637.150789.patch
Type: text/x-patch
Size: 7196 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180611/a2e44217/attachment.bin>


More information about the llvm-commits mailing list