Skip to content

Commit 698d85e

Browse files
authored
Use all @llvm constraints (#111)
1 parent afe82a1 commit 698d85e

8 files changed

Lines changed: 16 additions & 42 deletions

File tree

MODULE.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ bazel_dep(name = "platforms", version = "0.0.5")
5757
bazel_dep(name = "protobuf", version = "34.0.bcr.1", repo_name = "com_google_protobuf")
5858
bazel_dep(name = "rules_cc", version = "0.2.8")
5959
bazel_dep(name = "rules_proto", version = "7.1.0")
60-
bazel_dep(name = "llvm", version = "0.7.6")
60+
bazel_dep(name = "llvm", version = "0.7.7")
6161
bazel_dep(name = "aspect_tools_telemetry", version = "0.3.3")
6262

6363
osx = use_extension("@llvm//extensions:osx.bzl", "osx")

MODULE.bazel.lock

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ This is the default setup for new users. It provisions the patched `rules_rust`,
2626

2727
```bzl
2828
bazel_dep(name = "rules_rs", version = "0.0.61")
29-
bazel_dep(name = "llvm", version = "0.7.3")
29+
bazel_dep(name = "llvm", version = "0.7.7")
3030
bazel_dep(name = "platforms", version = "1.0.0")
3131

3232
toolchains = use_extension("@rules_rs//rs/toolchains:module_extension.bzl", "toolchains")
@@ -89,7 +89,7 @@ platform(
8989
name = "local_windows_msvc",
9090
parents = ["@platforms//host"],
9191
constraint_values = [
92-
"@rules_rs//rs/platforms/constraints:windows_msvc",
92+
"@llvm//constraints/windows/abi:msvc",
9393
],
9494
)
9595
```

rs/platforms/constraints/BUILD.bazel

Lines changed: 0 additions & 20 deletions
This file was deleted.

rs/platforms/triples.bzl

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,6 @@ load(
44
_triple_to_constraint_set = "triple_to_constraint_set",
55
)
66

7-
_WINDOWS_ABI_CONSTRAINTS = {
8-
"gnu": "@rules_rs//rs/platforms/constraints:windows_gnu",
9-
"gnullvm": "@rules_rs//rs/platforms/constraints:windows_gnullvm",
10-
"msvc": "@rules_rs//rs/platforms/constraints:windows_msvc",
11-
}
12-
137
def triple_to_constraint_set(target_triple):
148
constraints = _triple_to_constraint_set(target_triple)
159
t = triple(target_triple)
@@ -22,13 +16,13 @@ def triple_to_constraint_set(target_triple):
2216
else:
2317
constraints.append("@llvm//constraints/libc:gnu.2.28")
2418
elif t.system == "windows":
25-
constraints.append(_WINDOWS_ABI_CONSTRAINTS[t.abi])
19+
constraints.append("@llvm//constraints/windows/abi:" + t.abi)
2620

2721
# Rust links MSVCRT for both GNU Windows target specs:
2822
# https://github.com/rust-lang/rust/blob/c935696dd07ca51e6fba2f6579919eea2a50863b/compiler/rustc_target/src/spec/base/windows_gnullvm.rs#L19
2923
# https://github.com/rust-lang/rust/blob/c935696dd07ca51e6fba2f6579919eea2a50863b/compiler/rustc_target/src/spec/base/windows_gnu.rs#L44
3024
if t.abi in ("gnu", "gnullvm"):
31-
constraints.append("@llvm//toolchain:windows_crt_msvcrt")
25+
constraints.append("@llvm//constraints/windows/crt:msvcrt")
3226

3327
return constraints
3428

rs/toolchains/declare_rustc_toolchains.bzl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,9 @@ def declare_rustc_toolchains(
7373
"//conditions:default": "",
7474
}),
7575
staticlib_ext = select({
76-
"@rules_rs//rs/platforms/constraints:windows_gnu": ".a",
77-
"@rules_rs//rs/platforms/constraints:windows_gnullvm": ".a",
78-
"@rules_rs//rs/platforms/constraints:windows_msvc": ".lib",
76+
"@llvm//constraints/windows/abi:gnu": ".a",
77+
"@llvm//constraints/windows/abi:gnullvm": ".a",
78+
"@llvm//constraints/windows/abi:msvc": ".lib",
7979
"@platforms//os:none": "",
8080
"@platforms//os:emscripten": ".js",
8181
"@platforms//os:uefi": ".lib",
@@ -102,9 +102,9 @@ def declare_rustc_toolchains(
102102
"@platforms//os:nixos": ["-ldl", "-lpthread"],
103103
"@platforms//os:openbsd": ["-lpthread"],
104104
"@platforms//os:ios": ["-lSystem", "-lobjc", "-Wl,-framework,Security", "-Wl,-framework,Foundation", "-lresolv"],
105-
"@rules_rs//rs/platforms/constraints:windows_gnu": ["-lws2_32", "-luserenv", "-lbcrypt", "-lntdll", "-lsynchronization"],
106-
"@rules_rs//rs/platforms/constraints:windows_gnullvm": ["-lws2_32", "-luserenv", "-lbcrypt", "-lntdll", "-lsynchronization"],
107-
"@rules_rs//rs/platforms/constraints:windows_msvc": [
105+
"@llvm//constraints/windows/abi:gnu": ["-lws2_32", "-luserenv", "-lbcrypt", "-lntdll", "-lsynchronization"],
106+
"@llvm//constraints/windows/abi:gnullvm": ["-lws2_32", "-luserenv", "-lbcrypt", "-lntdll", "-lsynchronization"],
107+
"@llvm//constraints/windows/abi:msvc": [
108108
"advapi32.lib",
109109
"ws2_32.lib",
110110
"userenv.lib",

test/MODULE.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
bazel_dep(name = "platforms", version = "1.0.0")
2-
bazel_dep(name = "llvm", version = "0.7.6")
2+
bazel_dep(name = "llvm", version = "0.7.7")
33
bazel_dep(name = "apple_support", version = "1.24.2", repo_name = "build_bazel_apple_support")
44

55
# SBOM

test/MODULE.bazel.lock

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)