This is an archive of the discontinued LLVM Phabricator instance.

[ELF/AArch64] Set correct loader name in linking context
ClosedPublic

Authored by zatrazz on Jul 3 2015, 8:10 AM.

Details

Summary

This patch reimplements ELFLinkingContext::getDefaultInterpreter for aarch64
with correct loader name. It is required to exclude the loader from DT_NEEDED
in shared library creation.

Diff Detail

Event Timeline

zatrazz updated this revision to Diff 29022.Jul 3 2015, 8:10 AM
zatrazz retitled this revision from to [ELF/AArch64] Set correct loader name in linking context.
zatrazz updated this object.
zatrazz added reviewers: ruiu, shankar.easwaran.
zatrazz added a project: lld.
zatrazz added subscribers: lld, llvm-commits.
ruiu accepted this revision.Jul 3 2015, 9:25 AM
ruiu edited edge metadata.

LGTM

lib/ReaderWriter/ELF/AArch64/AArch64LinkingContext.h
94

Remove "virtual" -- "override" implies that.

This revision is now accepted and ready to land.Jul 3 2015, 9:25 AM
zatrazz closed this revision.Jul 9 2015, 11:53 AM