Re: [PATCH,rs6000] Replace swap of a loaded vector constant with load of a swapped vector constant

2017-09-19 Thread Bill Schmidt
Hi Kelvin, This is in quite good shape. In addition to Segher's comments, I have a few questions/requests below. > On Sep 15, 2017, at 4:04 PM, Kelvin Nilsen > wrote: > > @@ -385,6 +396,25 @@ const_load_sequence_p (swap_web_entry *insn_entry, > split_const (XVECEXP (tocrel_base, 0, 0)

Re: [PATCH,rs6000] Replace swap of a loaded vector constant with load of a swapped vector constant

2017-09-18 Thread Segher Boessenkool
Hi Kelvin, On Fri, Sep 15, 2017 at 03:04:52PM -0600, Kelvin Nilsen wrote: > On Power8 little endian, two instructions are needed to load from the > natural in-memory representation of a vector into a vector register: a > load followed by a swap. When the vector value to be loaded is a > constant,

[PATCH,rs6000] Replace swap of a loaded vector constant with load of a swapped vector constant

2017-09-15 Thread Kelvin Nilsen
On Power8 little endian, two instructions are needed to load from the natural in-memory representation of a vector into a vector register: a load followed by a swap. When the vector value to be loaded is a constant, more efficient code can be achieved by swapping the representation of the constan