[PATCH] D131376: [wasm-ld] Allow importing/exporting the output module's memory with arbitrary names
    Dan Gohman via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Aug 20 08:26:03 PDT 2022
    
    
  
sunfish added a comment.
In D131376#3737184 <https://reviews.llvm.org/D131376#3737184>, @sbc100 wrote:
> I think its probably simpler to only allow a single start `cabi_start` definition.. just like any other symbol.  It sounds like it should be a linker error to have more than one of them.
This sounds right to me. We already have the .init_array mechanism for registering constructors at link time, and it sounds like we can continue to use that with `cabi_start`.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131376/new/
https://reviews.llvm.org/D131376
    
    
More information about the llvm-commits
mailing list