[PATCH] D112006: [DebugInfo][InstrRef] Add unit tests for transfer-function building

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 20 03:56:49 PDT 2021


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

Squelch some clang-format complaints; I'm not re-formatting the MIR blocks though, clang-format can't know that the contents of the strings needs to be formatted in a readable way.

I've also fiddled with the "COPY" implementation too: super-registers of the destination weren't being def'd, so

  $rax = MOV64ri 0
  $eax = COPY $ebx

Would be interpreted as $rax still having the value from the MOV, which isn't true. Now all aliases are def'd, and new values copied across for any subregister that's part of the copy. I've also added a further unit test line for that.


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

https://reviews.llvm.org/D112006

Files:
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
  llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.h
  llvm/unittests/CodeGen/InstrRefLDVTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112006.380889.patch
Type: text/x-patch
Size: 25089 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211020/f8f16435/attachment.bin>


More information about the llvm-commits mailing list