Re: [patch,gomp-4_0-branch] misc reduction clause bug fixes

2014-07-14 Thread Cesar Philippidis
On 07/11/2014 02:15 AM, Thomas Schwinge wrote: > Hi Cesar! > > On Thu, 10 Jul 2014 11:43:11 -0700, Cesar Philippidis > wrote: >> This patch addresses two bugs openacc reduction clause bugs. > > Thanks! OK; one question/suggestion, though: > >> --- a/gcc/omp-low.c >> +++ b/gcc/omp-low.c >> @@

Re: [patch,gomp-4_0-branch] misc reduction clause bug fixes

2014-07-11 Thread Thomas Schwinge
Hi Cesar! On Thu, 10 Jul 2014 11:43:11 -0700, Cesar Philippidis wrote: > This patch addresses two bugs openacc reduction clause bugs. Thanks! OK; one question/suggestion, though: > --- a/gcc/omp-low.c > +++ b/gcc/omp-low.c > @@ -9679,11 +9679,23 @@ process_reduction_data (gimple_seq *body, gi

[patch,gomp-4_0-branch] misc reduction clause bug fixes

2014-07-10 Thread Cesar Philippidis
Hi Thomas, This patch addresses two bugs openacc reduction clause bugs. The first bug occurred because I didn't anticipate a GIMPLE_BIND stmt to be passed to process_reduction_data. Turns out, this could happen with the collapse clause. That's because the variables which were declared inside the i