[Mlir-commits] [mlir] [mlir][LowerToLLVM] Pass DataLayoutAnalysis to LowerToLLVMOptions in `ArithToLLVM`, `ControlFlowToLLVM`, and `VectorToLLVM` passes (PR #206380)

[Andrzej Warzyński]

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

----------------
banach-space wrote:

> How about trying to adopt the same approach in this PR? I’m happy to do that.

IMHO, that would be out-of-scope. Also, against the earlier [ask](https://github.com/llvm/llvm-project/pull/206380#discussion_r3645636647) by @ftynse .

> In any case, I want to point out that if this wasn't intentional, I don't think it's an easy fix. 

Ack

>  Let me know if you prefer to split them up...

I'd leave that to the maintainers ;-)

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