Re: [committed, gomp4] Fix release_dangling_ssa_names

2015-08-05 Thread Richard Biener
On Wed, 5 Aug 2015, Tom de Vries wrote: > On 05/08/15 11:30, Richard Biener wrote: > > On Wed, 5 Aug 2015, Tom de Vries wrote: > > > > > On 05/08/15 09:29, Richard Biener wrote: > > > > > This patch fixes that by making sure we reset the def stmt to NULL. > > > > > This > > > > > means > > > > >

Re: [committed, gomp4] Fix release_dangling_ssa_names

2015-08-05 Thread Tom de Vries
On 05/08/15 11:30, Richard Biener wrote: On Wed, 5 Aug 2015, Tom de Vries wrote: On 05/08/15 09:29, Richard Biener wrote: This patch fixes that by making sure we reset the def stmt to NULL. This means we can simplify release_dangling_ssa_names to just test for NULL def stmts. Not sure if I

Re: [committed, gomp4] Fix release_dangling_ssa_names

2015-08-05 Thread Richard Biener
On Wed, 5 Aug 2015, Tom de Vries wrote: > On 05/08/15 09:29, Richard Biener wrote: > > > This patch fixes that by making sure we reset the def stmt to NULL. This > > > means > > > >we can simplify release_dangling_ssa_names to just test for NULL def > > > stmts. > > Not sure if I understand the pr

Re: [committed, gomp4] Fix release_dangling_ssa_names

2015-08-05 Thread Tom de Vries
On 05/08/15 09:29, Richard Biener wrote: This patch fixes that by making sure we reset the def stmt to NULL. This means >we can simplify release_dangling_ssa_names to just test for NULL def stmts. Not sure if I understand the problem correctly but why are you not simply releasing the SSA name wh

Re: [committed, gomp4] Fix release_dangling_ssa_names

2015-08-05 Thread Richard Biener
On Wed, 5 Aug 2015, Tom de Vries wrote: > [ was: Re: Expand oacc kernels after pass_fre ] > On 04/06/15 18:02, Tom de Vries wrote: > > > Please move this out of the class body. > > > > > > > Fixed and committed (ommitting patch as trivial). > > > > > > +{ > > > > + unsigned res = execu