[Mlir-commits] [mlir] [mlir] Enable LICM for ops with only read side effects in scf.for (PR #120302)

[Thomas Raoux]

https://github.com/ThomasRaoux commented:

I don't have any objection to this PR but I find the interface a bit leaky and the transformation a bit opinionated for it to be live here.
I'm not actively maintaining this part of the code so if someone who does could weight in that would be better.
Otherwise like I said I don't any objection of this going in.

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