Re: [PATCH, PR68640] Clear restrict in install_var_field

2015-12-08 Thread Tom de Vries
On 08/12/15 10:41, Tom de Vries wrote: Cliques are not computed in IPA PTA and the above would require IPA. FTR, this is a generic (not openmp-related) example, and AFAIU a more generic form of the issue that is being raised. ... static void __attribute__((noinline, noclone)) bar (int *a, int *

Re: [PATCH, PR68640] Clear restrict in install_var_field

2015-12-08 Thread Tom de Vries
On 08/12/15 10:22, Richard Biener wrote: On Tue, 8 Dec 2015, Jakub Jelinek wrote: On Tue, Dec 08, 2015 at 09:49:49AM +0100, Tom de Vries wrote: diff --git a/gcc/omp-low.c b/gcc/omp-low.c index d1d1e3c..ac4a94d 100644 --- a/gcc/omp-low.c +++ b/gcc/omp-low.c @@ -1389,6 +1389,12 @@ install_var_fi

Re: [PATCH, PR68640] Clear restrict in install_var_field

2015-12-08 Thread Richard Biener
On Tue, 8 Dec 2015, Jakub Jelinek wrote: > On Tue, Dec 08, 2015 at 09:49:49AM +0100, Tom de Vries wrote: > > diff --git a/gcc/omp-low.c b/gcc/omp-low.c > > index d1d1e3c..ac4a94d 100644 > > --- a/gcc/omp-low.c > > +++ b/gcc/omp-low.c > > @@ -1389,6 +1389,12 @@ install_var_field (tree var, bool by_

Re: [PATCH, PR68640] Clear restrict in install_var_field

2015-12-08 Thread Jakub Jelinek
On Tue, Dec 08, 2015 at 09:49:49AM +0100, Tom de Vries wrote: > diff --git a/gcc/omp-low.c b/gcc/omp-low.c > index d1d1e3c..ac4a94d 100644 > --- a/gcc/omp-low.c > +++ b/gcc/omp-low.c > @@ -1389,6 +1389,12 @@ install_var_field (tree var, bool by_ref, int mask, > omp_context *ctx, > || !

Re: [PATCH, PR68640] Clear restrict in install_var_field

2015-12-08 Thread Richard Biener
On Tue, 8 Dec 2015, Tom de Vries wrote: > Hi, > > this patch fixes PR68640. > > Consider this testcase: > ... > int > foo (int *__restrict__ ap) > { > int *bp = ap; > #pragma omp parallel for > for (unsigned int idx = 0; idx < N; idx++) > ap[idx] = bp[idx]; > } > ... > > When declaring