[flang-commits] [flang] [mlir] [mlir][CSE] Introduce hoist-pure-ops logic to CSE pass (PR #180556)

[lonely eagle via flang-commits]

linuxlonelyeagle wrote:

> Given that LLVM's `gvn-hoist` (and not the regular CSE) is doing the same transformation, I would recommend adding an extra option to the CSE pass that would enable such hoisting in top of the current CSE logic.

I verified this point in the RFC; it indeed behaves differently from CSE in LLVM. I have considered this approach before(add  a option), but my concern is that the pass results might expose some underlying op design issues, similar to what we are discussing here.

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