Re: [PATCH], Add PowerPC ISA 3.0 vec_vinsert4b and vec_vextract4b built-in functions

2016-12-29 Thread Michael Meissner
Thanks. I fixed the error messages. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797

Re: [PATCH], Add PowerPC ISA 3.0 vec_vinsert4b and vec_vextract4b built-in functions

2016-12-28 Thread Andreas Schwab
On Dez 13 2016, Michael Meissner wrote: > @@ -15766,6 +15766,40 @@ altivec_expand_builtin (tree exp, rtx ta > case VSX_BUILTIN_VEC_EXT_V1TI: >return altivec_expand_vec_ext_builtin (exp, target); > > +case P9V_BUILTIN_VEXTRACT4B: > +case

Re: [PATCH], Add PowerPC ISA 3.0 vec_vinsert4b and vec_vextract4b built-in functions

2016-12-13 Thread Michael Meissner
On Tue, Dec 13, 2016 at 04:40:20PM -0600, Segher Boessenkool wrote: > > + /* If we got invalid arguments bail out before generating bad rtl. > > */ > > If what is invalid? Just remove the first comment? I had copied the code from DST before, and it returns (const_int 0). However, normal

Re: [PATCH], Add PowerPC ISA 3.0 vec_vinsert4b and vec_vextract4b built-in functions

2016-12-13 Thread Segher Boessenkool
Hi Mike, On Tue, Dec 13, 2016 at 01:16:56PM -0500, Michael Meissner wrote: > I have done bootstrap builds on a 64-bit power8 little endian system and a > 32/64-bit power7 big endian system. There were no regressions. Can I check > this into the GCC trunk? Yes, please apply. One remark below.