Re: scalar vector shift expansion problem on 64-bit

2011-11-01 Thread David Miller
From: David Miller Date: Fri, 28 Oct 2011 01:05:54 -0400 (EDT) > So should expand_vector_broadcast() really provide this invariant to > the vec_init expander, or does the vec_init expander need to tidy > things up with gen_lowpart() etc. calls? Richard I don't know if you had a chance to look in

Re: scalar vector shift expansion problem on 64-bit

2011-10-28 Thread Jakub Jelinek
On Fri, Oct 28, 2011 at 09:07:31AM -0700, Richard Henderson wrote: > I think this is the same problem as Jakub is attacking here: > > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02503.html It has been checked in already. But my patch only deals with the vector << vector case, vector << scalar

Re: scalar vector shift expansion problem on 64-bit

2011-10-28 Thread Richard Henderson
On 10/27/2011 10:05 PM, David Miller wrote: > > I'm getting an ICE on 64-bit sparc for some vector test cases but > I'm not sure where the fix belongs. > > When the compiler expands a vecor shift by scalar into a vector > shift by a vector it uses expand_vector_broadcast(), which has > a comment

scalar vector shift expansion problem on 64-bit

2011-10-27 Thread David Miller
I'm getting an ICE on 64-bit sparc for some vector test cases but I'm not sure where the fix belongs. When the compiler expands a vecor shift by scalar into a vector shift by a vector it uses expand_vector_broadcast(), which has a comment which states: "The mode of OP must be the element mode of