[clang] [llvm] [Offload] Change ELF machine type for SPIR-V OpenMP image (PR #159623)
Nick Sarnie via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 19 07:10:08 PDT 2025
================
@@ -423,9 +423,10 @@ Error offloading::intel::containerizeOpenMPSPIRVImage(
Header.Class = ELF::ELFCLASS64;
Header.Data = ELF::ELFDATA2LSB;
Header.Type = ELF::ET_DYN;
- // Use an existing Intel machine type as there is not one specifically for
- // Intel GPUs.
- Header.Machine = ELF::EM_IA_64;
+ // Use a fake machine type as there is not one specifically for
+ // Intel GPUs, the associated runtime plugin is looking for
+ // this value.
+ Header.Machine = 0x8086;
----------------
sarnex wrote:
I was worried we'd have to officially add it to ELF spec as per [this](https://gabi.xinuos.com/elf/a-emachine.html) link and then update all userspace tools, if you're okay with us only changing the LLVM ELF tools I'd be happy to do that
https://github.com/llvm/llvm-project/pull/159623
More information about the llvm-commits
mailing list