This is an archive of the discontinued LLVM Phabricator instance.

[lld] [ELF] [MIPS] Remove duplicate logic
ClosedPublic

Authored by rafaelauler on Oct 1 2014, 2:22 PM.

Details

Summary

With r218633, the logic that monitors which shared library symbols were used was copied from the MIPS lld backend to ELF classes, making it available to all ELF backends. However, this made the isDynSymEntryRequired() functions in MipsDynamicLibraryWriter.h/MipsELFWriters.h/MipsExecutableWriter.h to be duplicated logic, since this is already implemented in OutputELFWriter<>/DefaultLayout. This patch removes this duplicated code from MIPS.

Diff Detail

Event Timeline

rafaelauler updated this revision to Diff 14294.Oct 1 2014, 2:22 PM
rafaelauler retitled this revision from to [lld] [ELF] [MIPS] Remove duplicate logic.
rafaelauler updated this object.
rafaelauler edited the test plan for this revision. (Show Details)
rafaelauler added reviewers: Bigcheese, shankarke.
rafaelauler added a subscriber: Unknown Object (MLST).
shankarke accepted this revision.Oct 1 2014, 2:36 PM
shankarke edited edge metadata.
This revision is now accepted and ready to land.Oct 1 2014, 2:36 PM
rafaelauler closed this revision.Oct 1 2014, 6:40 PM

Committed r218846.