Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-04-05 Thread Tom de Vries
On 03/02/2018 08:18 PM, Cesar Philippidis wrote: The attached patch adjusts the existing goacc validate_dims target hook and introduces a new goacc adjust_parallelism target hook. The attached patch now just introduces the nvptx_adjust_parallelism target hook implementation, which enables

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-27 Thread Tom de Vries
On 03/26/2018 06:33 PM, Tom de Vries wrote: + loop->mask = targetm.goacc.adjust_parallelism (loop->mask, outer_mask); loop->mask |= this_mask; I committed the above, but the original: ... @@ -1397,6 +1407,8 @@ oacc_loop_auto_partitions (oacc_loop *loop, unsigned outer_mask,

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-26 Thread Tom de Vries
On 03/02/2018 08:18 PM, Cesar Philippidis wrote: The attached patch adjusts the existing goacc validate_dims target hook This is overkill. All we need is a function "int oacc_get_default_dim (int dim)". Thanks, - Tom

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-26 Thread Tom de Vries
On 03/02/2018 08:18 PM, Cesar Philippidis wrote: introduces a new goacc adjust_parallelism target hook. That's another separate patch. Committed. Thanks, - Tom [openacc] Add target hook TARGET_GOACC_ADJUST_PARALLELISM 2018-03-26 Cesar Philippidis Tom de Vries

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-26 Thread Cesar Philippidis
On 03/26/2018 07:14 AM, Tom de Vries wrote: > On 03/02/2018 08:18 PM, Cesar Philippidis wrote: >> diff --git a/gcc/omp-offload.c b/gcc/omp-offload.c >> index ba3f4317f4e..f15ce6b8f8d 100644 >> --- a/gcc/omp-offload.c >> +++ b/gcc/omp-offload.c >> @@ -626,7 +626,8 @@ oacc_parse_default_dims (const

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-26 Thread Tom de Vries
On 03/02/2018 08:18 PM, Cesar Philippidis wrote: diff --git a/gcc/omp-offload.c b/gcc/omp-offload.c index ba3f4317f4e..f15ce6b8f8d 100644 --- a/gcc/omp-offload.c +++ b/gcc/omp-offload.c @@ -626,7 +626,8 @@ oacc_parse_default_dims (const char *dims) function. */ static void

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-21 Thread Cesar Philippidis
On 03/21/2018 08:49 AM, Tom de Vries wrote: > On 03/02/2018 08:18 PM, Cesar Philippidis wrote: > >> og7-vl-part4-hooks.diff > >> diff --git a/gcc/config/nvptx/nvptx.c b/gcc/config/nvptx/nvptx.c >> index 5642941c6a3..507c8671704 100644 >> --- a/gcc/config/nvptx/nvptx.c >> +++

Re: [og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-21 Thread Tom de Vries
On 03/02/2018 08:18 PM, Cesar Philippidis wrote: og7-vl-part4-hooks.diff diff --git a/gcc/config/nvptx/nvptx.c b/gcc/config/nvptx/nvptx.c index 5642941c6a3..507c8671704 100644 --- a/gcc/config/nvptx/nvptx.c +++ b/gcc/config/nvptx/nvptx.c @@ -5205,14 +5205,36 @@ nvptx_simt_vf () return

[og7] vector_length extension part 4: target hooks and automatic parallelism

2018-03-02 Thread Cesar Philippidis
The attached patch adjusts the existing goacc validate_dims target hook and introduces a new goacc adjust_parallelism target hook. Now that vector length is no longer hard-coded to 32, there are four different ways to set it: 1) compiler default 2) explicitly via the vector_length clause 3)