Hi again,

On Mon, Oct 29, 2018 at 09:39:59AM -0500, Aaron Sawdey wrote:
>       * config/rs6000/rs6000.md (bswapdi2): Force address into register
>       if not in one already.

This isn't very correct, could you rephrase?

> +void
> +rs6000_force_indexed_or_indirect_mem (rtx x)
> +{
> +  rtx addr = XEXP (x, 0);

You can move this line down a block.

> +  machine_mode m = GET_MODE (x);
> +  if (!indexed_or_indirect_operand (x, m))
> +    {

(to here)

> +      addr = force_reg (Pmode, addr);
> +      x = replace_equiv_address_nv (x, addr);
> +    }
> +}


Okay for trunk (if it works ;-) )  Thanks!


Segher

Reply via email to