[PATCH] D62443: [WebAssembly] Move Emscripten-specific behavior under a --emscripten flag

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 3 17:03:29 PDT 2019


sunfish updated this revision to Diff 202826.
sunfish added a comment.

- Rebase
- Rename IMPLICITLY_USED to NO_STRIP

Talking with @sbc100 offline, an alternative to the --emscripten command-line flag would be to write the target triple into .o files, so that the linker can determine if it should enable Emscripten-specific behavior based on that, rather than needing a command-line flag.

The patch here just rebases the earlier patch and updates the name of the new NO_STRIP flag, so that it's easier to see what the changes are.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D62443

Files:
  docs/ReleaseNotes.rst
  docs/WebAssembly.rst
  test/wasm/export.ll
  test/wasm/pie.ll
  test/wasm/shared-needed.ll
  test/wasm/shared.ll
  test/wasm/undefined-data.ll
  wasm/Config.h
  wasm/Driver.cpp
  wasm/LTO.cpp
  wasm/MarkLive.cpp
  wasm/Options.td
  wasm/Symbols.cpp
  wasm/Symbols.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62443.202826.patch
Type: text/x-patch
Size: 10913 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190604/71791e60/attachment.bin>


More information about the llvm-commits mailing list