[llvm] [bazel] More port of #148286: only include spirv dep when enabled (PR #153531)

Jordan Rupprecht via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 13 20:52:11 PDT 2025


https://github.com/rupprecht created https://github.com/llvm/llvm-project/pull/153531

The fix in #153520 works so long as SPIRV is actually an enabled target. Otherwise, the build fails w/ `target 'SPIRVCodeGen' not declared in package ...`

>From 9259b43d4b2c973c8961dc3575a9cbe16bd9cbff Mon Sep 17 00:00:00 2001
From: Jordan Rupprecht <rupprecht at google.com>
Date: Wed, 13 Aug 2025 20:49:29 -0700
Subject: [PATCH] Only build if spirv is enabled

---
 utils/bazel/llvm-project-overlay/mlir/BUILD.bazel | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/utils/bazel/llvm-project-overlay/mlir/BUILD.bazel b/utils/bazel/llvm-project-overlay/mlir/BUILD.bazel
index 5b8c120dec5c5..6a15b0c88b86b 100644
--- a/utils/bazel/llvm-project-overlay/mlir/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/mlir/BUILD.bazel
@@ -8,6 +8,7 @@
 load("@bazel_skylib//rules:common_settings.bzl", "bool_flag")
 load("@bazel_skylib//rules:expand_template.bzl", "expand_template")
 load("@rules_cc//cc:defs.bzl", "cc_binary", "cc_library")
+load("//llvm:targets.bzl", "llvm_targets")
 load(
     ":build_defs.bzl",
     "cc_headers_only",
@@ -13839,11 +13840,13 @@ cc_library(
         "//llvm:BitWriter",
         "//llvm:Core",
         "//llvm:Object",
-        "//llvm:SPIRVCodeGen",
         "//llvm:Support",
         "//llvm:Target",
         "//llvm:config",
-    ],
+    ] + ([
+        "//llvm:SPIRVCodeGen",
+        "//llvm:SPIRVUtilsAndDesc",
+    ] if "SPIRV" in llvm_targets else []),
 )
 
 cc_library(



More information about the llvm-commits mailing list