[PATCH] D101271: [lld][WebAssembly] Add support for `--export-dynamic-symbol`

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

The existing `--export` flag, one could argue, already promotes symbols from the static world to the dyanmic (runtime) world, its just that it also implies `--require-defined=foo`.   In fact one could potentially argue that `--export` should be deprecated in favor of `--require-defined=` + `--export-dynamic-symbol=` (although I'm not doing that).

My original version of this patch used the name `--export-if-defined=`.. if you feel strongly we could go back to that name instead, but I like sharing flags with the meaning of this flag seems to match pretty well here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101271