[PATCH] D30351: De-template SharedSymbol.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 26 11:49:01 PST 2017


>    if (WasInserted || isa<Undefined>(S->body())) {
> -    replaceBody<SharedSymbol<ELFT>>(S, F, Name, Sym, Verdef);
> +    replaceBody<SharedSymbol>(S, File, Name, Sym.st_other, Sym.getType(),
> +                              Sym.st_shndx, Sym.st_value, Sym.st_size,
> +                              getAlignment(File, Sym), Verdef);

With this we know always compute the alignment, instead of doing so only
when the symbol needs a copy relocation. Why?

Cheers,
Rafael


More information about the llvm-commits mailing list