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

llvmlistbot at llvm.org llvmlistbot at llvm.org
Fri Oct 18 06:35:46 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);
   }
----------------
Max191 wrote:

There wouldn't be a failure before the change. The change just makes the pattern run fewer times to achieve the same result.

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


More information about the Mlir-commits mailing list