[PATCH] D44283: Don't treat .symver as a regular alias definition
Vitaly Buka via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 8 17:52:55 PST 2018
vitalybuka added inline comments.
================
Comment at: lib/MC/MCParser/ELFAsmParser.cpp:774
MCSymbol *Sym = getContext().getOrCreateSymbol(Name);
- const MCExpr *Value = MCSymbolRefExpr::create(Sym, getContext());
-
- getStreamer().EmitAssignment(Alias, Value);
getStreamer().emitELFSymverDirective(Alias, Sym);
return false;
----------------
vitalybuka wrote:
> pcc wrote:
> > vitalybuka wrote:
> > > Can we still pass AliasName instead of created Alias ?
> > >
> > >
> > Most clients will want to consume the MCSymbol, so it seems a little simpler to create it here.
> Then I am sure when to replace @@@ with @ or @@
I can e.g. discard already created MCSymbol and replace with new one
https://reviews.llvm.org/D44283
More information about the llvm-commits
mailing list