[PATCH] D33680: [ELF] - Resolve references properly when using .symver directive

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 14:47:35 PDT 2017


George Rimar via Phabricator <reviews at reviews.llvm.org> writes:

> +  for (Symbol *Sym : SymVector) {
> +    SymbolBody *Body = Sym->body();
> +    bool IsDefault = isDefaultVersion(Body);
> +    Body->parseSymbolVersion();
> +
> +    if (!IsDefault)
> +      continue;

I think you can omit the IsDefault variable:

if (!isDefaultVersion(Body))
  continue;

Cheers,
Rafael


More information about the llvm-commits mailing list