Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-12-03 Thread Tom de Vries
On 30/11/15 14:32, Jakub Jelinek wrote: On Mon, Nov 30, 2015 at 02:24:18PM +0100, Richard Biener wrote: OK for stage3 trunk if bootstrap and reg-test succeeds? -|| node->address_taken); +|| (node->address_taken +&& !no

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-12-01 Thread Tom de Vries
On 01/12/15 13:29, Jakub Jelinek wrote: On Tue, Dec 01, 2015 at 01:27:32PM +0100, Christophe Lyon wrote: >I've committed the attached patch as obvious: it adds >dg-require-effective-target fopenmp to these new tests >so that they are skipped e.g. on arm bare-metal targets >(using newlib). > >Not

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-12-01 Thread Jakub Jelinek
On Tue, Dec 01, 2015 at 01:27:32PM +0100, Christophe Lyon wrote: > I've committed the attached patch as obvious: it adds > dg-require-effective-target fopenmp to these new tests > so that they are skipped e.g. on arm bare-metal targets > (using newlib). > > Note that pr46032.c has some failures: >

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-12-01 Thread Christophe Lyon
On 30 November 2015 at 18:55, Tom de Vries wrote: > On 30/11/15 17:48, Jakub Jelinek wrote: >> >> On Mon, Nov 30, 2015 at 05:36:25PM +0100, Tom de Vries wrote: >>> >>> +int >>> +main (void) >>> +{ >>> + unsigned results[nEvents]; >>> + unsigned pData[nEvents]; >>> + unsigned coeff = 2; >>> + >>

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Tom de Vries
On 30/11/15 17:48, Jakub Jelinek wrote: On Mon, Nov 30, 2015 at 05:36:25PM +0100, Tom de Vries wrote: +int +main (void) +{ + unsigned results[nEvents]; + unsigned pData[nEvents]; + unsigned coeff = 2; + + init (&results[0], &pData[0]); + +#pragma omp parallel for + for (int idx = 0; idx < (

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Jakub Jelinek
On Mon, Nov 30, 2015 at 05:36:25PM +0100, Tom de Vries wrote: > +int > +main (void) > +{ > + unsigned results[nEvents]; > + unsigned pData[nEvents]; > + unsigned coeff = 2; > + > + init (&results[0], &pData[0]); > + > +#pragma omp parallel for > + for (int idx = 0; idx < (int)nEvents; idx++) >

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Tom de Vries
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 fixes PR46032. It handles a call: ... __builtin_GOMP_parallel (fn, data, num_threads, flags) ... as: ...

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Jakub Jelinek
On Mon, Nov 30, 2015 at 02:24:18PM +0100, Richard Biener wrote: > > OK for stage3 trunk if bootstrap and reg-test succeeds? > > -|| node->address_taken); > +|| (node->address_taken > +&& !node->parallelized_function)); >

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Richard Biener
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 fixes PR46032. > > > > > > It handles a call: > > > ... > > >__builtin_GOMP_parallel (fn, data, num_threads, flags) > > > ...

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Tom de Vries
On 30/11/15 10:16, Richard Biener wrote: On Mon, 30 Nov 2015, Tom de Vries wrote: Hi, this patch fixes PR46032. It handles a call: ... __builtin_GOMP_parallel (fn, data, num_threads, flags) ... as: ... fn (data) ... in ipa-pta. This improves ipa-pta alias analysis in the parallelized f

Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Richard Biener
On Mon, 30 Nov 2015, Tom de Vries wrote: > Hi, > > this patch fixes PR46032. > > It handles a call: > ... > __builtin_GOMP_parallel (fn, data, num_threads, flags) > ... > as: > ... > fn (data) > ... > in ipa-pta. > > This improves ipa-pta alias analysis in the parallelized function fn, and

[PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta

2015-11-30 Thread Tom de Vries
Hi, this patch fixes PR46032. It handles a call: ... __builtin_GOMP_parallel (fn, data, num_threads, flags) ... as: ... fn (data) ... in ipa-pta. This improves ipa-pta alias analysis in the parallelized function fn, and allows vectorization in the testcase without a runtime alias test. B