[PATCH] D30485: Mark .symver versioned symbol as global
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 28 19:40:12 PST 2017
pcc added inline comments.
================
Comment at: lib/MC/MCParser/ELFAsmParser.cpp:748
+ // for the linker.
+ getStreamer().EmitSymbolAttribute(Alias, MCSA_Global);
return false;
----------------
I don't think this is right. See `ELFObjectWriter::executePostLayoutBinding`: `.symver` symbols inherit their binding from the symbol they alias.
I think we may need something equivalent to the `executePostLayoutBinding` logic in `ModuleSymbolTable::CollectAsmSymbols`.
https://reviews.llvm.org/D30485
More information about the llvm-commits
mailing list