[PATCH] D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC).

[Wolfgang Pieb via Phabricator via llvm-commits]

wolfgangp updated this revision to Diff 162310.
wolfgangp marked 9 inline comments as done.
wolfgangp added a comment.

Addressed David's review comments:

- Changed the dumping of DWARF v4 range lists to resemble the new format and updated the tests. More detailed comments are inline.
- Use DW_RLE_offset_pair to model the DWARF 4 range list entries.
- Removed some dead code that will not be used for this patch.
- Answered other miscellaneous comments.

Addressed Victor's comments:

- renamed length() to getLength() for consistency.
- not sure if I want to store the DWARF version in the list entries.


https://reviews.llvm.org/D51081

Files:
  include/llvm/DebugInfo/DWARF/DWARFDebugRangeList.h
  include/llvm/DebugInfo/DWARF/DWARFDebugRnglists.h
  include/llvm/DebugInfo/DWARF/DWARFListTable.h
  include/llvm/DebugInfo/DWARF/DWARFUnit.h
  lib/DebugInfo/DWARF/CMakeLists.txt
  lib/DebugInfo/DWARF/DWARFContext.cpp
  lib/DebugInfo/DWARF/DWARFDebugRangeList.cpp
  lib/DebugInfo/DWARF/DWARFDebugRnglists.cpp
  lib/DebugInfo/DWARF/DWARFDie.cpp
  lib/DebugInfo/DWARF/DWARFListTable.cpp
  lib/DebugInfo/DWARF/DWARFUnit.cpp
  test/DebugInfo/Generic/cu-ranges.ll
  test/DebugInfo/Inputs/gmlt.ll
  test/DebugInfo/X86/dwarfdump-ranges-baseaddr.s
  test/DebugInfo/X86/dwarfdump-ranges-unrelocated.s
  test/DebugInfo/dwarfdump-ranges.test
  test/MC/ARM/dwarf-asm-multiple-sections.s
  tools/dsymutil/DwarfLinker.cpp
  tools/dsymutil/DwarfStreamer.cpp
  tools/dsymutil/DwarfStreamer.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51081.162310.patch
Type: text/x-patch
Size: 59724 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180824/49f0d49f/attachment.bin>