[PATCH] D91870: [WebAssembly] Add support for table linking to wasm-ld

[Derek Schuff via Phabricator via llvm-commits]

dschuff added a comment.

This seems to have a bug that we discovered on the emscripten builders where it's exporting the table twice.
The output (from wasm-objdump):

  Export[14]:
   - memory[0] -> "memory"
   - table[0] -> "__indirect_function_table"
   - func[3] <__wasm_call_ctors> -> "__wasm_call_ctors"
   - func[5] <main> -> "main"
   - table[0] -> "__indirect_function_table"
   - func[6] <stackSave> -> "stackSave"
   - func[7] <stackRestore> -> "stackRestore"
   - func[8] <stackAlloc> -> "stackAlloc"
   - func[9] <emscripten_stack_init> -> "emscripten_stack_init"
   - func[10] <emscripten_stack_get_free> -> "emscripten_stack_get_free"
   - func[11] <emscripten_stack_get_end> -> "emscripten_stack_get_end"
   - func[12] <saveSetjmp> -> "saveSetjmp"
   - func[13] <testSetjmp> -> "testSetjmp"
   - func[14] <setThrew> -> "setThrew"

I'll see if I can find a more useful reproducer


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91870