On Tue, Feb 20, 2024 at 05:39:39PM +0100, Tobias Burnus wrote:
> clang/lib/Headers/openmp_wrappers/complex:    device = {arch(amdgcn, nvptx, 
> nvptx64)},                                   \

That one doesn't really need the nvptx64 support.
> --- a/gcc/config/nvptx/nvptx.cc
> +++ b/gcc/config/nvptx/nvptx.cc
> @@ -6403,7 +6403,7 @@ nvptx_omp_device_kind_arch_isa (enum 
> omp_device_kind_arch_isa trait,
>      case omp_device_kind:
>        return strcmp (name, "gpu") == 0;
>      case omp_device_arch:
> -      return strcmp (name, "nvptx") == 0;
> +      return strcmp (name, "nvptx") == 0 || strcmp (name, "nvptx64") == 0;

Maybe guard the nvptx64 on TARGET_ABI64, at least as long as we have that?
Just in case we'd reconsider at some point the -m64 only thing.

Otherwise LGTM.

        Jakub

Reply via email to