[flang-commits] [mlir] [flang] [mlir][TilingInterface] Use `LoopLikeOpInterface` in tiling using SCF to unify tiling with `scf.for` and `scf.forall`. (PR #77874)

via flang-commits flang-commits at lists.llvm.org
Thu Jan 25 15:25:29 PST 2024


MaheshRavishankar wrote:

> This PR looks fine to me, thanks!
> 
> I'd like to see the cleanup work truly start before we land more code.
> 
> ```
> There is lot of context here spread all over the place w.r.t tiling. I want to unify all of it, but I dont think I can do that in one step. I would rather incrementally build up the required functionality in one place and deprecate/move pieces as needed. I do plan to do this in short order though..
> ```
> 
> To be clear, I did not ask for a making this PR more complex but specifically asked for a followup PR to see that work has truly unambiguously started on the deprecation of the legacy implementation (I already voiced this back in Nov: [#72178 (comment)](https://github.com/llvm/llvm-project/pull/72178#issuecomment-1810872756)).
> 
> Now is the time to see concrete movement on the deprecation before we land more code.

I agree, my next step is to move the transform dialect op to use this implementation. So thats top on the list.

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


More information about the flang-commits mailing list