Skip to content

Commit c6f433e

Browse files
authored
[bazel][libc] Remove target compatibility restrictions for float128 (#169292)
The restrictions here aren't nearly as much about the OS as the compiler and architecture, but the Bazel restriction was OS-based. Everything seems to work well on even Arm64 macOS, and I would expect most BSDs and other OSes to work well with Clang's support on x86-64. The source code here already handles detecting when there is compiler support for the type. And the users of this don't `select` or do anything else to conditionally include the header, so it seems better to not restrict access to the header from the build system, and instead continue making the source code compatible or a no-op on relevant configurations.
1 parent f0bb5cf commit c6f433e

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

utils/bazel/llvm-project-overlay/libc/BUILD.bazel

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -171,11 +171,6 @@ libc_support_library(
171171
libc_support_library(
172172
name = "llvm_libc_types_float128",
173173
hdrs = ["include/llvm-libc-types/float128.h"],
174-
target_compatible_with = select({
175-
"@platforms//os:linux": [],
176-
"@platforms//os:windows": [],
177-
"//conditions:default": ["@platforms//:incompatible"],
178-
}),
179174
deps = [":llvm_libc_macros_float_macros"],
180175
)
181176

0 commit comments

Comments
 (0)