[PATCH] D39107: [WebAssembly] MC: Don't allow zero sized data segments

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 16:23:29 PDT 2017


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

Yeah, this seems OK for now.
I think at some point we may want to revisit the overall goals of the representation that we discussed originally (i.e. not having to encode a symbol table outside of wasm globals, a wasm object file possibly being directly loadable, etc), once we have a prototype and the constraints are clearer. Probably then it will be more clear what to do in cases like this.


https://reviews.llvm.org/D39107





More information about the llvm-commits mailing list