llvm icon indicating copy to clipboard operation
llvm copied to clipboard

Update CudaArch structs and functions in clang to something more generic

Open npmiller opened this issue 4 years ago • 1 comments

The CudaArch struct and functions include AMD GPU values in:

  • https://github.com/intel/llvm/blob/sycl/clang/lib/Basic/Cuda.cpp#L73

It would be good to revisit this and change it to something more generic like maybe GPUArch or something like that as the Cuda tag doesn't really reflect what it is anymore.

This was discussed on the following PR:

  • https://github.com/intel/llvm/pull/4239#discussion_r684087808

npmiller avatar Aug 06 '21 14:08 npmiller

Upstream PR: https://github.com/llvm/llvm-project/pull/97028

jchlanda avatar Jun 28 '24 10:06 jchlanda