[PATCH] D79247: [lld][WebAssembly] Honor --allow-undefined for data symbols too

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

In D79247#2021187 <https://reviews.llvm.org/D79247#2021187>, @dschuff wrote:

> So the plan is to support `unresolved-symbols` and then eventually deprecate and remove this?


Eventually yes that would great.  However I think it will take several llvm version before we are ready to do that.  Today the `--allow-undefined` flag has many users including emscripten, and they rely on the current behviour of this flag which is to turn undefined function symbols into function imports.   We maybe or may not want to keep the behaviour somehow (either via this flag, or one with a better name such as `--import-undefined-functions`), even though we now have clang/llvm/symbol attributes to mark functions for import.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79247