[PATCH] D42520: Implement __attribute__((import_module("foo")))

Sergey Pepyakin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 00:35:25 PST 2018


pepyakin added inline comments.


================
Comment at: include/llvm/MC/MCSymbolWasm.h:34
 public:
-  // Use a module name of "env" for now, for compatibility with existing tools.
-  // This is temporary, and may change, as the ABI is not yet stable.
+  // A module name of "__unresolved" means that the symbol is to be
+  // resolved by the the linker, either to a definition in the module
----------------
I guess there should be "__linker_resolve"


Repository:
  rL LLVM

https://reviews.llvm.org/D42520





More information about the llvm-commits mailing list