[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