[PATCH] D80136: [mlir] Erase or clear blocks through ConversionPatternRewriter when applicable

[River Riddle via Phabricator via llvm-commits]

rriddle added inline comments.


================
Comment at: mlir/examples/toy/Ch5/mlir/LowerToAffineLoops.cpp:75
     auto loop = rewriter.create<AffineForOp>(loc, /*lb=*/0, dim, /*step=*/1);
-    loop.getBody()->clear();
+    for (Operation &nested : *loop.getBody())
+      rewriter.eraseOp(&nested);
----------------
This was because originally ensureTerminator wasn't tracked, so this could now be removed entirely. The create<AffineTerminatorOp> below would also need to be removed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80136/new/

https://reviews.llvm.org/D80136