This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Fix linker emulation for arm 32 big endian
ClosedPublic

Authored by spetrovic on Jun 6 2016, 7:37 AM.
Tokens
"Like" token, awarded by bseshadri.

Details

Summary

This patch fixes target linker emulation for arm 32 big endian. As I saw in gcc only supported linker emulation is armelfb_linux_eabi.

Diff Detail

Event Timeline

spetrovic updated this revision to Diff 59718.Jun 6 2016, 7:37 AM
spetrovic retitled this revision from to [ARM] Fix linker emulation for arm 32 big endian.
spetrovic updated this object.
spetrovic added reviewers: rafael, petarj.
spetrovic added subscribers: rankov, ivanbaev, cfe-commits.
peter.smith edited edge metadata.Jun 6 2016, 8:56 AM

That looks correct to me. Certainly armebelf_linux_eabi is not recognised on a recent ld.

lib/Driver/Tools.cpp
9098

armelfb_linux_eabi matches the emulation name of my installation of gnu ld and the name in ld/configure.tgt. I think it is at least as authoratitive as armelf_linux_eabi so I suggest removing the /* TODO */ comment.

spetrovic updated this revision to Diff 59870.Jun 7 2016, 5:27 AM
spetrovic edited edge metadata.
spetrovic marked an inline comment as done.

Comment addressed.

peter.smith accepted this revision.Jun 10 2016, 4:14 AM
peter.smith edited edge metadata.

I think the change looks fine and it looks small enough for me to approve.

This revision is now accepted and ready to land.Jun 10 2016, 4:14 AM
Eugene.Zelenko added a subscriber: Eugene.Zelenko.

Committed in r272402.