[PATCH] D87092: [llvm-readelf/obj] - Use `RelSymbol<ELFT>` instead of std::pair. NFCI.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 00:52:35 PDT 2020


jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

LGTM, with suggestion.



================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:118-119
+template <class ELFT> struct RelSymbol {
+  RelSymbol(const typename ELFT::Sym *S, const std::string &N)
+      : Sym(S), Name(N) {}
+  const typename ELFT::Sym *Sym;
----------------



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

https://reviews.llvm.org/D87092



More information about the llvm-commits mailing list