[PATCH] D81570: [DebugInfo] Don't print extended opcode operands if invalid

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 16 06:04:09 PDT 2020


jhenderson added a comment.

In D81570#2095372 <https://reviews.llvm.org/D81570#2095372>, @ikudrin wrote:

> Would not it be more useful if we dump a sequence of raw bytes, as we do for DWARF expressions? See `DWARFExpression::print()` and test `llvm/test/tools/llvm-dwarfdump/X86/verify_broken_exprloc.s` as an example.


Yes, it's probably possible somehow. I'll take a look. It would lead to slightly inconsistent behaviour with standard opcodes, where we don't have a known length for the operands, of course.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81570





More information about the llvm-commits mailing list