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

[Quinn Dawkins]

qedawkins wrote:

> 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).

I did not, it was just an attempt to keep the API NFC. I can happily drop the flag and default it to true then.

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