[PATCH] D115511: [WebAssembly] Lower global syms representing tables with .tabletype
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Dec 11 09:22:00 PST 2021
sbc100 added inline comments.
================
Comment at: llvm/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h:118
+ // The symbol declares a funcref table
+ MO_SYM_IS_FUNCREF_TABLE,
};
----------------
pmatos wrote:
> sbc100 wrote:
> > I'm not sure this enumeration is best place to declare that type of a symbols. These other values all refer to how the symbol is used, but what type the symbol is. For example that distinction between function and data symbols does not appear here.
> >
> This is the place to set TargetFlags, and we cannot set the type of the symbol directly here because we don't have access to the MCSymbol, so as far as I can understand we need to set a TargetFlag and later on set the right MCSymbol type based on the address TargetFlags. Do you have a suggestion on where to set this information if not as a TargetFlag?
I'm afraid its been a while since I touched this code.. so I'm not sure I have any useful suggestions other than looking at existing code and practices.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115511/new/
https://reviews.llvm.org/D115511
More information about the llvm-commits
mailing list