On Tue, Aug 09, 2016 at 11:53:26PM -0400, Michael Meissner wrote: > Index: gcc/config/rs6000/rs6000.c > =================================================================== > --- gcc/config/rs6000/rs6000.c (revision 239301) > +++ gcc/config/rs6000/rs6000.c (working copy) > @@ -7739,6 +7739,9 @@ mem_operand_ds_form (rtx op, machine_mod > int extra; > rtx addr = XEXP (op, 0); > > + if (!MEM_P (op) || !offsettable_address_p (false, mode, addr)) > + return false;
Either the MEM_P test is superfluous, or it should come before the XEXP above? Maybe the MEM_P part should be an assert? Okay with that fixed (for trunk and 6). Thanks, Segher
