[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