[PATCH] D70930: [llvm-objcopy][WebAssembly] Initial support for wasm in llvm-objcopy

Fri Dec 13 02:29:47 PST 2019

jhenderson added a comment.

In D70930#1782155 <https://reviews.llvm.org/D70930#1782155>, @dschuff wrote:

> Thanks for all your input!
>  Who else needs to review this? Can you say explicitly, so we can keep this from becoming one of those reviews that hangs in limbo for a long time without any clear path forward?

I'm mostly just interested in getting someone with wasm experience to review this, to make sure there are no wasm-specific details that we might have missed. I don't really have time unfortunately to try to get to grips with the file format myself, otherwise I'd be happy to.

Also, obviously @alexshap's comments should be addressed.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70930