[PATCH] D61323: [WebAssembly] Support EXPLICIT_NAME symbols in llvm-readobj

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 30 12:05:41 PDT 2019


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

I was thinking of embedding a minimal yaml in the test itself (and calling it test/tools/llvm-readobj/wasm-imports.yaml)..  e.g. llvm/test/Object/wasm-duplicate-name.test or llvm/test/tools/llvm-readobj/wasm-invalid.test.

I don't feel strongly about it though.  lgtm either way.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61323





More information about the llvm-commits mailing list