[PATCH] D44560: [DWARF] Rework debug line parsing to use llvm::Error and callbacks

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 23 06:13:04 PDT 2018


jhenderson updated this revision to Diff 143531.
jhenderson marked 3 inline comments as done.
jhenderson edited the summary of this revision.
jhenderson removed a subscriber: rafael.
jhenderson added a comment.

Address review comments + make other minor changes:

- Rebase
- Use lower-case 'e' for `emitInt64`
- Update comments for `emitInt64` and new `getLineTable` overload.
- clang-format
- Use updated `WithColor` methods for default printing of warnings.
- Rename custom error class and use this for reporting "fatal" errors, and `StringError` otherwise.


Repository:
  rL LLVM

https://reviews.llvm.org/D44560

Files:
  include/llvm/CodeGen/AsmPrinter.h
  include/llvm/DebugInfo/DWARF/DWARFContext.h
  include/llvm/DebugInfo/DWARF/DWARFDebugLine.h
  lib/CodeGen/AsmPrinter/AsmPrinter.cpp
  lib/DebugInfo/DWARF/DWARFContext.cpp
  lib/DebugInfo/DWARF/DWARFDebugLine.cpp
  test/DebugInfo/X86/dwarfdump-bogus-LNE.s
  test/tools/llvm-dwarfdump/X86/Inputs/debug_line_malformed.s
  test/tools/llvm-dwarfdump/X86/Inputs/debug_line_reserved_length.s
  test/tools/llvm-dwarfdump/X86/debug_line_invalid.test
  tools/dsymutil/DwarfLinker.cpp
  unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
  unittests/DebugInfo/DWARF/DwarfGenerator.cpp
  unittests/DebugInfo/DWARF/DwarfGenerator.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44560.143531.patch
Type: text/x-patch
Size: 65489 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180423/695d6418/attachment.bin>


More information about the llvm-commits mailing list