Re: [PATCH PR69652, Regression]

2016-02-29 Thread Jakub Jelinek
On Mon, Feb 29, 2016 at 05:03:38PM +0300, Yuri Rumyantsev wrote: > 2016-02-29 Yuri Rumyantsev > > PR tree-optimization/69652 > gcc/testsuite/ChangeLog: > * gcc.dg/torture/pr69652.c: Delete test. > * gcc.dg/vect/pr69652.c: New test. Ok, with: /* { dg-additional-options "-mavx -ffast-math" { tar

Re: [PATCH PR69652, Regression]

2016-02-29 Thread Yuri Rumyantsev
Jacub! Here is patch and ChangeLog to move pr69652.c to /vect directory. Is it OK for trunk. Thanks. Yuri. ChangeLog: 2016-02-29 Yuri Rumyantsev PR tree-optimization/69652 gcc/testsuite/ChangeLog: * gcc.dg/torture/pr69652.c: Delete test. * gcc.dg/vect/pr69652.c: New test. 2016-02-29 16:

Re: [PATCH PR69652, Regression]

2016-02-29 Thread Jakub Jelinek
On Mon, Feb 29, 2016 at 05:01:52AM -0800, H.J. Lu wrote: > On Mon, Feb 29, 2016 at 3:53 AM, Yuri Rumyantsev wrote: > > This test simply checks that ICE is not occurred but not any > > vectorization issues. > > Can we remove > > /* { dg-options "-O2 -ffast-math -ftree-vectorize " } */ > > then?

Re: [PATCH PR69652, Regression]

2016-02-29 Thread H.J. Lu
On Mon, Feb 29, 2016 at 3:53 AM, Yuri Rumyantsev wrote: > This test simply checks that ICE is not occurred but not any > vectorization issues. Can we remove /* { dg-options "-O2 -ffast-math -ftree-vectorize " } */ then? H.J. > Best regards. > Yuri. > > 2016-02-28 20:29 GMT+03:00 H.J. Lu : >>

Re: [PATCH PR69652, Regression]

2016-02-29 Thread Yuri Rumyantsev
This test simply checks that ICE is not occurred but not any vectorization issues. Best regards. Yuri. 2016-02-28 20:29 GMT+03:00 H.J. Lu : > On Wed, Feb 10, 2016 at 2:26 AM, Yuri Rumyantsev wrote: >> Thanks Richard for your comments. >> I changes algorithm to remove dead scalar statements as yo

Re: [PATCH PR69652, Regression]

2016-02-28 Thread H.J. Lu
On Wed, Feb 10, 2016 at 2:26 AM, Yuri Rumyantsev wrote: > Thanks Richard for your comments. > I changes algorithm to remove dead scalar statements as you proposed. > > Bootstrap and regression testing did not show any new failures on x86-64. > Is it OK for trunk? > > Changelog: > 2016-02-10 Yuri

Re: [PATCH PR69652, Regression]

2016-02-10 Thread Richard Biener
On Wed, Feb 10, 2016 at 11:26 AM, Yuri Rumyantsev wrote: > Thanks Richard for your comments. > I changes algorithm to remove dead scalar statements as you proposed. > > Bootstrap and regression testing did not show any new failures on x86-64. > Is it OK for trunk? Ok. Thanks, Richard. > Changel

Re: [PATCH PR69652, Regression]

2016-02-10 Thread Yuri Rumyantsev
Thanks Richard for your comments. I changes algorithm to remove dead scalar statements as you proposed. Bootstrap and regression testing did not show any new failures on x86-64. Is it OK for trunk? Changelog: 2016-02-10 Yuri Rumyantsev PR tree-optimization/69652 * tree-vect-loop.c (optimize_m

Re: [PATCH PR69652, Regression]

2016-02-09 Thread Richard Biener
On Fri, Feb 5, 2016 at 3:54 PM, Yuri Rumyantsev wrote: > Hi All, > > Here is updated patch - I came back to move call statements also since > masked loads are presented by internal call. I also assume that for > the following simple loop > for (i = 0; i < n; i++) > if (b1[i]) > a1[i]

Re: [PATCH PR69652, Regression]

2016-02-05 Thread Yuri Rumyantsev
Hi All, Here is updated patch - I came back to move call statements also since masked loads are presented by internal call. I also assume that for the following simple loop for (i = 0; i < n; i++) if (b1[i]) a1[i] = sqrtf(a2[i] * a2[i] + a3[i] * a3[i]); motion must be done for all vec

Re: [PATCH PR69652, Regression]

2016-02-04 Thread Jakub Jelinek
On Thu, Feb 04, 2016 at 05:46:27PM +0300, Yuri Rumyantsev wrote: > Here is a patch that cures the issues with non-correct vuse for scalar > statements during code motion, i.e. if vuse of scalar statement is > vdef of masked store which has been sunk to new basic block, we must > fix it up. The pat