[Mlir-commits] [mlir] [mlir] Fix typo in CSE transform (PR #137341)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Fri Apr 25 07:35:50 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
Author: bubblepipe (bubblepipe)
<details>
<summary>Changes</summary>
"Replace all uses, but do not _remote_ the operation yet" should be "Replace all uses, but do not _remove_ the operation yet".
---
Full diff: https://github.com/llvm/llvm-project/pull/137341.diff
1 Files Affected:
- (modified) mlir/lib/Transforms/CSE.cpp (+2-2)
``````````diff
diff --git a/mlir/lib/Transforms/CSE.cpp b/mlir/lib/Transforms/CSE.cpp
index 3affd88d158de..8e03f62894bab 100644
--- a/mlir/lib/Transforms/CSE.cpp
+++ b/mlir/lib/Transforms/CSE.cpp
@@ -140,7 +140,7 @@ void CSEDriver::replaceUsesAndDelete(ScopedMapTy &knownValues, Operation *op,
if (auto *rewriteListener =
dyn_cast_if_present<RewriterBase::Listener>(rewriter.getListener()))
rewriteListener->notifyOperationReplaced(op, existing);
- // Replace all uses, but do not remote the operation yet. This does not
+ // Replace all uses, but do not remove the operation yet. This does not
// notify the listener because the original op is not erased.
rewriter.replaceAllUsesWith(op->getResults(), existing->getResults());
opsToErase.push_back(op);
@@ -156,7 +156,7 @@ void CSEDriver::replaceUsesAndDelete(ScopedMapTy &knownValues, Operation *op,
if (all_of(v.getUses(), wasVisited))
rewriteListener->notifyOperationReplaced(op, existing);
- // Replace all uses, but do not remote the operation yet. This does not
+ // Replace all uses, but do not remove the operation yet. This does not
// notify the listener because the original op is not erased.
rewriter.replaceUsesWithIf(op->getResults(), existing->getResults(),
wasVisited);
``````````
</details>
https://github.com/llvm/llvm-project/pull/137341
More information about the Mlir-commits
mailing list