Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-28 Thread Andreas Schwab
Causes an ICE on ia64. spawn /usr/local/gcc/gcc-20131128/Build/gcc/xgcc -B/usr/local/gcc/gcc-20131128/Build/gcc/ /usr/local/gcc/gcc-20131128/gcc/testsuite/gcc.dg/vect/vect-simd-clone-1.c -fno-diagnostics-show-caret -fdiagnostics-color=never -flto -ffat-lto-objects -ftree-vectorize -fno-vect-co

Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-27 Thread Richard Biener
On Mon, 25 Nov 2013, Jakub Jelinek wrote: > On Fri, Nov 22, 2013 at 11:08:41AM +0100, Richard Biener wrote: > > The expr.c hunk is also ok independently of the patch. > > This is committed now. > > > Ah, here is the stuff moved from. I suppose the IPA param re-org > > is ok for trunk separately

Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-25 Thread Jan Hubicka
> On Mon, Nov 25, 2013 at 07:48:34PM +0100, Jan Hubicka wrote: > > > isn't exactly enabled by default ;) > > > > OK :)) > > > Anyway, all the pass needs is bodies of functions with "omp declare simd" > > > attribute which will be defined in the current partition, for functions > > > defined in oth

Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-25 Thread Jakub Jelinek
On Mon, Nov 25, 2013 at 07:48:34PM +0100, Jan Hubicka wrote: > > isn't exactly enabled by default ;) > > OK :)) > > Anyway, all the pass needs is bodies of functions with "omp declare simd" > > attribute which will be defined in the current partition, for functions > > defined in other partitions

Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-25 Thread Jan Hubicka
> On Mon, Nov 25, 2013 at 06:15:50PM +0100, Jan Hubicka wrote: > > > > What's the reason you cannot defer SIMD cloning to LTRANS stage > > > > as simple IPA pass next to IPA-PTA? > > > > > > Ok, deferring till after IPA-PTA was easy, just small ipa-cp.c changes > > > (look at the attribute rather

Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-25 Thread Jakub Jelinek
On Mon, Nov 25, 2013 at 06:15:50PM +0100, Jan Hubicka wrote: > > > What's the reason you cannot defer SIMD cloning to LTRANS stage > > > as simple IPA pass next to IPA-PTA? > > > > Ok, deferring till after IPA-PTA was easy, just small ipa-cp.c changes > > (look at the attribute rather than simd*cl

Re: [PATCH] OpenMP #pragma omp declare simd support (take 2)

2013-11-25 Thread Jan Hubicka
> > What's the reason you cannot defer SIMD cloning to LTRANS stage > > as simple IPA pass next to IPA-PTA? > > Ok, deferring till after IPA-PTA was easy, just small ipa-cp.c changes > (look at the attribute rather than simd*clone* fields), passes.def and > had to tweak ipa_add_new_function which

Re: [PATCH] OpenMP #pragma omp declare simd support

2013-11-22 Thread Martin Jambor
Hi, On Fri, Nov 22, 2013 at 12:19:33PM +0100, Jakub Jelinek wrote: > On Fri, Nov 22, 2013 at 11:08:41AM +0100, Richard Biener wrote: > > > @@ -284,6 +382,12 @@ public: > > >/* Declaration node used to be clone of. */ > > >tree former_clone_of; > > > > > > + /* If this is a SIMD clone, t

Re: [PATCH] OpenMP #pragma omp declare simd support

2013-11-22 Thread Richard Biener
On Fri, 22 Nov 2013, Jakub Jelinek wrote: > On Fri, Nov 22, 2013 at 11:08:41AM +0100, Richard Biener wrote: > > > @@ -284,6 +382,12 @@ public: > > >/* Declaration node used to be clone of. */ > > >tree former_clone_of; > > > > > > + /* If this is a SIMD clone, this points to the SIMD sp

Re: [PATCH] OpenMP #pragma omp declare simd support

2013-11-22 Thread Jakub Jelinek
On Fri, Nov 22, 2013 at 11:08:41AM +0100, Richard Biener wrote: > > @@ -284,6 +382,12 @@ public: > >/* Declaration node used to be clone of. */ > >tree former_clone_of; > > > > + /* If this is a SIMD clone, this points to the SIMD specific > > + information for it. */ > > + struct