[Mlir-commits] [mlir] [mlir] harden transform.test_emit_remark_and_erase (PR #136307)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Fri Apr 18 07:06:08 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
Author: Oleksandr "Alex" Zinenko (ftynse)
<details>
<summary>Changes</summary>
Executing this transform op may crash when the payload op is not erasable. Guard against that. Since this is a test op, no additional test is defined.
Fixes #<!-- -->128323.
---
Full diff: https://github.com/llvm/llvm-project/pull/136307.diff
1 Files Affected:
- (modified) mlir/test/lib/Dialect/Transform/TestTransformDialectExtension.cpp (+4-1)
``````````diff
diff --git a/mlir/test/lib/Dialect/Transform/TestTransformDialectExtension.cpp b/mlir/test/lib/Dialect/Transform/TestTransformDialectExtension.cpp
index face4de8e27d3..cdf44c2959d50 100644
--- a/mlir/test/lib/Dialect/Transform/TestTransformDialectExtension.cpp
+++ b/mlir/test/lib/Dialect/Transform/TestTransformDialectExtension.cpp
@@ -346,8 +346,11 @@ DiagnosedSilenceableFailure mlir::test::TestEmitRemarkAndEraseOperandOp::apply(
transform::TransformRewriter &rewriter,
transform::TransformResults &results, transform::TransformState &state) {
emitRemark() << getRemark();
- for (Operation *op : state.getPayloadOps(getTarget()))
+ for (Operation *op : state.getPayloadOps(getTarget())) {
+ if (!op->getUses().empty())
+ return emitSilenceableError() << "cannot erase an op that has uses";
rewriter.eraseOp(op);
+ }
if (getFailAfterErase())
return emitSilenceableError() << "silenceable error";
``````````
</details>
https://github.com/llvm/llvm-project/pull/136307
More information about the Mlir-commits
mailing list