[PATCH] D99887: [lld][WebAssembly] Add `--export-if-defined`

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 5 11:24:34 PDT 2021


sbc100 added a comment.

I dont love the name of this option.. but I failed to come up with any thing nicer.

I did consider weaking the existing `--export` to mean "export if defined" and then requiring `--require-defined` (which is an existing GNU ld option) to get the stronger meaning back.. but that has two disadvantages:

1. Potentially breaking existing users exceptions
2. Makes the common use case of `--export foo --require-defined foo` for each export a lot more verbose.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99887



More information about the llvm-commits mailing list