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