[Mlir-commits] [mlir] [mlir][reducer] Change mlir-reducer apply pattern logic (PR #195997)

[lonely eagle]

linuxlonelyeagle wrote:

> > Applying patterns to the entire module right from the start is too aggressive and has too broad an impact.
> 
> Ok, I didn't notice that you removed `OpsInRange` completely. So you reversed the reduction strategy here, meaning that we start from applying to no operations and then move on. Got it.

We can think about it abstractly: imagine there is a pool of operations. When I delete or apply patterns, I take out one half first, and then take out half of the remaining half, and so on...

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