Re: [PATCH] Fix PR80029

2017-04-07 Thread Thomas Schwinge
Hi Cesar! On Wed, 22 Mar 2017 06:40:03 -0700, Cesar Philippidis wrote: > In addition to resolving the memory leak involving OpenACC delare > clauses, this patch also corrects an ICE involving VLA variables as data > clause arguments used in acc declare constructs. More details can be > found her

Re: [PATCH] Fix PR80029

2017-03-22 Thread Jakub Jelinek
On Wed, Mar 22, 2017 at 06:48:09AM -0700, Cesar Philippidis wrote: > On 03/22/2017 06:42 AM, Jakub Jelinek wrote: > > >> 2017-03-22 Cesar Philippidis > >> > >>PR c++/80029 > >> > >>gcc/ > >>* gimplify.c (is_oacc_declared): New function. > >>(oacc_default_clause): Use it to set d

Re: [PATCH] Fix PR80029

2017-03-22 Thread Jakub Jelinek
On Wed, Mar 22, 2017 at 06:40:03AM -0700, Cesar Philippidis wrote: > In addition to resolving the memory leak involving OpenACC delare > clauses, this patch also corrects an ICE involving VLA variables as data > clause arguments used in acc declare constructs. More details can be > found here

Re: [PATCH] Fix PR80029

2017-03-22 Thread Cesar Philippidis
On 03/22/2017 06:42 AM, Jakub Jelinek wrote: >> 2017-03-22 Cesar Philippidis >> >> PR c++/80029 >> >> gcc/ >> * gimplify.c (is_oacc_declared): New function. >> (oacc_default_clause): Use it to set default flags for acc declared >> variables inside parallel regions. >>

[PATCH] Fix PR80029

2017-03-22 Thread Cesar Philippidis
In addition to resolving the memory leak involving OpenACC delare clauses, this patch also corrects an ICE involving VLA variables as data clause arguments used in acc declare constructs. More details can be found here . Is this OK for trun