[clang][Driver] Define soft float macros for PPC.

[alexrp]

Fixes #105972.

[llvmbot]

@llvm/pr-subscribers-backend-powerpc

@llvm/pr-subscribers-clang

Author: Alex Rønne Petersen (alexrp)

Changes

Fixes #105972.

---

Full diff: https://github.com/llvm/llvm-project/pull/106012.diff

1 Files Affected:

• (modified) clang/lib/Basic/Targets/PPC.cpp (+5-3)

diff --git a/clang/lib/Basic/Targets/PPC.cpp b/clang/lib/Basic/Targets/PPC.cpp
index 04dc436eb1b9cd..91145b256804c7 100644
--- a/clang/lib/Basic/Targets/PPC.cpp
+++ b/clang/lib/Basic/Targets/PPC.cpp
@@ -402,9 +402,13 @@ void PPCTargetInfo::getTargetDefines(const LangOptions &Opts,
     Builder.defineMacro("__VEC__", "10206");
     Builder.defineMacro("__ALTIVEC__");
   }
-  if (HasSPE) {
+  if (HasSPE)
     Builder.defineMacro("__SPE__");
+  if (HasSPE || FloatABI == SoftFloat)
     Builder.defineMacro("__NO_FPRS__");
+  if (FloatABI == SoftFloat) {
+    Builder.defineMacro("_SOFT_FLOAT");
+    Builder.defineMacro("_SOFT_DOUBLE");
   }
   if (HasVSX)
     Builder.defineMacro("__VSX__");
@@ -439,14 +443,12 @@ void PPCTargetInfo::getTargetDefines(const LangOptions &Opts,
   // FIXME: The following are not yet generated here by Clang, but are
   //        generated by GCC:
   //
-  //   _SOFT_FLOAT_
   //   __RECIP_PRECISION__
   //   __APPLE_ALTIVEC__
   //   __RECIP__
   //   __RECIPF__
   //   __RSQRTE__
   //   __RSQRTEF__
-  //   _SOFT_DOUBLE_
   //   __NO_LWSYNC__
   //   __CMODEL_MEDIUM__
   //   __CMODEL_LARGE__

[alexrp]

FWIW, if this patch is good, it would be nice if we could get it into 19.x since we're affected by it in Zig.

[ecnelises]

Could you pick the test change from ecnelises@9dcb9c9 into this patch?

[alexrp]

@ecnelises done, thank you!

[ecnelises]

Thanks

[chenzheng1030]

LGTM now. Thanks very much for the fix.

[alexrp]

Forgot to note in the PR description that I don't have commit access.