[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:21 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir-affine
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