Re: [PATCH] D21022: [ARM] Fix linker emulation for arm 32 big endian

2016-09-22 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko closed this revision. Eugene.Zelenko added a comment. Committed in r272402. https://reviews.llvm.org/D21022 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D21022: [ARM] Fix linker emulation for arm 32 big endian

2016-06-10 Thread Peter Smith via cfe-commits
peter.smith accepted this revision. peter.smith added a comment. This revision is now accepted and ready to land. I think the change looks fine and it looks small enough for me to approve. http://reviews.llvm.org/D21022 ___ cfe-commits mailing list

Re: [PATCH] D21022: [ARM] Fix linker emulation for arm 32 big endian

2016-06-07 Thread Strahinja Petrovic via cfe-commits
spetrovic updated this revision to Diff 59870. spetrovic marked an inline comment as done. spetrovic added a comment. Comment addressed. http://reviews.llvm.org/D21022 Files: lib/Driver/Tools.cpp test/Driver/linux-ld.c Index: test/Driver/linux-ld.c

Re: [PATCH] D21022: [ARM] Fix linker emulation for arm 32 big endian

2016-06-06 Thread Peter Smith via cfe-commits
peter.smith added a comment. That looks correct to me. Certainly armebelf_linux_eabi is not recognised on a recent ld. Comment at: lib/Driver/Tools.cpp:9098 @@ -9097,3 +9097,3 @@ case llvm::Triple::thumbeb: -return "armebelf_linux_eabi"; /* TODO: check which NAME. */ +

[PATCH] D21022: [ARM] Fix linker emulation for arm 32 big endian

2016-06-06 Thread Strahinja Petrovic via cfe-commits
spetrovic created this revision. spetrovic added reviewers: rafael, petarj. spetrovic added subscribers: rankov, ivanbaev, cfe-commits. Herald added subscribers: rengolin, aemerson. This patch fixes target linker emulation for arm 32 big endian. As I saw in gcc only supported linker emulation is