[Mlir-commits] [mlir] [mlir][bufferization] Enable moving dependent values in eliminate-empty-tensors (PR #169718)

[llvmlistbot at llvm.org]

https://github.com/MaheshRavishankar commented:

Did you have a use case for `ignoreSideEffects = true`. It is probably better to have `ignoreSideEffects = false` by default (or just not change the API and make sure that we always account for side-effecting operations).

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