[Mlir-commits] [mlir] [mlir][PartialReductionTilingInterface] Generalize implementation of `tileUsingSCF` for `ReductionTilingStrategy::PartialOuterReduction`. (PR #143467)

[Pietro Ghiglio]

PietroGhg wrote:

Hi @MaheshRavishankar, I have a question about a change you made in this PR: you added a check for parallel dimensions and `reductionStrategy != ReductionTilingStrategy::FullReduction` (`mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp` line 202 in this diff), I have a usage of `PartialTilingInterface` that has both parallel and reduction iteration types, which now fails due to this check, and I was wondering the rationale behind it. Can you explain why it's needed? Thank you :)

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