[Mlir-commits] [mlir] [mlir][Transforms][NFC] Dialect conversion: Simplify `EraseBlockRewrite` constructor (PR #99805)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Sun Jul 21 01:58:51 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
@llvm/pr-subscribers-mlir-core
Author: Matthias Springer (matthias-springer)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/99805.diff
1 Files Affected:
- (modified) mlir/lib/Transforms/Utils/DialectConversion.cpp (+4-7)
``````````diff
diff --git a/mlir/lib/Transforms/Utils/DialectConversion.cpp b/mlir/lib/Transforms/Utils/DialectConversion.cpp
index 0b552a7e1ca3b..a045868b66031 100644
--- a/mlir/lib/Transforms/Utils/DialectConversion.cpp
+++ b/mlir/lib/Transforms/Utils/DialectConversion.cpp
@@ -293,10 +293,9 @@ class CreateBlockRewrite : public BlockRewrite {
/// original location.
class EraseBlockRewrite : public BlockRewrite {
public:
- EraseBlockRewrite(ConversionPatternRewriterImpl &rewriterImpl, Block *block,
- Region *region, Block *insertBeforeBlock)
- : BlockRewrite(Kind::EraseBlock, rewriterImpl, block), region(region),
- insertBeforeBlock(insertBeforeBlock) {}
+ EraseBlockRewrite(ConversionPatternRewriterImpl &rewriterImpl, Block *block)
+ : BlockRewrite(Kind::EraseBlock, rewriterImpl, block),
+ region(block->getParent()), insertBeforeBlock(block->getNextNode()) {}
static bool classof(const IRRewrite *rewrite) {
return rewrite->getKind() == Kind::EraseBlock;
@@ -1440,9 +1439,7 @@ void ConversionPatternRewriterImpl::notifyOpReplaced(Operation *op,
}
void ConversionPatternRewriterImpl::notifyBlockIsBeingErased(Block *block) {
- Region *region = block->getParent();
- Block *origNextBlock = block->getNextNode();
- appendRewrite<EraseBlockRewrite>(block, region, origNextBlock);
+ appendRewrite<EraseBlockRewrite>(block);
}
void ConversionPatternRewriterImpl::notifyBlockInserted(
``````````
</details>
https://github.com/llvm/llvm-project/pull/99805
More information about the Mlir-commits
mailing list