[Mlir-commits] [mlir] [mlir][IR] Add rewriter API for moving operations (PR #78988)

[Matthias Springer]

================
@@ -193,9 +193,7 @@ struct HoistEligibleOps : public OpRewritePattern<test::OneRegionOp> {
       return failure();
     if (!toBeHoisted->hasAttr("eligible"))
       return failure();
-    // Hoisting means removing an op from the enclosing op. I.e., the enclosing
----------------
matthias-springer wrote:

This was a workaround around the fact that there were no notifications for `moveBefore`. This used to trigger a failed "expensive pattern check" (IR changed but rewriter was not notified).


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