[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