Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-29 Thread Segher Boessenkool
Hi Bill, On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote: > Here's the fix for the obvious pasto separated out. CCing Richi and > Jakub as I'd appreciate release manager approval to include this in > gcc-6-branch. This fixes some cases where built-in functions are > connected to the

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-28 Thread Segher Boessenkool
On Wed, Apr 27, 2016 at 01:30:54PM -0500, Bill Schmidt wrote: > 2016-04-27 Bill Schmidt > > * config/rs6000/altivec.h: Change definitions of vec_xl and > vec_xst. > * config/rs6000/rs6000-builtin.def (LD_ELEMREV_V2DF): New. > (LD_ELEMREV_V2DI): New. > (LD_ELEMREV_V

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-27 Thread Bill Schmidt
Hi, While looking into documenting the new built-ins, I realized that these instructions provide correct support for the vec_xl and vec_xst built-ins required by the vector API. I've therefore reworked the patch to provide those as overloaded built-ins, rather than the separate per-mode built-ins

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-25 Thread Jakub Jelinek
On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote: > Hi Segher, > > Here's the fix for the obvious pasto separated out. CCing Richi and > Jakub as I'd appreciate release manager approval to include this in > gcc-6-branch. This fixes some cases where built-in functions are > connected

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-25 Thread Bill Schmidt
On Mon, 2016-04-25 at 17:06 +0200, Jakub Jelinek wrote: > On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote: > > Hi Segher, > > > > Here's the fix for the obvious pasto separated out. CCing Richi and > > Jakub as I'd appreciate release manager approval to include this in > > gcc-6-bran

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-25 Thread Segher Boessenkool
On Mon, Apr 25, 2016 at 09:09:03AM -0500, Bill Schmidt wrote: > Here's the fix for the obvious pasto separated out. CCing Richi and > Jakub as I'd appreciate release manager approval to include this in > gcc-6-branch. This fixes some cases where built-in functions are > connected to the wrong exp

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-25 Thread Bill Schmidt
Hi Segher, Here's the fix for the obvious pasto separated out. CCing Richi and Jakub as I'd appreciate release manager approval to include this in gcc-6-branch. This fixes some cases where built-in functions are connected to the wrong expanders because of copy-paste issues. These tend not to be

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-24 Thread Bill Schmidt
On Sun, 2016-04-24 at 15:52 -0500, Segher Boessenkool wrote: > On Sun, Apr 24, 2016 at 02:06:47PM -0500, Bill Schmidt wrote: > > ISA 3.0 adds the lvxh8x, lvxb16x, stvxh8x, and stvxb16x instructions, > > lxvh8x etc. It looks like you only swapped things in this message, > not in the actual patch :

Re: [PATCH, rs6000] Add support for vector element-reversal built-ins

2016-04-24 Thread Segher Boessenkool
On Sun, Apr 24, 2016 at 02:06:47PM -0500, Bill Schmidt wrote: > ISA 3.0 adds the lvxh8x, lvxb16x, stvxh8x, and stvxb16x instructions, lxvh8x etc. It looks like you only swapped things in this message, not in the actual patch :-) > (While working on this patch, I happened to notice that the exist