[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