[PATCH] D131376: [wasm-ld] Allow importing/exporting the output module's memory with arbitrary names

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

In D131376#3711300 <https://reviews.llvm.org/D131376#3711300>, @Liamolucko wrote:

>> 1. Are there going to be other cabi-relatated things, that warrent just putting this behind a single "cabi" flag rather than requiring N different flags for this?
>
> Yeah, the linker will also have to merge together `cabi_start` definitions, so using a unified flag makes more sense. I'll make a new patch to do it that way. Does Phabricator have a way to close this one?

That does `cabi_start` represent?  If its the equivalent of `main` we probably don't want the linker to merge them but instead reject programs that contain more than one (just like any other symbol).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131376