Re: [PATCH v3, rs6000] gcc mainline, add builtin support for vec_float, vec_float2, vec_floate, vec_floate, builtins

2017-06-12 Thread Carl E. Love
GCC Maintainers: I have updated the define_expands for float2_v2di, uns_float2_v2di, unsfloatev2di, floato, unsfloatov2di per the comments from Michael Meissner. I retested the changes on powerpc64le-unknown-linux-gnu (Power 8 LE) only. Please let me know if there are any additional issues th

Re: [PATCH v2, rs6000] gcc mainline, add builtin support for vec_float, vec_float2, vec_floate, vec_floate, builtins

2017-06-12 Thread Carl E. Love
On Mon, 2017-06-12 at 14:09 -0400, Michael Meissner wrote: > > > > > > +(define_insn "vsx_xvcvsxwsp" > > > > + [(set (match_operand:V4SF 0 "vsx_register_operand" "=v") > > > > + (unspec:V4SF[(match_operand:V4SI 1 "vsx_register_operand" "v")] > > > > + UNSPEC_VSX_CVSXWSP))

Re: [PATCH v2, rs6000] gcc mainline, add builtin support for vec_float, vec_float2, vec_floate, vec_floate, builtins

2017-06-09 Thread Carl E. Love
GCC Maintainers: On Fri, 2017-06-09 at 16:05 -0500, Segher Boessenkool wrote: Fixed the various formatting (spaces) issues. Been toying with how to write a space checker for patches. Have to take some time to really think about how to do that > > + > > + /* The vector merge instruction vm

[PATCH, rs6000] gcc mainline, add builtin support for vec_float, vec_float2, vec_floate, vec_floate, builtins

2017-06-09 Thread Carl E. Love
GCC Maintainers: This patch adds support for the various vec_float, vec_float2, vec_floate, vec_floate, builtins. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) and on powerpc64-unknown-linux-gnu (Power 8 BE) with no regressions. Is the patch OK for gcc mainline?

[PATCH, rs6000] PR 80982 gcc.target/powerpc/builtins-3-runnable.c fails starting with its introduction in r248846

2017-06-07 Thread Carl E. Love
GCC Maintainers: The support for vec_doublel () builtin didn't get re-tested on BE before submission. The following buzilla was filed against the builtin support. Bug 80982 - gcc.target/powerpc/builtins-3-runnable.c fails starting with its introduction in r248846. The failure was due to GEN_I

[PATCH, rs6000] Fix vec_mulo and vec_mule builtin implementations

2017-06-06 Thread Carl E. Love
GCC Maintainers: The support for the vec_mulo and vec_mule that was recently submitted has a couple of bugs. Specifically, they were implemented with int/unsigned int args and return int/unsigned int. The return types should have been long long/unsigned long long. Additionally it was noted that

[PATCH v3, rs6000] gcc mainline, add builtin support for vec_doublee, vec_doubleo, vec_doublel builtins

2017-06-01 Thread Carl E. Love
GCC Maintainers: This is version 3 of the patch to add support for the various vec_doublee, vec_doubleo, vec_doublel, vec_doubleh builtin-ins. I have addressed the formatting comments from Segher on version 2. I have run the patch through contrib/check_GNU_style.sh to check for obvious formattin

[PATCH v2, rs6000] gcc mainline, add builtin support for vec_doublee, vec_doubleo, vec_doublel builtins

2017-05-26 Thread Carl E. Love
GCC Maintainers: This is version 2 of the patch to add support for the various vec_doublee, vec_doubleo, vec_doublel, vec_doubleh builtin-ins. I have addressed the various comments from Segher. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the pa

[PATCH, rs6000] gcc mainline, add builtin support for vec_doublee, vec_doubleo, vec_doublel builtins

2017-05-17 Thread Carl E. Love
GCC Maintainers: This patch adds support for the various vec_doublee, vec_doubleo, vec_doublel, vec_doubleh builtin-ins. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the patch OK for gcc mainline? Carl Love ---

[PATCH, rs6000] gcc mainline, add builtin support for vec_bperm(), vec_mule() and vec_mulo and vec_sldw() builtins

2017-05-15 Thread Carl E. Love
GCC Maintainers: This patch adds support for the various vec_bperm(), vec_mule() and vec_mulo and vec_sldw() builtins. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the patch OK for gcc mainline? Carl Love -

[PATCH, rs6000] gcc mainline, add builtin support for vec_popcnt() and vec_slo()

2017-05-11 Thread Carl E. Love
GCC Maintainers: This patch adds support for the various vec_popcnt() and vec_slo() builtins. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the patch OK for gcc mainline? Carl Love -

[PATCH, rs6000] gcc mainline, add builtin support for vec_neg()

2017-05-05 Thread Carl E. Love
GCC Maintainers: This patch adds support for the various vec_neg() builtins. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the patch OK for gcc mainline? Carl Love --

[PATCH, rs6000] gcc 6 back port of xvcvsxdsp and xvcvuxdsp RTL fixes

2017-02-17 Thread Carl E. Love
GCC Maintainers: Here is the GCC 6 branch back port of the mainline fixes for the RTL definitions for xvcvsxdsp and xvcvuxdsp instructions, commit r245460 on 2017-02-14. The GCC 5 backport of these fixes have already been approved and committed. The RTL defined the instructions with a V2DF argu

[PATCH, rs6000] gcc 5 back port of xvcvsxdsp and xvcvuxdsp RTL fixes

2017-02-16 Thread Carl E. Love
GCC Maintainers: I have backported the mainline fixes for the RTL definitions for xvcvsxdsp and xvcvuxdsp instructions, commit r245460 on 2017-02-14 to the GCC 5 branch. The RTL defined the instructions with a V2DF argument and returning V4SI. They should take a V2DI argument and return a V4SF ba

[PATCH, rs6000] Fix RTL definitions of the xvcvsxdsp and xvcvuxdsp instructions

2017-02-14 Thread Carl E. Love
GCC Maintainers: The following patch addresses errors in the RTL define_insn statements for the xvcvsxdsp and xvcvuxdsp instructions. The RTL defined the instructions with a V2DF argument and returning V4SI. They should take a V2DI argument and return a V4SF based on the Power ISA document. Ad

[PATCH, rs6000] gcc 5 back port of vec_packs and vec_vgbbd builtin table entry fixes

2017-02-03 Thread Carl E. Love
GCC Maintainers: The GCC 6 branch for the back port of the following two mainline commits was approved and committed to GCC 6. The following patch is the back port for the two commits for the GCC 5 branch. The only difference relative to the GCC 6 patch is the line numbers. mainline commits. co

[PATCH, rs6000] gcc 6 back port of vec_packs and vec_vgbbd builtin table entry fixes

2017-02-03 Thread Carl E. Love
GCC Maintainers: I have squashed the following two patches from mainline commit r244943 Remove bogus entries for the P8V_BUILTIN_VEC_VGBBD built-ins commit r244904 Fix order of entries for ALTIVEC_BUILTIN_VEC_PACKS and P8V_BUILTIN_VEC_VGBBD. and back ported

[PATCH,rs6000] Remove invalid P8V_BUILTIN_VEC_VGBBD entries

2017-01-25 Thread Carl E. Love
GCC Maintainers: After further discussion of the two P8V_BUILTIN_VGBBD built-ins that do not take any arguments, it was determined they should just be removed as they are not valid. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the patch OK for t

Re: [PATCH, rs6000] Fix for entries in table of overloaded built-in functions

2017-01-25 Thread Carl E. Love
Bill: > > > > I don't see any tests for the two built-in entries in rs6000-c.c which the > > patch moves, i.e. > > > > { P8V_BUILTIN_VEC_VGBBD, P8V_BUILTIN_VGBBD, > > > > RS6000_BT

Re: [PATCH, rs6000] Fix for entries in table of overloaded built-in functions

2017-01-24 Thread Carl E. Love
On Tue, 2017-01-24 at 11:08 -0600, Segher Boessenkool wrote: > On Tue, Jan 24, 2017 at 08:28:37AM -0800, Carl E. Love wrote: > > The following patch fixes an issue with the entries in the table of > > built-in functions. All of the entries for a given built-in, must occur > &

[PATCH, rs6000] Fix for entries in table of overloaded built-in functions

2017-01-24 Thread Carl E. Love
GCC maintainers: The following patch fixes an issue with the entries in the table of built-in functions. All of the entries for a given built-in, must occur in the table as a single block of entries. Otherwise the code that searches the table for a given built-in definition will stop looking onc

[PATCH, rs6000] Fix for test_ne_int() built-in test case.

2017-01-13 Thread Carl E. Love
GCC maintainers: The following patch fixes an issue with the builtin test test_ne_long(). The issue is the arguments need to be "bool long long" not "bool long" to get the correct 64-bit value when running in 32-bit mode. The patch has been tested on: powerpc64-unknown-linux-gnu (Power 7 64-bit,

Re: [PATCH, rs6000] Add vec_nabs builtin support

2017-01-13 Thread Carl E. Love
Segar: The issues you pointed out below have been addressed in the following updated patch. Please let me know if the changes are acceptable. Thanks for your help and feedback. Carl Love > rs6000-c.c > > > vector signed char vec_nabs (vector signed char) > > vecto

Re: [PATCH, rs6000] Add vec_nabs builtin support

2017-01-09 Thread Carl E. Love
Oops, accidentally hit send. Was trying to insert file. On Mon, 2017-01-09 at 09:58 -0800, Carl E. Love wrote: > GCC maintainers: > > The following patch adds two more built-ins that are missing. > Specifically: > > vector signed char vec_nabs (vector signed ch

[PATCH, rs6000] Add vec_nabs builtin support

2017-01-09 Thread Carl E. Love
GCC maintainers: The following patch adds two more built-ins that are missing. Specifically: vector signed char vec_nabs (vector signed char) vector signed short vec_nabs (vector signed short) vector signed int vec_nabs (vector signed int) vector signed long long v

Re: [PATCH, rs6000] Add support for vec_pack and vec_sld built-ins

2016-12-13 Thread Carl E. Love
On Thu, 2016-12-08 at 21:03 -0600, Segher Boessenkool wrote: > On Thu, Dec 08, 2016 at 08:28:22AM -0800, Carl E. Love wrote: > > The following patch adds two more built-ins that are missing. > > Specifically: > > > > vector floatvec_packvec

[PATCH, rs6000] Add support for vec_pack and vec_sld built-ins

2016-12-08 Thread Carl E. Love
GCC maintainers: The following patch adds two more built-ins that are missing. Specifically: vector floatvec_packvector doublevector double vector double vec_sld vector doublevector double The patch has been boot strapped and tested on powerpc64le-un

Re: [PATCH v3, rs6000] Add built-in support for vector compare listed in the ABI

2016-12-07 Thread Carl E. Love
On Tue, 2016-12-06 at 15:59 -0600, Segher Boessenkool wrote: > On Tue, Dec 06, 2016 at 01:23:24PM -0800, Carl E. Love wrote: > > Per the feedback I received on the first and second versions of the > > patch I have created the following patch to modify the original commit > > f

[PATCH v3, rs6000] Add built-in support for vector compare listed in the ABI

2016-12-06 Thread Carl E. Love
GCC maintainers: Per the feedback I received on the first and second versions of the patch I have created the following patch to modify the original commit for the vector compare built-in support. Specifically, the tests were broken into files for the processors that supports them. This allows f

[PATCH v2, rs6000] Add built-in support for vector compare

2016-11-21 Thread Carl E. Love
Segher: I realized over the weekend that I forgot to update the built-in documentation file, doc/extend.texi. I have updated the patch with these additions and fixed the issues you mentioned before. The Power ABI document lists a number of built-ins that it is supposed to support. There are sti

[PATCH, rs6000] Add built-in support for vector compare listed in the ABI

2016-11-17 Thread Carl E. Love
GCC maintainers: The Power ABI document lists a number of built-ins that it is supposed to support. There are still some missing. This patch adds the built-in support for the following built-ins: vector bool char vec_cmpeq vector bool char vector bool char vector bool int vec_c

Re: MAINTAINERS update

2016-10-31 Thread Carl E. Love
On Sat, 2016-10-29 at 12:10 -0500, Segher Boessenkool wrote: Hi Carl, > > On Fri, Oct 28, 2016 at 08:11:11AM -0700, Carl E. Love wrote: > > Manuel López-Ibáñez > > Martin v. Löwis > > H.J. Lu > > +Carl Love

Re: MAINTAINERS update

2016-10-28 Thread Carl E. Love
Hi, I added myself to the MAINTAINERS file (Write After Approval) on 10/27/2016. The commit was r241636. Sorry, forgot the patch the first time. Carl Love Index: ChangeLog === --- ChangeLog (revision 241636) +++ Cha

MAINTAINERS update

2016-10-28 Thread Carl E. Love
Hi, I added myself to the MAINTAINERS file (Write After Approval) on 10/27/2016. The commit was r241636. Carl Love