[Mlir-commits] [mlir] Fix llvm-mlir-use-after-erase findings (PR #210733)

[llvmlistbot at llvm.org]

================
@@ -456,17 +457,18 @@ void NormalizeMemRefs::normalizeFuncOpMemRefs(func::FuncOp funcOp,
                                               /*userFilterFn=*/nullptr,
                                               /*allowNonDereferencingOps=*/true,
                                               /*replaceInDeallocOp=*/true))) {
-            newOp->erase();
             replacingMemRefUsesFailed = true;
-            continue;
+            break;
----------------
Harald-R wrote:

I have added a test and reverted `newOp->erase()` being moved after the loop.

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