[Mlir-commits] [mlir] [mlir][ROCDL] Set the LLVM data layout when lowering to ROCDL LLVM (PR #74501)

Krzysztof Drewniak llvmlistbot at llvm.org
Thu Dec 7 08:51:16 PST 2023


krzysz00 wrote:

@arsenm We _did_ set it before, but down in `SerializeToHsaco`, where we generated the LLVM module.

The potential issue is that here, when we're translating to the LLVM IR dialect, we don't have the data layout yet.

Speaking of, would it be possible to get, in a public header, `StringRef getAMDGPUDataLayout(...)` (... might be a chip name or a triple or something, not sure) so that I don't have to fragilely copy the string over?

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


More information about the Mlir-commits mailing list