[Mlir-commits] [mlir] [MLIR][DLTI] Fix access to Data Layout info in -to-llvm's type converter (PR #127239)

Fabian Mora llvmlistbot at llvm.org
Mon Feb 17 04:48:45 PST 2025


fabianmcg wrote:

> Note that `LLVMTypeConverter` does not have access to the op, so the responsibility lies with the pass.

I thought that it was possible to get the default layout from `DataLayoutAnalysis` but it appears that I was wrong https://github.com/llvm/llvm-project/blob/main/mlir/include/mlir/Analysis/DataLayoutAnalysis.h . I'm okay with the change, but please add a description and fix the formatting.

> It turns out other passes are also not behaving as expected

Do you mean wrt to the layout?

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


More information about the Mlir-commits mailing list