Hi Richard, On Tue, Nov 15, 2016 at 10:49:26AM +0000, Richard Sandiford wrote: > simplify_shift_const_1 handles both shifts of scalars by scalars > and shifts of vectors by scalars. For vectors this means that > each element is shifted by the same amount. > > However: > > (a) the two cases weren't always distinguished, so we'd try > things for vectors that only made sense for scalars. > > (b) a lot of the range and bitcount checks were based on the > bitsize or precision of the full shifted operand, rather > than the mode of each element. > > Fixing (b) accidentally exposed more optimisation opportunities, > although that wasn't the point of the patch. > > Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install?
Yes please. Thanks! Segher
