commit 593f45ee021f5310d6979f435fa4b6fcfaa5dabf (tree)
parent 9e1e54d32449dadeb21bc2ce0fe9d246c466ebf0
Author: Alex Rønne Petersen <alex@alexrp.com>
Date: Mon, 19 Jan 2026 20:43:42 +0100
llvm: switch most targets to using half and fp128 IR types
As of LLVM 22, most backends can (finally) handle legalization of these types as
necessary, so we don't need to do it ourselves anymore.
closes https://github.com/ziglang/zig/issues/23674
Diffstat:
2 files changed, 1 insertion(+), 41 deletions(-)
diff --git a/lib/compiler_rt.zig b/lib/compiler_rt.zig
@@ -447,29 +447,8 @@ pub const gnu_f16_abi = switch (builtin.cpu.arch) {
pub const want_sparc_abi = builtin.cpu.arch.isSPARC();
-/// This seems to mostly correspond to `clang::TargetInfo::HasFloat16`.
pub fn F16T(comptime OtherType: type) type {
return switch (builtin.cpu.arch) {
- .amdgcn,
- .arm,
- .armeb,
- .thumb,
- .thumbeb,
- .aarch64,
- .aarch64_be,
- .hexagon,
- .loongarch32,
- .loongarch64,
- .nvptx,
- .nvptx64,
- .riscv32,
- .riscv32be,
- .riscv64,
- .riscv64be,
- .s390x,
- .spirv32,
- .spirv64,
- => f16,
.x86, .x86_64 => if (builtin.target.os.tag.isDarwin()) switch (OtherType) {
// Starting with LLVM 16, Darwin uses different abi for f16
// depending on the type of the other return/argument..???
@@ -477,7 +456,7 @@ pub fn F16T(comptime OtherType: type) type {
f80, f128 => f16,
else => unreachable,
} else f16,
- else => u16,
+ else => f16,
};
}
diff --git a/src/codegen/llvm.zig b/src/codegen/llvm.zig
@@ -4646,20 +4646,6 @@ fn toLlvmGlobalAddressSpace(wanted_address_space: std.builtin.AddressSpace, targ
/// or if it produces miscompilations.
pub fn backendSupportsF16(target: *const std.Target) bool {
return switch (target.cpu.arch) {
- // https://github.com/llvm/llvm-project/issues/97981
- .csky,
- // https://github.com/llvm/llvm-project/issues/97981
- .powerpc,
- .powerpcle,
- .powerpc64,
- .powerpc64le,
- // https://github.com/llvm/llvm-project/issues/97981
- .wasm32,
- .wasm64,
- // https://github.com/llvm/llvm-project/issues/97981
- .sparc,
- .sparc64,
- => false,
.arm,
.armeb,
.thumb,
@@ -4686,11 +4672,6 @@ pub fn backendSupportsF128(target: *const std.Target) bool {
return switch (target.cpu.arch) {
// https://github.com/llvm/llvm-project/issues/121122
.amdgcn,
- // Test failures all over the place.
- .mips64,
- .mips64el,
- // https://github.com/llvm/llvm-project/issues/41838
- .sparc,
=> false,
.arm,
.armeb,