[PATCH] D115511: [WebAssembly] Lower global syms representing tables with .tabletype

[Paulo Matos via Phabricator via llvm-commits]

pmatos added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h:118
+  // The symbol declares a funcref table
+  MO_SYM_IS_FUNCREF_TABLE,
 };
----------------
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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115511