[Mlir-commits] [mlir] [mlir][CSE] Remove the opsToErase container and immediately delete dead ops. (PR #203702)

[lonely eagle]

================

----------------
linuxlonelyeagle wrote:

Perhaps the current implementation is acceptable for now. However, since the logic to update the `*changed `variable is embedded inside eraseDeadOp, it runs redundantly multiple times for every single dead op being deleted. I doubt this is necessary; we should probably just update it once at the very end. If there are no further questions, I will merge this.😉

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