On Thu, Nov 03, 2016 at 06:22:11PM -0400, Michael Meissner wrote: > Aaron has been running tests on the simulator, and some of the tests fails on > little endian systems. The failing tests do int extracts from a V4SImode > vector. In looking at the code, the vector index was adjusted when the low > level extract instruction was created, and then adjusted again within the > insn. This patch removes the second adjustment. > > I have done bootstraps and make check on both big endian and little endian > power8 systems with no regressions. I have verrified that the tests now pass > in the simulator for both little and big endian targets. Can I install this > patch on the trunk?
Certainly! Thanks for the fix. One nit, see below. Segher > 2016-11-03 Michael Meissner <meiss...@linux.vnet.ibm.com> > > PR target/78192 > * config/rs6000/vsx.md (vsx_extract_<mode>_di): The element number > has already been adjusted for endianess, so don't adjust it any > further. Three "n"s in endianness (another instance in the patch itself).
