[Mlir-commits] [mlir] [mlir][scf] Fix scf.forall to scf.parallel pass walker (PR #95385)

[Spenser Bauman]

================
@@ -71,8 +71,9 @@ struct ForallToParallelLoop final
 
     parentOp->walk([&](scf::ForallOp forallOp) {
       if (failed(scf::forallToParallelLoop(rewriter, forallOp))) {
-        return signalPassFailure();
+        return WalkResult::skip();
----------------
sabauma wrote:

The `scf::forallToParallelLoop` function internally calls `notifyMatchFailure`, so some diagnostic should occur. That may not mean much if the pass terminates successfully though.

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