[PATCH] D91637: [WebAssembly] Only emit indirect function table import if needed

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 17 11:01:30 PST 2020


sbc100 added a comment.

What I was hoping for here is minimal change to the object writer to remove the table from object files that don't reference it.  This seem separable from the question of whether the indirection function table should be first class MC symbol .. although maybe I'm taking this CL splitting thing to far.



================
Comment at: llvm/test/MC/WebAssembly/weak-alias.s:242
 # CHECK-NEXT:         Alignment:       3
-# CHECK-NEXT:         Flags:           [ ]
 # CHECK-NEXT: ...
----------------
Why did these changes occur?  Do you have some tool to automatically updating the expected output?  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91637



More information about the llvm-commits mailing list