[clang] [Clang][Driver][Hurd] Enable __float128 support on x86 on Hurd (PR #153925)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 15 22:01:13 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Pino Toscano (pinotree)
<details>
<summary>Changes</summary>
It works fine, and this makes the Hurd driver more consistent with other drivers.
---
Full diff: https://github.com/llvm/llvm-project/pull/153925.diff
1 Files Affected:
- (modified) clang/lib/Basic/Targets/OSTargets.h (+13-1)
``````````diff
diff --git a/clang/lib/Basic/Targets/OSTargets.h b/clang/lib/Basic/Targets/OSTargets.h
index 94b018a0751d1..8d3e232dde1a5 100644
--- a/clang/lib/Basic/Targets/OSTargets.h
+++ b/clang/lib/Basic/Targets/OSTargets.h
@@ -325,9 +325,21 @@ class LLVM_LIBRARY_VISIBILITY HurdTargetInfo : public OSTargetInfo<Target> {
Builder.defineMacro("_REENTRANT");
if (Opts.CPlusPlus)
Builder.defineMacro("_GNU_SOURCE");
+ if (this->HasFloat128)
+ Builder.defineMacro("__FLOAT128__");
}
public:
- using OSTargetInfo<Target>::OSTargetInfo;
+ HurdTargetInfo(const llvm::Triple &Triple, const TargetOptions &Opts)
+ : OSTargetInfo<Target>(Triple, Opts) {
+ switch (Triple.getArch()) {
+ default:
+ break;
+ case llvm::Triple::x86:
+ case llvm::Triple::x86_64:
+ this->HasFloat128 = true;
+ break;
+ }
+ }
};
// Linux target
``````````
</details>
https://github.com/llvm/llvm-project/pull/153925
More information about the cfe-commits
mailing list