[Mlir-commits] [mlir] [MLIR][Affine] NFC. Fix outdated comment in affine analysis utils (PR #128406)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Sat Feb 22 23:23:20 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Uday Bondhugula (bondhugula)

<details>
<summary>Changes</summary>



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


1 Files Affected:

- (modified) mlir/lib/Dialect/Affine/Analysis/Utils.cpp (+2-3) 


``````````diff
diff --git a/mlir/lib/Dialect/Affine/Analysis/Utils.cpp b/mlir/lib/Dialect/Affine/Analysis/Utils.cpp
index 7f06bc5160b9d..ba6f045cff408 100644
--- a/mlir/lib/Dialect/Affine/Analysis/Utils.cpp
+++ b/mlir/lib/Dialect/Affine/Analysis/Utils.cpp
@@ -282,9 +282,8 @@ bool MemRefDependenceGraph::init() {
       // Create graph node for top-level op unless it is known to be
       // memory-effect free. This covers all unknown/unregistered ops,
       // non-affine ops with memory effects, and region-holding ops with a
-      // well-defined control flow. During the fusion validity checks, we look
-      // for non-affine ops on the path from source to destination, at which
-      // point we check which memrefs if any are used in the region.
+      // well-defined control flow. During the fusion validity checks, edges
+      // to/from these ops get looked at.
       Node *node = addNodeToMDG(&op, *this, memrefAccesses);
       if (!node)
         return false;

``````````

</details>


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


More information about the Mlir-commits mailing list