[PATCH] D94075: [lld][WebAssembly] Add support for handling table symbols

Andy Wingo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 01:43:16 PST 2021


wingo added a comment.

In D94075#2492422 <https://reviews.llvm.org/D94075#2492422>, @sbc100 wrote:

> It would be good to add some test cases with multi-table and input and output.  Is there a fundamental reason we can't do that yet?

Unfortunately things are still broken with table imports: imported table numbers start at 0, whereas at this point in the patch set there are still a few things that expect the __indirect_function_table to be 0, even if it's not imported.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94075



More information about the llvm-commits mailing list