Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-05-15 Thread Rainer Orth
Jakub Jelinek writes: > On Thu, May 15, 2014 at 02:37:33PM +0200, Rainer Orth wrote: >> > If OpenMP declare simd doesn't work on Solaris/x86 (due to the bogus hw cap >> > stuff), then supposedly vect_simd_clones effective target should fail >> > there. >> >> I don't think it's bogus: it guards

Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-05-15 Thread Jakub Jelinek
On Thu, May 15, 2014 at 02:37:33PM +0200, Rainer Orth wrote: > > If OpenMP declare simd doesn't work on Solaris/x86 (due to the bogus hw cap > > stuff), then supposedly vect_simd_clones effective target should fail there. > > I don't think it's bogus: it guards against a similar kind of problems >

Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-05-15 Thread Rainer Orth
Jakub Jelinek writes: > On Tue, May 13, 2014 at 07:38:52PM +0200, Rainer Orth wrote: >> Rainer Orth writes: >> >> > Jakub Jelinek writes: >> > >> >> 2014-04-18 Jakub Jelinek >> >> >> >> PR tree-optimization/60823 >> >> * omp-low.c (ipa_simd_modify_function_body): Go through >> >> all

Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-05-13 Thread Jakub Jelinek
On Tue, May 13, 2014 at 07:38:52PM +0200, Rainer Orth wrote: > Rainer Orth writes: > > > Jakub Jelinek writes: > > > >> 2014-04-18 Jakub Jelinek > >> > >>PR tree-optimization/60823 > >>* omp-low.c (ipa_simd_modify_function_body): Go through > >>all SSA_NAMEs and for those refering

Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-05-13 Thread Rainer Orth
Rainer Orth writes: > Jakub Jelinek writes: > >> 2014-04-18 Jakub Jelinek >> >> PR tree-optimization/60823 >> * omp-low.c (ipa_simd_modify_function_body): Go through >> all SSA_NAMEs and for those refering to vector arguments >> which are going to be replaced adjust SSA_NA

Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-05-13 Thread Rainer Orth
Jakub Jelinek writes: > 2014-04-18 Jakub Jelinek > > PR tree-optimization/60823 > * omp-low.c (ipa_simd_modify_function_body): Go through > all SSA_NAMEs and for those refering to vector arguments > which are going to be replaced adjust SSA_NAME_VAR and, > if it i

Re: [PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-04-22 Thread Richard Biener
On Fri, 18 Apr 2014, Jakub Jelinek wrote: > Hi! > > This patch fixes the adjustments performed by ipa_simd_modify_function_body > on omp declare simd clones. > Previously we've been trying to replace typically SSA_NAMEs with underlying > PARM_DECLs of the to be replaced arguments with loads/store

[PATCH] Fix omp declare simd cloning (PR tree-optimization/60823)

2014-04-18 Thread Jakub Jelinek
Hi! This patch fixes the adjustments performed by ipa_simd_modify_function_body on omp declare simd clones. Previously we've been trying to replace typically SSA_NAMEs with underlying PARM_DECLs of the to be replaced arguments with loads/stores from/to array refs (that will be hopefully vectorized