On Mon, Mar 20, 2017 at 01:43:37PM -0500, Segher Boessenkool wrote: > > --- gcc/config/rs6000/rs6000.md (revision 246224) > > +++ gcc/config/rs6000/rs6000.md (working copy) > > @@ -6727,7 +6727,7 @@ > > [(set (match_operand:SI 0 "rs6000_nonimmediate_operand" > > "=r, r, r, ?*wI, ?*wH, > > m, ?Z, ?Z, r, r, > > - r, ?*wIwH, ?*wJwK, ?*wK, ?*wJwK, > > + r, ?*wIwH, ?*wJwK, ?*wJwK, ?*wu, > > ?*wJwK, ?*wH, ?*wK, ?*wIwH, ?r, > > r, *c*l, *h, *h") > > The change to alt 13 is obvious enough. Could you explain alt 14 a > bit more though? It used to allow more than it does after this patch; > are those handled by other alternatives?
Okay, so it is handled by alt 18 (and a reload of the second operand) in that case; thanks for looking! The patch is okay (with the changelog fixed). Thanks, Segher
