[Mlir-commits] [mlir] [mlir] Add forall canonicalization to replace constant induction vars (PR #112764)

Mehdi Amini llvmlistbot at llvm.org
Thu Oct 17 16:05:40 PDT 2024


================
@@ -1787,9 +1787,9 @@ struct ForallOpReplaceConstantInductionVar : public OpRewritePattern<ForallOp> {
       }
       rewriter.replaceAllUsesWith(
           iv, getValueOrCreateConstantIndexOp(rewriter, loc, lb));
-      return success();
+      changed = true;
     }
-    return failure();
+    return success(changed);
   }
----------------
joker-eph wrote:

In this commit you removed an early return: can you add a test demonstrating the behavior change here?

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


More information about the Mlir-commits mailing list