[PATCH] D81562: [DebugInfo] Unify Cursor usage for all debug line opcodes

[James Henderson via Phabricator via llvm-commits]

jhenderson created this revision.
jhenderson added reviewers: JDevlieghere, dblaikie, MaskRay, aprantl, ikudrin, probinson.
Herald added subscribers: cmtice, hiraditya.
Herald added a project: LLVM.
jhenderson edited the summary of this revision.
jhenderson added a parent revision: D81470: [DebugInfo] Report errors for truncated debug line standard opcode.
jhenderson added a reviewer: labath.
jhenderson added a child revision: D81570: [DebugInfo] Don't print extended opcode operands if invalid.

This is a natural extension of the previous changes to use the Cursor class independently in the standard and extended opcode paths, and in turn allows delaying error handling until the entire line has been printed in verbose mode, removing interleaved output in some cases.

Depends on D81470 <https://reviews.llvm.org/D81470>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D81562

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: D81562.269812.patch
Type: text/x-patch
Size: 4457 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200610/ad79a86d/attachment.bin>