[PATCH] D31755: [DebugInfo][X86] Fix handling of DBG_VALUE's in post-RA scheduler.

Andrew Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 05:27:50 PDT 2017


andrewng created this revision.

This patch fixes a bug with the updating of DBG_VALUE's in
BreakAntiDependencies. Previously, it would only attempt to update the first
DBG_VALUE following the instruction whose register is being changed,
potentially leaving DBG_VALUE's referring to the wrong register. Now the code
will attempt to update all DBG_VALUE's that immediately follow the instruction.

This issue was detected as a result of an optimized codegen difference with
"-g" where an X86 byte/word fixup was not performed due to a DBG_VALUE
referencing the wrong register.


https://reviews.llvm.org/D31755

Files:
  lib/CodeGen/AggressiveAntiDepBreaker.cpp
  lib/CodeGen/AntiDepBreaker.h
  lib/CodeGen/CriticalAntiDepBreaker.cpp
  test/CodeGen/X86/post-ra-sched-with-debug.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31755.94351.patch
Type: text/x-patch
Size: 27569 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170406/a8aecacb/attachment.bin>


More information about the llvm-commits mailing list