Index: cfe/trunk/lib/Driver/ToolChains/Arch/ARM.cpp =================================================================== --- cfe/trunk/lib/Driver/ToolChains/Arch/ARM.cpp +++ cfe/trunk/lib/Driver/ToolChains/Arch/ARM.cpp @@ -430,8 +430,8 @@ llvm::ARM::getFPUFeatures(llvm::ARM::FK_NONE, Features); // Disable hardware FP features which have been enabled. - // FIXME: Disabling vfp2 and neon should be enough as all the other - // features are dependent on these 2 features in LLVM. However + // FIXME: Disabling fpregs should be enough all by itself, since all + // the other FP features are dependent on it. However // there is currently no easy way to test this in clang, so for // now just be explicit and disable all known dependent features // as well. @@ -439,6 +439,11 @@ "neon", "crypto", "dotprod", "fp16fml"}) if (std::find(std::begin(Features), std::end(Features), "+" + Feature) != std::end(Features)) Features.push_back(Args.MakeArgString("-" + Feature)); + + // Disable the base feature unconditionally, even if it was not + // explicitly in the features list (e.g. if we had +vfp3, which + // implies it). + Features.push_back("-fpregs"); } // En/disable crc code generation. Index: cfe/trunk/test/CodeGen/arm-mfpu-none.c =================================================================== --- cfe/trunk/test/CodeGen/arm-mfpu-none.c +++ cfe/trunk/test/CodeGen/arm-mfpu-none.c @@ -0,0 +1,8 @@ +// REQUIRES: arm-registered-target +// RUN: %clang -target arm-none-eabi -mcpu=cortex-m4 -mfpu=none -S -o - %s | FileCheck %s + +// CHECK-LABEL: compute +// CHECK-NOT: {{s[0-9]}} +float compute(float a, float b) { + return (a+b) * (a-b); +} Index: cfe/trunk/test/Driver/arm-mfpu.c =================================================================== --- cfe/trunk/test/Driver/arm-mfpu.c +++ cfe/trunk/test/Driver/arm-mfpu.c @@ -318,6 +318,7 @@ // RUN: | FileCheck --check-prefix=CHECK-NO-FP %s // CHECK-NO-FP-NOT: "-target-feature" "+soft-float" // CHECK-NO-FP: "-target-feature" "+soft-float-abi" +// CHECK-NO-FP: "-target-feature" "-fpregs" // CHECK-NO-FP: "-target-feature" "-vfp2" // CHECK-NO-FP: "-target-feature" "-vfp3" // CHECK-NO-FP: "-target-feature" "-vfp4" @@ -363,6 +364,7 @@ // CHECK-SOFT-ABI-FP: "-target-feature" "-fp-armv8" // CHECK-SOFT-ABI-FP: "-target-feature" "-neon" // CHECK-SOFT-ABI-FP: "-target-feature" "-crypto" +// CHECK-SOFT-ABI-FP: "-target-feature" "-fpregs" // RUN: %clang -target arm-linux-androideabi21 %s -### -c 2>&1 \ // RUN: | FileCheck --check-prefix=CHECK-ARM5-ANDROID-FP-DEFAULT %s Index: llvm/trunk/lib/Support/ARMTargetParser.cpp =================================================================== --- llvm/trunk/lib/Support/ARMTargetParser.cpp +++ llvm/trunk/lib/Support/ARMTargetParser.cpp @@ -198,6 +198,7 @@ Features.push_back("-fp-armv8"); break; case FPUVersion::NONE: + Features.push_back("-fpregs"); Features.push_back("-vfp2"); Features.push_back("-vfp3"); Features.push_back("-fp16");