================ @@ -362,14 +362,20 @@ mlir::scf::tileUsingSCFForOp(RewriterBase &rewriter, TilingInterface op, auto clonedOp = cast<TilingInterface>( cloneOpAndUpdateDestinationArgs(rewriter, op, clonedOpDestination)); - // 5b. Tile the cloned operation. + // 5b. Early return cloned op if tiling is not happening. ---------------- MaheshRavishankar wrote:
Yeah, can't replace an op with itself. In theory you could say the caller should check for this and not do the replace, but that is strange. Let's go with this. https://github.com/llvm/llvm-project/pull/75410 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits