[PATCH] D106897: [lld][WebAssembly] Prefer objdump -d over obj2yaml for tests. NFC

Andy Wingo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 01:25:04 PDT 2021


wingo added a comment.

Neat :)  LGTM as well of course.

An idle thought -- I think it may be in the spirit of llvm-objdump to allow `llvm-objdump --disassemble-all` to produce some kind of structured representation for non-code sections, perhaps in the grammar accepted for .S files.  As it is, it tries to parse the bodies of non-code sections as bytecode, which obviously produces garbage and may run into similar asserts as in https://bugs.llvm.org/show_bug.cgi?id=50957.  That would allow you to use one tool for more things.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106897



More information about the llvm-commits mailing list