Re: [PATCH] Don't ICE on long long shifts in vectorizable_shift

2011-10-28 Thread Richard Guenther
On Fri, 28 Oct 2011, Jakub Jelinek wrote: > On Fri, Oct 28, 2011 at 10:22:15AM +0200, Richard Guenther wrote: > > Hm, but you are testing vector modes in the path that is supposed to > > handle shifts by a scalar. That looks odd. Also it should be easy > > No, I'm testing it in the path that is

Re: [PATCH] Don't ICE on long long shifts in vectorizable_shift

2011-10-28 Thread Jakub Jelinek
On Fri, Oct 28, 2011 at 10:22:15AM +0200, Richard Guenther wrote: > Hm, but you are testing vector modes in the path that is supposed to > handle shifts by a scalar. That looks odd. Also it should be easy No, I'm testing it in the path that is supposed to handle shifts by a vector. That block s

Re: [PATCH] Don't ICE on long long shifts in vectorizable_shift

2011-10-28 Thread Richard Guenther
On Thu, 27 Oct 2011, Jakub Jelinek wrote: > Hi! > > With the patch I'm going to post momentarily which adds vlshrv{4,2}di and > vashlv{4,2}di patterns for -mavx2 vectorizable_shift ICEs, because the > frontends for long_long_var1 << long_long_var2 emit long_long_var1 << (int) > long_long_var2 >

[PATCH] Don't ICE on long long shifts in vectorizable_shift

2011-10-27 Thread Jakub Jelinek
Hi! With the patch I'm going to post momentarily which adds vlshrv{4,2}di and vashlv{4,2}di patterns for -mavx2 vectorizable_shift ICEs, because the frontends for long_long_var1 << long_long_var2 emit long_long_var1 << (int) long_long_var2 and vectorizable_shift isn't prepared to handle type prom