[PATCH] D85094: [obj2yaml] Add support for dumping the .debug_aranges section.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 01:02:29 PDT 2020


jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

LGTM. If you want to emit a warning for unparseable contents, you migth want to do it more widely for other sections too, but in a separate patch. Up to you.



================
Comment at: llvm/tools/obj2yaml/elf2yaml.cpp:409
+      // If the DWARF section cannot be successfully parsed, emit raw content
+      // instead or an entry in the DWARF section of the YAML.
+      if (Err)
----------------
Always check what I wrote makes sense - I make typos too :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85094



More information about the llvm-commits mailing list