[llvm] [Bazel] Add missing dependencies. (PR #172688)

Weiwei C via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 17 09:24:33 PST 2025


================
@@ -2643,15 +2643,32 @@ cc_library(
         ":LLVMDialect",
         ":LinalgDialect",
         ":LinalgInterfaces",
+        ":MemRefUtils",
         ":Pass",
         ":TransformUtils",
         ":VectorDialect",
         ":VectorUtils",
         ":X86VectorDialect",
+        ":X86VectorUtils",
         "//llvm:Support",
     ],
 )
 
+cc_library(
+    name = "X86VectorUtils",
+    srcs = glob(["lib/Dialect/X86Vector/Utils/*.cpp"]),
+    hdrs = glob(["include/mlir/Dialect/X86Vector/Utils/*.h"]),
+    includes = ["include"],
+    deps = [
+        ":AffineDialect",
+        ":DialectUtils",
+        ":FuncDialect",
+        ":IR",
+        ":LinalgDialect",
----------------
weiweichen wrote:

hmm, the corresponding cmake changes don't seem to have mlir support, LinalgInterfaces and llvm::Support https://github.com/llvm/llvm-project/pull/170267/changes#diff-60c75ecf0e82a7b9a4f240c0461827fc1556c75c3d002065263afbc12f15586cR7-R13, nor do they exist in current `main` https://github.com/llvm/llvm-project/blob/2f715e96c0271e0767f1fc06ba757ee959484bda/mlir/lib/Dialect/X86Vector/Utils/CMakeLists.txt#L7-L13
I also have tested this locally (since we rely bazel overlay to compile our llvm dependency and we keep a weekly update) which works fine. If these are transitive dependencies, I think other bazel rules would cover them so that they don't have to be spelled out explicitly here? 

https://github.com/llvm/llvm-project/pull/172688


More information about the llvm-commits mailing list