================ @@ -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; ---------------- jhuber6 wrote:
It'd obviously be best to go the official route if this is going to be used in a real context, if it's just a test then I guess we can keep it as a constant. But I would definitely question why Intel hasn't already defined this if it's their official ELF format. https://github.com/llvm/llvm-project/pull/159623 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
