[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