Re: Don't dump low gimple functions in gimple dump

2015-10-22 Thread Jakub Jelinek
On Thu, Oct 22, 2015 at 02:51:34PM +0200, Tom de Vries wrote: > On 22/10/15 14:27, Jakub Jelinek wrote: > >The state before the patch is: > >1) the omp_fn children created during the pre-SSA ompexp pass are dumped > >first in the *.ssa dump and in all the following ones (these are created > >

Re: Don't dump low gimple functions in gimple dump

2015-10-22 Thread Tom de Vries
On 22/10/15 14:27, Jakub Jelinek wrote: The state before the patch is: 1) the omp_fn children created during the pre-SSA ompexp pass are dumped first in the *.ssa dump and in all the following ones (these are created as low gimple, non-SSA) Hi, I do see those child fns before the ssa d

Re: Don't dump low gimple functions in gimple dump

2015-10-22 Thread Jakub Jelinek
On Thu, Jun 04, 2015 at 05:02:42PM +0200, Tom de Vries wrote: > >So why does add_new_function not do the dumping and to the correct > >place? That is, > >you are dumping things twice here, once in omp expansion and then again when > >the > >new function reaches omp expansion? > > > > Dumping twi

Re: Don't dump low gimple functions in gimple dump

2015-06-08 Thread Richard Biener
On Thu, Jun 4, 2015 at 5:02 PM, Tom de Vries wrote: > On 22/05/15 11:27, Richard Biener wrote: >> >> On Thu, May 21, 2015 at 5:36 PM, Thomas Schwinge >> wrote: >>> >>> Hi! >>> >>> It's just been a year. ;-P >>> >>> In early March, I (hopefully correctly) adapted Tom's patch to apply to >>> then-

Re: Don't dump low gimple functions in gimple dump

2015-06-04 Thread Tom de Vries
On 22/05/15 11:27, Richard Biener wrote: On Thu, May 21, 2015 at 5:36 PM, Thomas Schwinge wrote: Hi! It's just been a year. ;-P In early March, I (hopefully correctly) adapted Tom's patch to apply to then-current GCC trunk sources; posting this here. Is the general approach OK? On Tue, 20

Re: Don't dump low gimple functions in gimple dump

2015-05-22 Thread Richard Biener
On Thu, May 21, 2015 at 5:36 PM, Thomas Schwinge wrote: > Hi! > > It's just been a year. ;-P > > In early March, I (hopefully correctly) adapted Tom's patch to apply to > then-current GCC trunk sources; posting this here. Is the general > approach OK? > > On Tue, 20 May 2014 10:16:45 +0200, Tom

Re: Don't dump low gimple functions in gimple dump

2015-05-21 Thread Thomas Schwinge
Hi! It's just been a year. ;-P In early March, I (hopefully correctly) adapted Tom's patch to apply to then-current GCC trunk sources; posting this here. Is the general approach OK? On Tue, 20 May 2014 10:16:45 +0200, Tom de Vries wrote: > Honza, > > Consider this program: > ... > int > main