Support setting ARM dynamic linker name based on hard-float ABI being specified on compiler commandline

[raj.khem]

Summary

Currently trigger to select hard-float linker is only based of -gnueabihf
appearing in target triplet, but we should also select it when hardfloat
is requested via cmdline

[rengolin]

I think this change makes sense. Please add a test without "gnueabihf" and with -mfloat-abi=hard on test/Driver/linux-ld.c.

Also, please use "diff -U999" when submitting patches to Phabricator, so we can see the context.

cheers,
--renato

[raj.khem]

Create a test-case as suggested

[rengolin]

LGTM, thanks!

[raj.khem]

Can we pull it into branch_37 as well please ?

[rengolin]

You have to send an email to Tom Stellard, copying the list, with the commit in question.

[raj.khem]

Hi Renato,

This patch is still not applied. is there anything needed from my side ?

[rengolin]

Sorry for the delay, committed in r243262. I'll apply it to 3.7.0.

lib/Driver/Tools.cpp

   else if (Arch == llvm::Triple::aarch64_be)
     return "/lib/ld-linux-aarch64_be.so.1";
   else if (Arch == llvm::Triple::arm || Arch == llvm::Triple::thumb) {
-    if (ToolChain.getTriple().getEnvironment() == llvm::Triple::GNUEABIHF)
+    if (ToolChain.getTriple().getEnvironment() == llvm::Triple::GNUEABIHF ||
+        tools::arm::getARMFloatABI(ToolChain.getDriver(), Args, ToolChain.getTriple()) == "hard")
       return "/lib/ld-linux-armhf.so.3";
     else
       return "/lib/ld-linux.so.3";
   } else if (Arch == llvm::Triple::armeb || Arch == llvm::Triple::thumbeb) {
     // TODO: check which dynamic linker name.
-    if (ToolChain.getTriple().getEnvironment() == llvm::Triple::GNUEABIHF)
+    if (ToolChain.getTriple().getEnvironment() == llvm::Triple::GNUEABIHF ||
+        tools::arm::getARMFloatABI(ToolChain.getDriver(), Args, ToolChain.getTriple()) == "hard")
       return "/lib/ld-linux-armhf.so.3";
     else
       return "/lib/ld-linux.so.3";