[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:09 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

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