[PATCH] D99169: [DebugInfo] Replace debug uses in replaceUsesOutsideBlock

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 23 03:46:09 PDT 2021


Orlando created this revision.
Orlando added reviewers: jmorse, vsk, aprantl, rnk, dblaikie.
Orlando added projects: debug-info, LLVM.
Herald added subscribers: dexonsmith, hiraditya.
Orlando requested review of this revision.
Herald added a subscriber: llvm-commits.

Value::replaceUsesOutsideBlock doesn't replace debug uses which leads to an
unnecessary reduction in variable location coverage. Fix this, add a unittest for
it, and add a regression test demonstrating the change through instcombine's
replacedSelectWithOperand.


https://reviews.llvm.org/D99169

Files:
  llvm/lib/IR/Value.cpp
  llvm/test/DebugInfo/Generic/instcombine-replaced-select-with-operand.ll
  llvm/unittests/IR/ValueTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99169.332599.patch
Type: text/x-patch
Size: 11442 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210323/3b4fb288/attachment.bin>


More information about the llvm-commits mailing list