[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