Re: [Patch] nvptx: Add -mptx=6.0 + -misa=sm_70

On 2/22/22 17:03, Tobias Burnus wrote: Hi Tom, On 22.02.22 15:43, Tom de Vries wrote: On 2/17/22 18:24, Tobias Burnus wrote: --- a/gcc/config/nvptx/t-omp-device +++ b/gcc/config/nvptx/t-omp-device @@ -1,4 +1,4 @@ echo kind: gpu > $@ echo arch: nvptx >> $@ -    echo isa: sm_30 sm_35

Re: [Patch] nvptx: Add -mptx=6.0 + -misa=sm_70

Hi Tom, On 22.02.22 15:43, Tom de Vries wrote: On 2/17/22 18:24, Tobias Burnus wrote: --- a/gcc/config/nvptx/t-omp-device +++ b/gcc/config/nvptx/t-omp-device @@ -1,4 +1,4 @@ echo kind: gpu > $@ echo arch: nvptx >> $@ -echo isa: sm_30 sm_35 >> $@ +echo isa: sm_30 sm_35 sm_53

Re: [Patch] nvptx: Add -mptx=6.0 + -misa=sm_70

On 2/17/22 18:24, Tobias Burnus wrote: diff --git a/gcc/config/nvptx/t-omp-device b/gcc/config/nvptx/t-omp-device index 8765d9f1881..4228218a424 100644 --- a/gcc/config/nvptx/t-omp-device +++ b/gcc/config/nvptx/t-omp-device @@ -1,4 +1,4 @@ omp-device-properties-nvptx:

Re: [Patch] nvptx: Add -mptx=6.0 + -misa=sm_70

On 2/17/22 18:24, Tobias Burnus wrote: SM version (-misa=) [Patch adds -misa=sm_70] * The compiler supports internally: SM_30, SM_35, SM_53, SM_70, SM_75, SM_80. I'd formulate it like: it uses SM_70 internally to accurately formulate when certain insns can be used. I think it makes sense

Re: [Patch] nvptx: Add -mptx=6.0 + -misa=sm_70

On 2/17/22 18:24, Tobias Burnus wrote: PTX version (-mptx=) [patch adds -mptx=6.0 as option] * Currently supported internally are 3.1 (CUDA 5.0, used by GCC <= 11),   6.0 (CUDA 9.0, current GCC 12 default), 6.3 (CUDA 10.0), 7.0 (CUDA 11.0) * -mptx= supports 3.1, 6.3, 7.0 – but not the internal

[Patch] nvptx: Add -mptx=6.0 + -misa=sm_70

This patch exposes two -m* option values which are already internally available. I think it makes sense to expose them explicitly to the user (see below), but there are also arguments against. Thoughts? PTX version (-mptx=) [patch adds -mptx=6.0 as option] * Currently supported internally are