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~
