[PATCH] D84817: Add verification for DW_AT_decl_file and DW_AT_call_file.

[Greg Clayton via Phabricator via llvm-commits]

clayborg updated this revision to Diff 283124.
clayborg added a comment.

Made DWARFDebugLine::Prologue::getLastValidFileIndex() and DWARFDebugLine::getLastValidFileIndex() return an Optional<uint64_t> and modified the error message for when a file index is out of range to specify the valid range, or specify that there are no files in the prologue file table. Added a test to cover this case as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84817/new/

https://reviews.llvm.org/D84817

Files:
  llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h
  llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
  llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp
  llvm/test/tools/llvm-dwarfdump/X86/verify_attr_file_indexes.yaml
  llvm/test/tools/llvm-dwarfdump/X86/verify_attr_file_indexes_no_files.yaml
  llvm/test/tools/llvm-dwarfdump/X86/verify_debug_info.s
  llvm/test/tools/llvm-dwarfdump/X86/verify_file_encoding.yaml

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84817.283124.patch
Type: text/x-patch
Size: 19758 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200805/a2daa607/attachment.bin>