[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