[PATCH] D44220: [DWARF] Suppress split line tables more carefully.

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 7 11:33:59 PST 2018


probinson created this revision.
probinson added reviewers: dblaikie, aprantl.
probinson added a project: debug-info.
Herald added subscribers: llvm-commits, JDevlieghere, hiraditya.

If a given split type unit does not have source locations, don't have
it refer to the split line table.
If no split type unit refers to the split line table, don't emit the
line table at all.

This will save a little space on rare occasions, but also refactors
things a bit to improve which class is responsible for what.

Responding to review comments on r326395.


Repository:
  rL LLVM

https://reviews.llvm.org/D44220

Files:
  llvm/include/llvm/MC/MCDwarf.h
  llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
  llvm/lib/CodeGen/AsmPrinter/DwarfDebug.h
  llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
  llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h
  llvm/lib/MC/MCDwarf.cpp
  llvm/test/CodeGen/X86/dwarf-split-line-1.ll
  llvm/test/CodeGen/X86/dwarf-split-line-2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44220.137444.patch
Type: text/x-patch
Size: 9635 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180307/b8b37bf5/attachment.bin>


More information about the llvm-commits mailing list