[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
Thu Mar 22 07:49:18 PDT 2018
jhenderson updated this revision to Diff 139440.
jhenderson added a comment.
Lots of small changes:
- Rename some functions, tests and parameters in the unit tests, to remove reference to Minor/Major, reflecting the changein the code.
- Added checking of the Prologue fields to the valid table case.
- Added a missing argument to one of the test functions, causing it to test the wrong thing.
- Added a TODO to extend the valid-table testing to verify the body of the program. I chose not to do this at this time, as it requires fairly comprehensive testing, and the behaviour isn't changing.
- Reduced the number of test cases used for the paramterised tests, based on comments on llvm-dev.
Repository:
rL LLVM
https://reviews.llvm.org/D44560
Files:
include/llvm/CodeGen/AsmPrinter.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/DebugInfo/X86/dwarfdump-line-dwo.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.139440.patch
Type: text/x-patch
Size: 63574 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180322/e62ec0db/attachment.bin>
More information about the llvm-commits
mailing list