On Thu, Jan 16, 2014 at 11:08 AM, Michael Meissner <meiss...@linux.vnet.ibm.com> wrote: > This patch fixes the problem that was noticed when we attempted to boostrap > the > compiler on a little endian power8 system with the -O3 -mcpu=power8 options. > > During the development of the power8 direct move patches, I put a guard test > in > the 128-bit direct move insns to make the patterns big endian only. Since I > used gen_highpart and gen_lowpart to generate the SUBREGs in the split, the > existing code is safe for little endian. This patch removes the big endian > guard tests. > > This patch has been regression tested on a little endian power8 system, and it > allows the code to compile. I looked at the assembler generated, and it looks > correct. Is this patch ok to apply? > > 2014-01-16 Michael Meissner <meiss...@linux.vnet.ibm.com> > > * config/rs6000/rs6000.md (reload_vsx_from_gprsf): Add little > endian support, remove tests for WORDS_BIG_ENDIAN. > (p8_mfvsrd_3_<mode>): Likewise. > (reload_gpr_from_vsx<mode>): Likewise. > (reload_gpr_from_vsxsf): Likewise. > (p8_mfvsrd_4_disf): Likewise.
Okay. Thanks Mike and Bill for tracking this down and testing. Thanks, David
