[PATCH] D108096: [Object] Move llvm-nm's symbol version utility to ELFObjectFile::readDynsymVersions

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 16 09:36:53 PDT 2021


MaskRay added a comment.

`SymbolVersion` has been replaced by `VersionEntry` defined in include/llvm/Object.

  if (!VerSec)
    return std::vector<VersionEntry>();

has been added so that llvm-objdump can know whether the version section exists or not. llvm-nm code already does `if (!SymbolVersions.empty() ...` so llvm-nm doesn't need a change.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108096/new/

https://reviews.llvm.org/D108096



More information about the llvm-commits mailing list