Re: [RS6000] power8 internal compiler errors

[David Edelsohn]

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);