[PATCH], PR 78192, Fix PowerPC ISA 3.0 xxextractaw/vextractu{b,h} on little endian

Segher Boessenkool segher@kernel.crashing.org
Thu Nov 3 23:19:00 GMT 2016


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  <meissner@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).



More information about the Gcc-patches mailing list