[PATCH] D34734: [WebAssembly] MC: Don't generate extra types for weak alias

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 5 11:36:03 PDT 2017


dschuff accepted this revision.
dschuff added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/MC/WasmObjectWriter.cpp:962
   for (const MCSymbol &S : Asm.symbols()) {
+    if (S.isVariable())
+      continue;
----------------
sbc100 wrote:
> dschuff wrote:
> > so a weak alias both `isVariable()` and `isFunction()`?
> Yup
That's.... fun. Maybe should add a comment here about that.


https://reviews.llvm.org/D34734





More information about the llvm-commits mailing list