[Mlir-commits] [mlir] [mlir][TilingInterface] NFC Improve comment for `tiledAndFusedOps` member of `SCFTileAndFuseResult` (PR #145397)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Mon Jun 23 12:54:11 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir-scf
Author: Aaron St George (AaronStGeorge)
<details>
<summary>Changes</summary>
Comment was a little unclear, hopefully this change is an improvement.
---
Full diff: https://github.com/llvm/llvm-project/pull/145397.diff
1 Files Affected:
- (modified) mlir/include/mlir/Dialect/SCF/Transforms/TileUsingInterface.h (+4-3)
``````````diff
diff --git a/mlir/include/mlir/Dialect/SCF/Transforms/TileUsingInterface.h b/mlir/include/mlir/Dialect/SCF/Transforms/TileUsingInterface.h
index 9feb04dbe03c1..7b43aa43c7517 100644
--- a/mlir/include/mlir/Dialect/SCF/Transforms/TileUsingInterface.h
+++ b/mlir/include/mlir/Dialect/SCF/Transforms/TileUsingInterface.h
@@ -272,9 +272,10 @@ FailureOr<SmallVector<Operation *>> yieldReplacementForFusedProducer(
struct SCFTileAndFuseResult {
/// List of untiled operations that were fused with the tiled consumer.
llvm::SetVector<Operation *> fusedProducers;
- /// List of tiled and fused operations generated. The first one in this list
- /// is guaranteed to be the tiled operations generated during tiling of the
- /// generated operation.
+ /// List of tiled and fused operations generated. The first element is always
+ /// the tiled version of the original consumer operation processed by
+ /// `tileConsumerAndFuseProducersUsingSCF`, followed by any operations that
+ /// were fused with it.
llvm::SetVector<Operation *> tiledAndFusedOps;
/// The `scf.for` operations that iterate over the tiles.
SmallVector<LoopLikeOpInterface> loops;
``````````
</details>
https://github.com/llvm/llvm-project/pull/145397
More information about the Mlir-commits
mailing list