Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-11-13 Thread Thomas Schwinge
Hi! On Wed, 12 Nov 2014 16:58:54 +0100, I wrote: > Hi Casar! > > Can you please help me trying to understand the following change, that > you've done a long time ago: > > On Tue, 8 Jul 2014 07:28:24 -0700, Cesar Philippidis > wrote: > > On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > > > > >

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-11-12 Thread Thomas Schwinge
Hi Casar! Can you please help me trying to understand the following change, that you've done a long time ago: On Tue, 8 Jul 2014 07:28:24 -0700, Cesar Philippidis wrote: > On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > > wrote: > >

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-11-11 Thread Thomas Schwinge
Hi! On Tue, 11 Nov 2014 16:03:05 +0100, I wrote: > On Tue, 8 Jul 2014 07:28:24 -0700, Cesar Philippidis > wrote: > > On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > > > > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > > > wrote: > > >> This patch is the first step to enabling paral

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-11-11 Thread Thomas Schwinge
Hi! On Tue, 8 Jul 2014 07:28:24 -0700, Cesar Philippidis wrote: > On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > > wrote: > >> This patch is the first step to enabling parallel reductions in openacc. > I've committed this updated v

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-10-23 Thread Thomas Schwinge
Hi! On Mon, 28 Jul 2014 10:36:03 -0700, Cesar Philippidis wrote: > On 07/28/2014 10:02 AM, Thomas Schwinge wrote: > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > > wrote: > >> This patch is the first step to enabling parallel reductions in openacc. > > > > I think I have found one

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-09-25 Thread Thomas Schwinge
Hi! On Tue, 8 Jul 2014 07:28:24 -0700, Cesar Philippidis wrote: > On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > > wrote: > >> This patch is the first step to enabling parallel reductions in openacc. > --- /dev/null > +++ b/gcc/testsu

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-28 Thread Cesar Philippidis
On 07/28/2014 10:02 AM, Thomas Schwinge wrote: > Hi Cesar! > > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > wrote: >> This patch is the first step to enabling parallel reductions in openacc. > > I think I have found one issue in this code -- but please verify that my > understanding o

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-28 Thread Thomas Schwinge
Hi Cesar! On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis wrote: > This patch is the first step to enabling parallel reductions in openacc. I think I have found one issue in this code -- but please verify that my understanding of reductions is correct. Namely: > --- a/gcc/omp-low.c > +++

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-08 Thread Cesar Philippidis
On 07/08/2014 07:28 AM, Cesar Philippidis wrote: > Thanks for catching those problems! I've committed this updated version > of the patch. I forgot to remove the support for the collapse clause in from the loop construct in the c frontend. I did so upstream, but not internally. I've checked in th

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-08 Thread Cesar Philippidis
On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis > wrote: >> This patch is the first step to enabling parallel reductions in openacc. > > Thanks! > >> As mentioned earlier, this patch isn't complete yet. For starters, parts >> of it depends

Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-07 Thread Thomas Schwinge
Hi Cesar! On Sun, 6 Jul 2014 16:10:56 -0700, Cesar Philippidis wrote: > This patch is the first step to enabling parallel reductions in openacc. Thanks! > As mentioned earlier, this patch isn't complete yet. For starters, parts > of it depends on our internal ptx backend. I've temporarily rema

[patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-07-06 Thread Cesar Philippidis
This patch is the first step to enabling parallel reductions in openacc. The approach I'm using here is a bit different the to the existing method in openmp. For clarity, consider the following example: for (i = 0, i < n; i++) sum += 1; Currently, gcc breaks up the for loop into m partitions, o