[llvm] [Bazel] NFC refactor out redundant is_x86_64_non_windows config (PR #180296)

via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 6 15:02:07 PST 2026


https://github.com/HighW4y2H3ll updated https://github.com/llvm/llvm-project/pull/180296

>From 3330bb26b43f2d5710441d39340400c143c874f1 Mon Sep 17 00:00:00 2001
From: h2h <h2h at meta.com>
Date: Fri, 6 Feb 2026 14:22:36 -0800
Subject: [PATCH 1/3] [Bazel] NFC refactor out redundant is_x86_64_non_windows
 config

---
 .../llvm-project-overlay/llvm/BUILD.bazel     | 53 ++++---------------
 1 file changed, 10 insertions(+), 43 deletions(-)

diff --git a/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel b/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
index e83a237a5397f..70ccf00220f3a 100644
--- a/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
@@ -256,41 +256,6 @@ config_setting(
     flag_values = {"@rules_cc//cc/compiler:compiler": "clang"},
 )
 
-# TODO(zbarsky): Is there a better way to express this?
-config_setting(
-    name = "_stamp_binary_on_check",
-    values = {"stamp": "1"},
-)
-
-config_setting(
-    name = "_stamp_binary_off_check",
-    values = {"stamp": "0"},
-)
-
-selects.config_setting_group(
-    name = "_true",
-    match_any = [
-        ":_stamp_binary_on_check",
-        ":_stamp_binary_off_check",
-    ],
-)
-
-alias(
-    name = "non_windows",
-    actual = select({
-        "@platforms//os:windows": "@platforms//os:macos",
-        "//conditions:default": ":_true",
-    }),
-)
-
-selects.config_setting_group(
-    name = "is_x86_64_non_windows",
-    match_all = [
-        "@platforms//cpu:x86_64",
-        ":non_windows",
-    ],
-)
-
 BLAKE3_x86_64_ASM_SOURCE_PATTERNS = [
     "lib/Support/BLAKE3/blake3_avx2_x86-64_%s.S",
     "lib/Support/BLAKE3/blake3_avx512_x86-64_%s.S",
@@ -326,14 +291,16 @@ cc_library(
         "@platforms//cpu:aarch64": [
             "lib/Support/BLAKE3/blake3_neon.c",
         ],
-        ":is_x86_64_windows_clang_mingw": [
-            pattern % "windows_gnu"
-            for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
-        ],
-        ":is_x86_64_non_windows": [
-            pattern % "unix"
-            for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
-        ],
+        "@platforms//cpu:x86_64": select({
+            ":is_x86_64_windows_clang_mingw": [
+                pattern % "windows_gnu"
+                for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
+            ],
+            "//conditions:default": [
+                pattern % "unix"
+                for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
+            ],
+        }),
         "//conditions:default": [
         ],
     }),

>From 9301fa8978eb3a93462e1ec4f5ac4914ff18acac Mon Sep 17 00:00:00 2001
From: h2h <h2h at meta.com>
Date: Fri, 6 Feb 2026 14:57:51 -0800
Subject: [PATCH 2/3] Introduce filegroup to aggregate x86_64 blake3 asm

---
 .../llvm-project-overlay/llvm/BUILD.bazel     | 25 +++++++++++--------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel b/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
index 70ccf00220f3a..51793cb2482f5 100644
--- a/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
@@ -263,6 +263,20 @@ BLAKE3_x86_64_ASM_SOURCE_PATTERNS = [
     "lib/Support/BLAKE3/blake3_sse41_x86-64_%s.S",
 ]
 
+filegroup(
+    name = "blake3_x86_64_asm",
+    srcs = select({
+        ":is_x86_64_windows_clang_mingw": [
+            pattern % "windows_gnu"
+            for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
+        ],
+        "//conditions:default": [
+            pattern % "unix"
+            for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
+        ],
+    }),
+)
+
 cc_library(
     name = "Support",
     srcs = glob([
@@ -291,16 +305,7 @@ cc_library(
         "@platforms//cpu:aarch64": [
             "lib/Support/BLAKE3/blake3_neon.c",
         ],
-        "@platforms//cpu:x86_64": select({
-            ":is_x86_64_windows_clang_mingw": [
-                pattern % "windows_gnu"
-                for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
-            ],
-            "//conditions:default": [
-                pattern % "unix"
-                for pattern in BLAKE3_x86_64_ASM_SOURCE_PATTERNS
-            ],
-        }),
+        "@platforms//cpu:x86_64": ":blake3_x86_64_asm",
         "//conditions:default": [
         ],
     }),

>From 716b08221be13de5f3f89a2c023860afca7a67c0 Mon Sep 17 00:00:00 2001
From: h2h <h2h at meta.com>
Date: Fri, 6 Feb 2026 15:01:54 -0800
Subject: [PATCH 3/3] wrap filegroup taget in array

---
 utils/bazel/llvm-project-overlay/llvm/BUILD.bazel | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel b/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
index 51793cb2482f5..5d11d04f32001 100644
--- a/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
@@ -305,7 +305,7 @@ cc_library(
         "@platforms//cpu:aarch64": [
             "lib/Support/BLAKE3/blake3_neon.c",
         ],
-        "@platforms//cpu:x86_64": ":blake3_x86_64_asm",
+        "@platforms//cpu:x86_64": [":blake3_x86_64_asm"],
         "//conditions:default": [
         ],
     }),



More information about the llvm-commits mailing list