Re: [PATCH] [ARM] Fix widen-sum pattern in neon.md.

2015-04-19 Thread Xingxing Pan
On 04/15/2015 03:13 AM, Ramana Radhakrishnan wrote: On Thu, Mar 5, 2015 at 1:34 PM, Xingxing Pan wrote: Hi, The expanding of widen-sum pattern always fails. The vectorizer expects the operands to have the same size, while the current implementation of widen-sum pattern dose not conform to this

Re: [PATCH] [ARM] Fix widen-sum pattern in neon.md.

2015-04-14 Thread Ramana Radhakrishnan
On Thu, Mar 5, 2015 at 1:34 PM, Xingxing Pan wrote: > Hi, > > The expanding of widen-sum pattern always fails. The vectorizer expects the > operands to have the same size, while the current implementation of > widen-sum pattern dose not conform to this. > > This patch implements the widen-sum patt

Re: [PATCH] [ARM] Fix widen-sum pattern in neon.md.

2015-03-05 Thread James Greenhalgh
Hi Xingxing, I'm a little confused by your reasons for adding testcases marked XFAIL. On Thu, Mar 05, 2015 at 01:34:25PM +, Xingxing Pan wrote: > +/* { dg-final { scan-tree-dump-times "pattern recognized.*w\\\+" 1 "vect" { > xfail *-*-* } } } */ > +/* { dg-final { cleanup-tree-dump "vect" }

Re: [PATCH] [ARM] Fix widen-sum pattern in neon.md.

2015-03-05 Thread Kyrill Tkachov
On 05/03/15 13:34, Xingxing Pan wrote: Hi, Hi Xingxing, Thanks for improving this! Some comments inline. The expanding of widen-sum pattern always fails. The vectorizer expects the operands to have the same size, while the current implementation of widen-sum pattern dose not conform to this.

[PATCH] [ARM] Fix widen-sum pattern in neon.md.

2015-03-05 Thread Xingxing Pan
Hi, The expanding of widen-sum pattern always fails. The vectorizer expects the operands to have the same size, while the current implementation of widen-sum pattern dose not conform to this. This patch implements the widen-sum pattern with vpadal. Change the vaddw pattern to anonymous. Add