Page MenuHomePhabricator

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

Authored by grimar on Thu, Sep 3, 7:52 AM.

Details

Summary

We have the RelSymbol<ELFT> struct and can use it instead
of std::pair<const Elf_Sym *, std::string> in a few methods.

This is a bit cleaner, but also should allow to get rid of
printDynamicRelocation methods for both styles after landing D87089.

Diff Detail

Event Timeline

grimar created this revision.Thu, Sep 3, 7:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Sep 3, 7:52 AM
Herald added a subscriber: rupprecht. · View Herald Transcript
grimar requested review of this revision.Thu, Sep 3, 7:52 AM
jhenderson accepted this revision.Fri, Sep 4, 12:52 AM

LGTM, with suggestion.

llvm/tools/llvm-readobj/ELFDumper.cpp
118–119
This revision is now accepted and ready to land.Fri, Sep 4, 12:52 AM