[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