Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2016-01-14 Thread Richard Biener
On Thu, 14 Jan 2016, Thomas Schwinge wrote: > Hi! > > On Fri, 8 Jan 2016 09:16:55 +0100, Richard Biener wrote: > > On Wed, 16 Dec 2015, Tom de Vries wrote: > > > On 10/12/15 14:14, Tom de Vries wrote: > > > > > On Thu, 3 Dec 2015, Tom de Vries wrote: > > > > > > Essentially we have two situation

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2016-01-14 Thread Thomas Schwinge
Hi! On Fri, 8 Jan 2016 09:16:55 +0100, Richard Biener wrote: > On Wed, 16 Dec 2015, Tom de Vries wrote: > > On 10/12/15 14:14, Tom de Vries wrote: > > > > On Thu, 3 Dec 2015, Tom de Vries wrote: > > > > > Essentially we have two situations: > > > > > - in the host compiler, there is no need for t

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2016-01-08 Thread Richard Biener
On Wed, 16 Dec 2015, Tom de Vries wrote: > On 10/12/15 14:14, Tom de Vries wrote: > > [ copy-pasting-with-quote from > > https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00420.html , for some > > reason I didn't get this email ] > > > > > On Thu, 3 Dec 2015, Tom de Vries wrote: > > > > The flag is s

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-16 Thread Tom de Vries
On 10/12/15 14:14, Tom de Vries wrote: [ copy-pasting-with-quote from https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00420.html , for some reason I didn't get this email ] On Thu, 3 Dec 2015, Tom de Vries wrote: The flag is set here in expand_omp_target: ... 12682 /* Prevent IPA from re

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-10 Thread Tom de Vries
[ copy-pasting-with-quote from https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00420.html , for some reason I didn't get this email ] On Thu, 3 Dec 2015, Tom de Vries wrote: The flag is set here in expand_omp_target: ... 12682 /* Prevent IPA from removing child_fn as unreachable,

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-03 Thread Jakub Jelinek
On Thu, Dec 03, 2015 at 12:09:04PM +0100, Tom de Vries wrote: > The flag is set here in expand_omp_target: > ... > 12682 /* Prevent IPA from removing child_fn as unreachable, > since there are no > 12683refs from the parent function to child_fn in offload >

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-03 Thread Richard Biener
On Thu, 3 Dec 2015, Tom de Vries wrote: > On 03/12/15 09:59, Richard Biener wrote: > > On Thu, 3 Dec 2015, Tom de Vries wrote: > > > > > On 03/12/15 01:10, Tom de Vries wrote: > > > > > > > > I've managed to reproduce it. The difference between pass and fail is > > > > whether the compiler is co

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-03 Thread Tom de Vries
On 03/12/15 09:59, Richard Biener wrote: On Thu, 3 Dec 2015, Tom de Vries wrote: On 03/12/15 01:10, Tom de Vries wrote: I've managed to reproduce it. The difference between pass and fail is whether the compiler is configured with or without accelerator. I'll look into it. In the configurat

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-03 Thread Richard Biener
On Thu, 3 Dec 2015, Tom de Vries wrote: > On 03/12/15 01:10, Tom de Vries wrote: > > > > I've managed to reproduce it. The difference between pass and fail is > > whether the compiler is configured with or without accelerator. > > > > I'll look into it. > > In the configuration with accelerator

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-02 Thread Tom de Vries
On 03/12/15 01:10, Tom de Vries wrote: I've managed to reproduce it. The difference between pass and fail is whether the compiler is configured with or without accelerator. I'll look into it. In the configuration with accelerator, the flag node->force_output is on for foo._omp.fn. This cau

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-02 Thread Tom de Vries
On 03/12/15 00:31, Tom de Vries wrote: On 02/12/15 18:58, Thomas Schwinge wrote: Hi! On Tue, 1 Dec 2015 15:25:42 +0100, Tom de Vries wrote: >Handle BUILT_IN_GOACC_PARALLEL in ipa-pta >* c-c++-common/goacc/kernels-alias-ipa-pta-2.c: New test. >* c-c++-common/goacc/kernels-alias-ipa-pta

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-02 Thread Tom de Vries
On 02/12/15 18:58, Thomas Schwinge wrote: Hi! On Tue, 1 Dec 2015 15:25:42 +0100, Tom de Vries wrote: >Handle BUILT_IN_GOACC_PARALLEL in ipa-pta >* c-c++-common/goacc/kernels-alias-ipa-pta-2.c: New test. >* c-c++-common/goacc/kernels-alias-ipa-pta-3.c: New test. >* c-c++-common/goac

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-02 Thread Thomas Schwinge
Hi! On Tue, 1 Dec 2015 15:25:42 +0100, Tom de Vries wrote: > Handle BUILT_IN_GOACC_PARALLEL in ipa-pta > * c-c++-common/goacc/kernels-alias-ipa-pta-2.c: New test. > * c-c++-common/goacc/kernels-alias-ipa-pta-3.c: New test. > * c-c++-common/goacc/kernels-alias-ipa-pta.c: New tes

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-02 Thread Jakub Jelinek
On Wed, Dec 02, 2015 at 12:46:47AM +0100, Tom de Vries wrote: > Does this explanation address your concern? Yeah, for now it is fine I hope. Jakub

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-01 Thread Tom de Vries
On 01/12/15 15:44, Jakub Jelinek wrote: On Tue, Dec 01, 2015 at 03:25:42PM +0100, Tom de Vries wrote: Handle BUILT_IN_GOACC_PARALLEL in ipa-pta 2015-12-01 Tom de Vries * tree-ssa-structalias.c (find_func_aliases_for_builtin_call) (find_func_clobbers, ipa_pta_execute): Handle

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-01 Thread Richard Biener
On Tue, 1 Dec 2015, Tom de Vries wrote: > [ was: Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta ] > > On 30/11/15 17:36, Tom de Vries wrote: > > On 30/11/15 14:24, Richard Biener wrote: > > > On Mon, 30 Nov 2015, Tom de Vries wrote: > > > > > > > On 30/11/15 10:16, Richard Biener

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-01 Thread Jakub Jelinek
On Tue, Dec 01, 2015 at 03:25:42PM +0100, Tom de Vries wrote: > Handle BUILT_IN_GOACC_PARALLEL in ipa-pta > > 2015-12-01 Tom de Vries > > * tree-ssa-structalias.c (find_func_aliases_for_builtin_call) > (find_func_clobbers, ipa_pta_execute): Handle BUILT_IN_GOACC_PARALLEL. Isn't th

[PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-01 Thread Tom de Vries
[ was: Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta ] On 30/11/15 17:36, Tom de Vries wrote: On 30/11/15 14:24, Richard Biener wrote: On Mon, 30 Nov 2015, Tom de Vries wrote: On 30/11/15 10:16, Richard Biener wrote: On Mon, 30 Nov 2015, Tom de Vries wrote: Hi, this patch