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

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 16:41:36 PDT 2021


sbc100 added a comment.

In D106897#2908921 <https://reviews.llvm.org/D106897#2908921>, @dschuff wrote:

> I'd say from the llvm-objdump perspective (which only thinks of sections and not anything more finegrained other than instructions), anything other than the section header would be payload, including subsections etc. Unless we want llvm-objdump to be even smarter, a la wasm-objdump. I'm not sure if there's precedent for that in other formats or not.

Even if we do that it doesn't really help our use case here since the `DATA` section still has some seeming random bytes that describe the segments, which follow the section header and precede any actual user data.


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