Re: [PATCH][1/2] Fix PR68553

2015-12-07 Thread Richard Biener
On Fri, 4 Dec 2015, Alan Lawrence wrote: > On 04/12/15 17:46, Ramana Radhakrishnan wrote: > > > > > > On 04/12/15 16:04, Richard Biener wrote: > > > On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence > > > wrote: > > > > On 27/11/15 08:30, Richard Biener wrote: > > > > > > > > > > This is

Re: [PATCH][1/2] Fix PR68553

2015-12-04 Thread Alan Lawrence
On 04/12/15 17:46, Ramana Radhakrishnan wrote: On 04/12/15 16:04, Richard Biener wrote: On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence wrote: On 27/11/15 08:30, Richard Biener wrote: This is part 1 of a fix for PR68533 which shows that some targets cannot can_vec_perm_p on an ide

Re: [PATCH][1/2] Fix PR68553

2015-12-04 Thread Ramana Radhakrishnan
On 04/12/15 16:04, Richard Biener wrote: > On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence > wrote: >> On 27/11/15 08:30, Richard Biener wrote: >>> >>> This is part 1 of a fix for PR68533 which shows that some targets >>> cannot can_vec_perm_p on an identity permutation. I chose to fix

Re: [PATCH][1/2] Fix PR68553

2015-12-04 Thread Richard Biener
On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence wrote: >On 27/11/15 08:30, Richard Biener wrote: >> >> This is part 1 of a fix for PR68533 which shows that some targets >> cannot can_vec_perm_p on an identity permutation. I chose to fix >> this in the vectorizer by detecting the identity

Re: [PATCH][1/2] Fix PR68553

2015-12-04 Thread Alan Lawrence
On 27/11/15 08:30, Richard Biener wrote: This is part 1 of a fix for PR68533 which shows that some targets cannot can_vec_perm_p on an identity permutation. I chose to fix this in the vectorizer by detecting the identity itself but with the current structure of vect_transform_slp_perm_load this

[PATCH][1/2] Fix PR68553

2015-11-27 Thread Richard Biener
This is part 1 of a fix for PR68533 which shows that some targets cannot can_vec_perm_p on an identity permutation. I chose to fix this in the vectorizer by detecting the identity itself but with the current structure of vect_transform_slp_perm_load this is somewhat awkward. Thus the following n