Re: [patch, ARM] Fix PR target/48252

2011-05-06 Thread Richard Earnshaw
On Sun, 2011-05-01 at 10:30 +0300, Ira Rosen wrote: > > Ramana Radhakrishnan wrote on 07/04/2011 > 03:16:44 PM: > > > > > On 07/04/11 08:42, Ira Rosen wrote: > > > Hi, > > > > > > This patch makes both outputs of neon_vzip/vuzp/vtrn_internal > > > explicitly dependent on both inputs, preventing

Re: [patch, ARM] Fix PR target/48252

2011-05-03 Thread Richard Guenther
On Tue, 3 May 2011, Ramana Radhakrishnan wrote: > > > > I have no objections to this going into 4.5 and 4.6 since it corrects > > > the implementation of the neon intrinsics but please check with the > > > release managers. > > > > OK to backport to 4.5 and 4.6 - both tested on arm-linux-gnueabi

Re: [patch, ARM] Fix PR target/48252

2011-05-03 Thread Ramana Radhakrishnan
I have no objections to this going into 4.5 and 4.6 since it corrects the implementation of the neon intrinsics but please check with the release managers. OK to backport to 4.5 and 4.6 - both tested on arm-linux-gnueabi? Ok. Please allow 24 hours for an RM (cc'd) to comment since this is

Re: [patch, ARM] Fix PR target/48252

2011-05-01 Thread Ira Rosen
Ramana Radhakrishnan wrote on 07/04/2011 03:16:44 PM: > > On 07/04/11 08:42, Ira Rosen wrote: > > Hi, > > > > This patch makes both outputs of neon_vzip/vuzp/vtrn_internal > > explicitly dependent on both inputs, preventing incorrect > > optimization: > > for > > (a,b)<- vzip (c,d) > > and > >

Re: [patch, ARM] Fix PR target/48252

2011-04-07 Thread Ramana Radhakrishnan
On 07/04/11 08:42, Ira Rosen wrote: Hi, This patch makes both outputs of neon_vzip/vuzp/vtrn_internal explicitly dependent on both inputs, preventing incorrect optimization: for (a,b)<- vzip (c,d) and (e,f)<- vzip (g,d) CSE decides that b==f, since b and f depend only on d. Tested on arm-linux-

[patch, ARM] Fix PR target/48252

2011-04-07 Thread Ira Rosen
Hi, This patch makes both outputs of neon_vzip/vuzp/vtrn_internal explicitly dependent on both inputs, preventing incorrect optimization: for (a,b)<- vzip (c,d) and (e,f) <- vzip (g,d) CSE decides that b==f, since b and f depend only on d. Tested on arm-linux-gnueabi. OK for trunk? OK for 4.6 aft