On Mon, Feb 10, 2014 at 5:18 PM, Alan Modra <amo...@gmail.com> wrote:
Shouldn't addr_op2 also be set from find_replacement?
- David
> @@ -16201,7 +16200,7 @@
> || !rs6000_legitimate_offset_address_p (PTImode, addr,
> false, true)))
> {
> - addr_op1 = XEXP (addr, 0);
> + addr_op1 = find_replacement (&XEXP (addr, 0));
> addr_op2 = XEXP (addr, 1);
^^^^^
> if (!legitimate_indirect_address_p (addr_op1, false))
> rs6000_secondary_reload_fail (__LINE__, reg, mem, scratch,
> store_p);
> @@ -16310,7 +16308,7 @@
>
> else if (GET_CODE (addr) == PLUS)
> {
> - addr_op1 = XEXP (addr, 0);
> + addr_op1 = find_replacement (&XEXP (addr, 0));
> addr_op2 = XEXP (addr, 1);
^^^^^
> if (!REG_P (addr_op1))
> rs6000_secondary_reload_fail (__LINE__, reg, mem, scratch,
> store_p);
