Re: [nvptx] vector length patch series

2019-01-08 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0016-nvptx-Add-vector_length-128-testcases.patch > --- /dev/null > > +++ b/libgomp/testsuite/libgomp.oacc-c-c++-common/vred2d-128.c >

Re: [nvptx] vector length patch series

2019-01-05 Thread Tom de Vries
On 22-12-18 03:13, Tom de Vries wrote: > If you have a test-case where this is indeed failing without the > proposed hook implementation, then please try to remove the hardcoding > of vector_length > 32 from the test-source and instead set it using > -fopenacc-dim. AFAIU, the proposed hook does

Re: [nvptx] vector length patch series

2019-01-03 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0016-nvptx-Add-vector_length-128-testcases.patch > * testsuite/libgomp.oacc-c-c++-common/vred2d-128.c: New test. > > * testsuite/libgomp.oacc-fortran/gemm.f90: New test. >

Re: [nvptx] vector length patch series

2019-01-03 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0012-nvptx-Add-axis_dim.patch > diff --git a/gcc/config/nvptx/nvptx.c b/gcc/config/nvptx/nvptx.c > index 74a0d4b04d9..02ecf12bd84 100644 > --- a/gcc/config/nvptx/nvptx.c > +++ b/gcc/config/nvptx/nvptx.c > @@ -2885,6 +2885,23 @@ struct offload_attrs >

Re: [nvptx] vector length patch series

2019-01-03 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0007-nvptx-consolidate-offloaded-function-attributes-into.patch > diff --git a/gcc/config/nvptx/nvptx.c b/gcc/config/nvptx/nvptx.c > index a3169febbb4..dcfa57d10ca 100644 > --- a/gcc/config/nvptx/nvptx.c > +++ b/gcc/config/nvptx/nvptx.c > @@ -2872,6

Re: [nvptx] vector length patch series -- openacc parts

2019-01-03 Thread Tom de Vries
On 18-12-18 00:52, Tom de Vries wrote: > On 14-12-18 20:58, Tom de Vries wrote: > >> 0003-openacc-Add-target-hook-TARGET_GOACC_ADJUST_PARALLEL.patch > Hi Thomas, ping. OK for trunk? Thanks, - Tom

Re: [nvptx] vector length patch series

2018-12-21 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0003-openacc-Add-target-hook-TARGET_GOACC_ADJUST_PARALLEL.patch > 0017-nvptx-Enable-large-vectors.patch 1. If I void nvptx_adjust_parallelism like this: ... static unsigned nvptx_adjust_parallelism (unsigned inner_mask, unsigned outer_mask) { return

[nvptx] vector length patch series -- openacc parts

2018-12-17 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0003-openacc-Add-target-hook-TARGET_GOACC_ADJUST_PARALLEL.patch > 0017-nvptx-Enable-large-vectors.patch > 0023-nvptx-Force-vl32-if-calling-vector-partitionable-rou.patch Thomas, these patches are openacc (0003) or have openacc components (0017, 0023).

Re: [nvptx] vector length patch series

2018-12-17 Thread Tom de Vries
On 14-12-18 20:58, Tom de Vries wrote: > 0009-nvptx-Fix-whitespace-in-nvptx_single-and-nvptx_neute.patch Committed (Could have been committed as obvious). Thanks, - Tom [nvptx] Fix whitespace in nvptx_single and nvptx_neuter_pars Fix whitespace in nvptx_single and nvptx_neuter_pars. Build and

Re: [nvptx] vector length patch series

2018-12-14 Thread Tom de Vries
On 29-10-18 20:28, Cesar Philippidis wrote: > On 10/5/18 23:22, Tom de Vries wrote: >> On 9/18/18 10:04 PM, Cesar Philippidis wrote: >>> 591973d3c3a [nvptx] use user-defined vectors when possible >> >> If I drop this patch, I get the same test results. Can you find a >> testcase for which this

Re: [nvptx] vector length patch series

2018-10-29 Thread Cesar Philippidis
On 10/5/18 23:22, Tom de Vries wrote: > On 9/18/18 10:04 PM, Cesar Philippidis wrote: >> 591973d3c3a [nvptx] use user-defined vectors when possible > > If I drop this patch, I get the same test results. Can you find a > testcase for which this patch has an effect? I just revisited the vector

Re: [nvptx] vector length patch series

2018-10-06 Thread Tom de Vries
On 9/18/18 10:04 PM, Cesar Philippidis wrote: > 591973d3c3a [nvptx] use user-defined vectors when possible If I drop this patch, I get the same test results. Can you find a testcase for which this patch has an effect? Thanks, - Tom

[nvptx] vector length patch series

2018-09-18 Thread Cesar Philippidis
Hi Tom, Here is a link to our nvptx vector length patches on github: https://github.com/cesarjp/gcc/tree/trunk-og8-vl-private Specifically, the code lives in the trunk-og8-vl-private branch. There are a couple of outstanding dependency patches: * Teach gfortran to lower OpenACC routine