[Mlir-commits] [mlir] [mlir][linalg] Fix module dependency issue due to unused import (PR #145727)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Wed Jun 25 08:51:04 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir-linalg

Author: Spenser Bauman (sabauma)

<details>
<summary>Changes</summary>

This include introduces a dependency for LinalgTransforms on LinalgTransformOps, which is unspecified in the module dependencies, and would produce a cyclic dependency if it were specified.

The include is unused in WinogradConv2D.cpp, so this change removes it.

---
Full diff: https://github.com/llvm/llvm-project/pull/145727.diff


1 Files Affected:

- (modified) mlir/lib/Dialect/Linalg/Transforms/WinogradConv2D.cpp (-1) 


``````````diff
diff --git a/mlir/lib/Dialect/Linalg/Transforms/WinogradConv2D.cpp b/mlir/lib/Dialect/Linalg/Transforms/WinogradConv2D.cpp
index c61b23c63dc56..4e90defebcf5c 100644
--- a/mlir/lib/Dialect/Linalg/Transforms/WinogradConv2D.cpp
+++ b/mlir/lib/Dialect/Linalg/Transforms/WinogradConv2D.cpp
@@ -15,7 +15,6 @@
 #include "mlir/Dialect/Affine/IR/AffineOps.h"
 #include "mlir/Dialect/Arith/IR/Arith.h"
 #include "mlir/Dialect/Linalg/IR/Linalg.h"
-#include "mlir/Dialect/Linalg/TransformOps/LinalgTransformOps.h"
 #include "mlir/Dialect/Linalg/Utils/Utils.h"
 #include "mlir/Dialect/Tensor/IR/Tensor.h"
 #include "mlir/Dialect/Utils/StaticValueUtils.h"

``````````

</details>


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


More information about the Mlir-commits mailing list