Author: rovka Date: Mon Aug 8 03:27:36 2016 New Revision: 277985 URL: http://llvm.org/viewvc/llvm-project?rev=277985&view=rev Log: Fix two bugs for musl-libc on ARM
Bug 1: triples like armv7-pc-linux-musl use the wrong linker name ld-musl-armv7.so.1; the right name should be ld-musl-arm.so.1, disregarding the subarch field. Bug 2: when compiler option -mhard-float is used, we should use the "hardfloat" linker, no matter whether the triple itself mentions "hardfloat". Patch by Lei Zhang! Differential Revision: https://reviews.llvm.org/D22904 Modified: cfe/trunk/lib/Driver/ToolChains.cpp cfe/trunk/test/Driver/linux-ld.c Modified: cfe/trunk/lib/Driver/ToolChains.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=277985&r1=277984&r2=277985&view=diff ============================================================================== --- cfe/trunk/lib/Driver/ToolChains.cpp (original) +++ cfe/trunk/lib/Driver/ToolChains.cpp Mon Aug 8 03:27:36 2016 @@ -4274,19 +4274,28 @@ std::string Linux::getDynamicLinker(cons if (Triple.isAndroid()) return Triple.isArch64Bit() ? "/system/bin/linker64" : "/system/bin/linker"; - else if (Triple.isMusl()) { + + if (Triple.isMusl()) { std::string ArchName; + bool IsArm = false; + switch (Arch) { + case llvm::Triple::arm: case llvm::Triple::thumb: ArchName = "arm"; + IsArm = true; break; + case llvm::Triple::armeb: case llvm::Triple::thumbeb: ArchName = "armeb"; + IsArm = true; break; default: ArchName = Triple.getArchName().str(); } - if (Triple.getEnvironment() == llvm::Triple::MuslEABIHF) + if (IsArm && + (Triple.getEnvironment() == llvm::Triple::MuslEABIHF || + tools::arm::getARMFloatABI(*this, Args) == tools::arm::FloatABI::Hard)) ArchName += "hf"; return "/lib/ld-musl-" + ArchName + ".so.1"; Modified: cfe/trunk/test/Driver/linux-ld.c URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/linux-ld.c?rev=277985&r1=277984&r2=277985&view=diff ============================================================================== --- cfe/trunk/test/Driver/linux-ld.c (original) +++ cfe/trunk/test/Driver/linux-ld.c Mon Aug 8 03:27:36 2016 @@ -1613,24 +1613,36 @@ // RUN: --target=thumb-pc-linux-musleabihf \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMHF %s // RUN: %clang %s -### -o %t.o 2>&1 \ +// RUN: --target=thumbv7-pc-linux-musleabi -mhard-float \ +// RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMHF %s +// RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=thumbeb-pc-linux-musleabi \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMEB %s // RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=thumbeb-pc-linux-musleabihf \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMEBHF %s // RUN: %clang %s -### -o %t.o 2>&1 \ +// RUN: --target=thumbv7eb-pc-linux-musleabi -mhard-float \ +// RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMEBHF %s +// RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=arm-pc-linux-musleabi \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARM %s // RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=arm-pc-linux-musleabihf \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMHF %s // RUN: %clang %s -### -o %t.o 2>&1 \ +// RUN: --target=armv7-pc-linux-musleabi -mhard-float \ +// RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMHF %s +// RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=armeb-pc-linux-musleabi \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMEB %s // RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=armeb-pc-linux-musleabihf \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMEBHF %s // RUN: %clang %s -### -o %t.o 2>&1 \ +// RUN: --target=armv7eb-pc-linux-musleabi -mhard-float \ +// RUN: | FileCheck --check-prefix=CHECK-MUSL-ARMEBHF %s +// RUN: %clang %s -### -o %t.o 2>&1 \ // RUN: --target=aarch64-pc-linux-musleabi \ // RUN: | FileCheck --check-prefix=CHECK-MUSL-AARCH64 %s // RUN: %clang %s -### -o %t.o 2>&1 \ _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits