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

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.