[llvm] lit_test : check if there is already a deps key in kwargs (PR #87290)
Abhinav Gunjal via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 1 15:26:39 PDT 2024
https://github.com/abhigunj created https://github.com/llvm/llvm-project/pull/87290
This change checks if there is already a deps key in kwargs and concatenate it to avoid multiple values for 'deps' key argument.
background:
https://github.com/llvm/llvm-project/pull/87022 recently added explicit `deps` to the lit_test. This is causing StableHLO bazel build failures at https://github.com/openxla/stablehlo/actions/runs/8511888283/job/23312383380?pr=2147
>From 1b33d98179c3b6f71b6c0dca0edfa6df5d9f5630 Mon Sep 17 00:00:00 2001
From: Abhinav <agunjal at google.com>
Date: Mon, 1 Apr 2024 20:49:23 +0000
Subject: [PATCH] check if there is already a deps key in kwargs
---
utils/bazel/llvm-project-overlay/llvm/lit_test.bzl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/utils/bazel/llvm-project-overlay/llvm/lit_test.bzl b/utils/bazel/llvm-project-overlay/llvm/lit_test.bzl
index f754a9fc7d5e48..b5cb9f375ebce6 100644
--- a/utils/bazel/llvm-project-overlay/llvm/lit_test.bzl
+++ b/utils/bazel/llvm-project-overlay/llvm/lit_test.bzl
@@ -35,7 +35,7 @@ def lit_test(
args = args + ["-v"] + ["$(execpath %s)" % src for src in srcs],
data = data + srcs,
legacy_create_init = False,
- deps = [Label("//llvm:lit")],
+ deps = [Label("//llvm:lit")] + kwargs.pop("deps", []),
**kwargs
)
More information about the llvm-commits
mailing list