[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