[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