This is an archive of the discontinued LLVM Phabricator instance.

[ELF] Introduce getValue() for ELF Symbols
ClosedPublic

Authored by davide on Jun 8 2015, 4:35 PM.

Details

Summary

COFF has a similar function, because linkers, or tool in general, need it. At least, my ELF backend rewrite does.

Diff Detail

Repository
rL LLVM

Event Timeline

davide updated this revision to Diff 27347.Jun 8 2015, 4:35 PM
davide retitled this revision from to [ELF] Introduce getValue() for ELF Symbols.
davide updated this object.
davide edited the test plan for this revision. (Show Details)
davide added a reviewer: rafael.
davide set the repository for this revision to rL LLVM.
davide added a project: lld.
davide added a subscriber: Unknown Object (MLST).
rafael accepted this revision.Jun 11 2015, 12:20 PM
rafael edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jun 11 2015, 12:20 PM

Side question, why do we have Elf_Sym_Base and Elf_Sym_Impl?

This revision was automatically updated to reflect the committed changes.