[Mlir-commits] [mlir] [MLIR][LLVM] Import LLVM target triple into MLIR LLVM Dialect (PR #125084)

[Krzysztof Drewniak]

krzysz00 wrote:

If I had a vote, there should be a lossless `[some sort of attribute] toMLIRDataLayout(const llvm::DataLayout& DL)` - or perhaps `[some sort of attribute] toMLIRDataLayout(StringRef llvmDataLayoutSpec)` and its inverse.

But that's not this PR, I don't think.

I'll also toss out the radical position that a conversion from, say, `memref` to LLVM IR isn't valid unless there's a data layout spec present, since things like `sizeof(!llvm.ptr)` - or even `sizeof(index)` - are target-dependent and target-independent LLVM IR is an illusion.

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