[PATCH][PR target/80083][7 regression] fix power9 vsx-small-integer issue caused by wrong constraints

Segher Boessenkool segher@kernel.crashing.org
Mon Mar 20 20:37:00 GMT 2017


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



More information about the Gcc-patches mailing list