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

[PATCH PR69652, Regression]

2016-02-04 Thread Yuri Rumyantsev
Hi All, 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 patch also fixed almost all remarks pointed out by Jacub. Bootstra