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

Andy Wingo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 18 07:21:12 PST 2021


wingo marked an inline comment as done.
wingo added inline comments.


================
Comment at: lld/test/wasm/export-table-test-2.test:1
+# RUN: llvm-mc -filetype=obj -triple=wasm32-unknown-unknown %p/Inputs/start.s -o %t.start.o
+# RUN: wasm-ld --export-table -o %t.wasm %t.start.o
----------------
sbc100 wrote:
> Can we come up with a better name for this new test?   I don't have any great idea but just calling it "-test-2" doesn't seem useful.  
> 
> How about `export-table-explicit.test`?    Since this test includes and explicit declaration of the __indirect_function_table?
Good idea; done :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91870



More information about the llvm-commits mailing list