On Tue, Dec 13, 2016 at 10:15:02AM -0500, Michael Meissner wrote: > This patch should address the comments in the last patch. > > I have tested this patch with bootstrap builds and make check regression tests > on a little endian Power8 64-bit system and a big endian Power7 32/64-bit > system with no regressions. Can I check this into the trunk?
> + else if (mode == V8HImode) > + { > + rtx tmp_gpr_si = (GET_CODE (tmp_gpr) == SCRATCH > + ? dest_si > + : gen_rtx_REG (SImode, REGNO (tmp_gpr))); I think you have these the wrong way around? You didn't address the reload_completed on all the splitters yet; is there a reason for it? Okay with these fixed. Segher