[Mlir-commits] [mlir] [mlir][SCF] Add `RecursiveMemoryEffects` to `scf.reduce` (PR #75314)

[Mehdi Amini]

joker-eph wrote:

> scf.reduce has a weird semantics, it describes how values should be reduced between scf.parallel loop interations.

Ah right I misread and in my head was thinking about the loop construct itself.

Any reason the multiple `scf.reduce` can't be consolidated as a terminator? 

(I don't know if there are other ops like `scf.reduce` that would want to "communicate with the parent op" without being a terminator or have side-effects)

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