[PATCH] D57323: [WebAssembly] Enable main-function signature rewriting for WASI

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 28 16:40:18 PST 2019


sunfish updated this revision to Diff 183988.
sunfish edited the summary of this revision.
sunfish added a comment.

- Remove the target constraint, allowing the main rewrite logic to work on all wasm32 targets.
- Remove the now-unneeded "wasm-temporary-workarounds" flag altogether.
- This uncovered some bugs with wrappers for main when main is only a declaration; fix those as well.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57323

Files:
  lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp
  test/CodeGen/WebAssembly/call.ll
  test/CodeGen/WebAssembly/function-bitcasts-varargs.ll
  test/CodeGen/WebAssembly/function-bitcasts.ll
  test/CodeGen/WebAssembly/main-declaration.ll
  test/CodeGen/WebAssembly/main-no-args.ll
  test/CodeGen/WebAssembly/main-three-args.ll
  test/CodeGen/WebAssembly/main-with-args.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57323.183988.patch
Type: text/x-patch
Size: 8602 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190129/89622842/attachment.bin>


More information about the llvm-commits mailing list