[PATCH] D86152: [DwarfDebug] Improve multi-BB single location detection in validThroughout (3/4)

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 18 10:16:02 PDT 2020


Orlando created this revision.
Orlando added projects: LLVM, debug-info.
Herald added subscribers: llvm-commits, hiraditya.
Orlando requested review of this revision.

With the changes introduced in D86151 <https://reviews.llvm.org/D86151> we can now easily check for single locations
which span multiple blocks for inlined scopes and blocks.

CTMark shows a geomean binary size reduction of 2.2% for RelWithDebInfo builds.
llvm-locstats (using D85636 <https://reviews.llvm.org/D85636>) shows a very small variable location coverage
change in 5 of 10 binaries, but just like in D86151 <https://reviews.llvm.org/D86151> it is only in the order of
10s of bytes.


https://reviews.llvm.org/D86152

Files:
  llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
  llvm/test/DebugInfo/X86/location-range-inlined-xblock.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86152.286327.patch
Type: text/x-patch
Size: 10932 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200818/1f1bcf8d/attachment.bin>


More information about the llvm-commits mailing list