[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