[Mlir-commits] [mlir] [mlir][bufferize] Add hoist-dynamic-allocs-option to buffer-results-to-out-params (PR #160985)

[llvmlistbot at llvm.org]

srcarroll wrote:

> > maybe a naive question, but is there a reason to have separate options for static and dynamic allocs instead of just one `hoist-allocs`? When would one want to hoist one but not the other?
> 
> Good question. Separating them can improve the coupling of the Pass, but we can also use static allocation and dynamic allocation together. Please tell us your thoughts.

I dont have a strong opinion on it, and I admit that I'm not confident about the repercussions of combining them.  I think having options is always good, so fine with me to keep separate.

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