[PATCH] D88355: [DWARFYAML] Make the opcode_base and the standard_opcode_lengths fields optional.

Xing GUO via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 05:57:10 PDT 2020


Higuoxing updated this revision to Diff 294677.
Higuoxing marked 3 inline comments as done.
Higuoxing added a comment.

- Make `getStandardOpcodeLengths()` return the standard_opcode_lengths according to the latest DWARF spec by default.
- Add one test case where the opcode_base is greater than 13.
- Address some inline comments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88355

Files:
  llvm/include/llvm/ObjectYAML/DWARFYAML.h
  llvm/lib/ObjectYAML/DWARFEmitter.cpp
  llvm/lib/ObjectYAML/DWARFYAML.cpp
  llvm/test/tools/yaml2obj/ELF/DWARF/debug-line.yaml
  llvm/tools/obj2yaml/dwarf2yaml.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88355.294677.patch
Type: text/x-patch
Size: 12404 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200928/461d81b2/attachment.bin>


More information about the llvm-commits mailing list