[llvm] [LLVM] Update CUDA ELF flags for their new ABI (PR #149534)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 21 00:54:24 PDT 2025
================
@@ -295,7 +295,8 @@ TEST(ELFObjectFileTest, CheckOSAndTriple) {
{ELF::EM_X86_64, ELF::ELFOSABI_AIX, "x86_64--aix"},
{ELF::EM_X86_64, ELF::ELFOSABI_FREEBSD, "x86_64--freebsd"},
{ELF::EM_X86_64, ELF::ELFOSABI_OPENBSD, "x86_64--openbsd"},
- {ELF::EM_CUDA, ELF::ELFOSABI_CUDA, "nvptx64-nvidia-cuda"}};
+ {ELF::EM_CUDA, ELF::ELFOSABI_CUDA, "nvptx64-nvidia-cuda"},
+ {ELF::EM_CUDA, ELF::ELFOSABI_CUDA_V2, "nvptx64-nvidia-cuda"}};
----------------
jh7370 wrote:
Similar to above, it feels like it's a mistake for the same string to be used for both variants. NB: I know nothing about Cuda and am approaching this purely from a binary utilities point of view, so it might actually make sense, but I need more explanation.
https://github.com/llvm/llvm-project/pull/149534
More information about the llvm-commits
mailing list