Skip to content

Commit

Permalink
Automerge: [mlir][bazel] Fix no-allow-shlib-undefined errors.
Browse files Browse the repository at this point in the history
The BUILD file changes in llvm/llvm-project#127544 adds `LinalgInterfaces` which is incomplete without `LinalgDialect`.

For now, just add the `LinalgDialect` as dependency to tests which do not otherwise depend on it (but depend on `LinalgInterfaces` through e.g. `TensorDialect`).

This is a temporary solution until the dependency of `TensorDialect` is trimmed to just the `linalg::RelayoutOpInterface`, but not the other linalg interfaces. See llvm/llvm-project#127544 (review).
  • Loading branch information
chsigg authored and github-actions[bot] committed Feb 18, 2025
2 parents 07d60d5 + d64f177 commit 6478cea
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,7 @@ cc_test(
"//mlir:ArithDialect",
"//mlir:FuncDialect",
"//mlir:IR",
"//mlir:LinalgDialect",
"//mlir:Parser",
"//mlir:SCFDialect",
"//mlir:SideEffectInterfaces",
Expand Down Expand Up @@ -211,6 +212,7 @@ cc_test(
"//llvm:Support",
"//llvm:TestingSupport",
"//mlir:IR",
"//mlir:LinalgDialect",
"//mlir:SPIRVBinaryUtils",
"//mlir:SPIRVDeserialization",
"//mlir:SPIRVDialect",
Expand Down

0 comments on commit 6478cea

Please sign in to comment.