On 03/27/2014 06:51 AM, Jakub Jelinek wrote: > Did you mean Jeff's original change, or say: > --- gcc/config/i386/i386.c 2014-03-20 17:41:45.917689676 +0100 > +++ gcc/config/i386/i386.c 2014-03-27 14:47:21.876254288 +0100 > @@ -13925,13 +13925,13 @@ ix86_legitimize_address (rtx x, rtx oldx > if (GET_CODE (XEXP (x, 0)) == MULT) > { > changed = 1; > - XEXP (x, 0) = force_operand (XEXP (x, 0), 0); > + XEXP (x, 0) = copy_addr_to_reg (XEXP (x, 0));
I meant more like this. >> How about doing both? Jakub's simplify_gen_binary change looked like a good >> idea regardless of whatever else happens. Seems a shame not to go with it. > > Agreed. Certainly. r~