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.  */
+    return "armelfb_linux_eabi"; /* TODO: check which NAME.  */
   case llvm::Triple::ppc:
----------------
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.


http://reviews.llvm.org/D21022



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to