[PATCH] D77576: [globalisel] Add lost debug locations verifier

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 14 16:20:27 PDT 2020


dsanders updated this revision to Diff 257546.
dsanders added a comment.

One more last-minute fix: If there was a line-0 location in the input
and the output it was matching them up. If another location folded
into that line-0 location it would falsely be considered lost despite
being handled correctly because the code to treat line-0 locations
as covering everything was not reached.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77576

Files:
  llvm/include/llvm/CodeGen/GlobalISel/Legalizer.h
  llvm/include/llvm/CodeGen/GlobalISel/LostDebugLocObserver.h
  llvm/lib/CodeGen/GlobalISel/CMakeLists.txt
  llvm/lib/CodeGen/GlobalISel/Legalizer.cpp
  llvm/lib/CodeGen/GlobalISel/LostDebugLocObserver.cpp
  llvm/unittests/CodeGen/GlobalISel/LegalizerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77576.257546.patch
Type: text/x-patch
Size: 15157 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200414/f798d09a/attachment.bin>


More information about the llvm-commits mailing list