[PATCH] D71768: [DebugInfo] Remove redundant checks for past-the-end of prologue
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 20 07:18:28 PST 2019
jhenderson created this revision.
jhenderson added reviewers: ikudrin, dblaikie, JDevlieghere, probinson, MaskRay.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.
The V5 directory and filename table parsing had checks in to make sure we hadn't read past the end of the line tabel prologue. Since previous changes to the data extractor class sure we never read past the end, these checks are now redundant, so this patch removes them.
There is still a check to show that the whole prologue remains within the prologue length, which is now hit instead.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D71768
Files:
llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
llvm/test/tools/llvm-dwarfdump/X86/debug_line_invalid.test
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71768.234895.patch
Type: text/x-patch
Size: 5601 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191220/aa488861/attachment.bin>
More information about the llvm-commits
mailing list