[PATCH] D70877: [WebAssebmly][MC] Support .import_name/.import_field asm directives

Sam Clegg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 5 14:35:52 PST 2019


sbc100 marked an inline comment as done.
sbc100 added inline comments.


================
Comment at: llvm/test/MC/WebAssembly/import-module.s:13
+  .import_module  foo, bar
+  .import_name  foo, qux
+
----------------
sbc100 wrote:
> dschuff wrote:
> > What should happen if there's a directive that refers to a symbol that doesn't exist in the asm file?
> I seems that it does the same thing we do for `.functype` above, basically nothing.
To be a little more specific it seems that for these kind of attributes we call `Ctx.getOrCreateSymbol` then set a propoerty of the symbol, but if that symbol isn't used elsewhere it isn't included in the output.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70877





More information about the cfe-commits mailing list