Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-06-08 Thread Michael Meissner via Gcc-patches
On Mon, Jun 07, 2021 at 03:25:06PM -0500, Segher Boessenkool wrote: > On Tue, May 18, 2021 at 04:26:06PM -0400, Michael Meissner wrote: > > This patch adds the support for the IEEE 128-bit floating point C minimum > > and > > maximum instructions. > > > gcc/ > > 2021-05-18 Michael Meissner >

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-06-07 Thread Segher Boessenkool
On Tue, May 18, 2021 at 04:26:06PM -0400, Michael Meissner wrote: > This patch adds the support for the IEEE 128-bit floating point C minimum and > maximum instructions. > gcc/ > 2021-05-18 Michael Meissner > > * config/rs6000/rs6000.c (rs6000_emit_minmax): Add support for ISA >

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-06-07 Thread Segher Boessenkool
Hi! On Thu, May 20, 2021 at 09:38:49PM -0400, Michael Meissner wrote: > Basically for code generation tests, I see the following cases: > > 1) Test code targetting precisley power8 (or power9, power10), etc. Hopefully > these are rare. -mdejagnu-cpu= works perfectly for this. You may need a

Ping #2: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-06-01 Thread Michael Meissner via Gcc-patches
Ping again. Original patch (Add IEEE 128-bit min/max support on PowerPC): | Date: Tue, 18 May 2021 16:26:06 -0400 | Subject: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC. | Message-ID: <20210518202606.ga14...@ibm-toto.the-meissners.org> | https://gcc.gnu.org/pipermail/gcc-patche

Ping [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-05-24 Thread Michael Meissner via Gcc-patches
Ping patch: | Subject: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC. | Message-ID: <20210518202606.ga14...@ibm-toto.the-meissners.org> -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.ibm.com, phone: +1 (978) 899-4797

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-05-20 Thread Michael Meissner via Gcc-patches
On Thu, May 20, 2021 at 02:25:58PM -0500, will schmidt wrote: > I'd throw the ternary term in there, easier to search for later. > s/?: operations/ternary (?:) operations / Thanks. > So, presumably the float128-minmax-2.c test adds/replaces the power10 > code gen tests that were removed or

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-05-20 Thread will schmidt via Gcc-patches
On Tue, 2021-05-18 at 16:26 -0400, Michael Meissner wrote: > [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC. > Hi, > This patch adds the support for the IEEE 128-bit floating point C minimum and > maximum instructions. The next patch will add the support for using t

[PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-05-18 Thread Michael Meissner via Gcc-patches
[PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC. This patch adds the support for the IEEE 128-bit floating point C minimum and maximum instructions. The next patch will add the support for using the compare and set mask instruction to implement conditional moves. This patch does not try

[PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.

2021-04-15 Thread Michael Meissner via Gcc-patches
[PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC. This patch adds the support for the IEEE 128-bit floating point C minimum and maximum instructions. The next patch will add the support for using the compare and set mask instruction to implement conditional moves. I removed

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-14 Thread Michael Meissner via Gcc-patches
On Wed, Apr 14, 2021 at 02:15:47PM -0500, Segher Boessenkool wrote: > On Wed, Apr 14, 2021 at 03:09:13PM -0400, Michael Meissner wrote: > > On Tue, Apr 13, 2021 at 05:19:12PM -0500, Segher Boessenkool wrote: > > > > * config/rs6000/rs6.h (FLOAT128_MIN_MAX_FPMASK_P): New > > > > macro.

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-14 Thread Segher Boessenkool
On Wed, Apr 14, 2021 at 03:09:13PM -0400, Michael Meissner wrote: > On Tue, Apr 13, 2021 at 05:19:12PM -0500, Segher Boessenkool wrote: > > > * config/rs6000/rs6.h (FLOAT128_MIN_MAX_FPMASK_P): New macro. > > > > As said in the other mail, don't do the macro; just write its expansion > > in

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-14 Thread Michael Meissner via Gcc-patches
On Tue, Apr 13, 2021 at 05:19:12PM -0500, Segher Boessenkool wrote: > Hi! > > On Fri, Apr 09, 2021 at 10:42:50AM -0400, Michael Meissner wrote: > > Since then the patch seems to have gone into a limbo state. > > Patches I cannot immediately handle take time, and if they aren't > pinged, they can

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-13 Thread Segher Boessenkool
Hi! On Fri, Apr 09, 2021 at 10:42:50AM -0400, Michael Meissner wrote: > Since then the patch seems to have gone into a limbo state. Patches I cannot immediately handle take time, and if they aren't pinged, they can fall off the map. So a) ping your patches, once a week for example; and b) write

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-13 Thread Segher Boessenkool
Hi! On Fri, Apr 09, 2021 at 11:54:57AM -0500, will schmidt wrote: > On Fri, 2021-04-09 at 10:42 -0400, Michael Meissner wrote: > > * config/rs6000/rs6000.c (rs6000_emit_minmax): Add support for ISA > > 3.1 IEEE 128-bit floating point xsmaxcqp and xsmincqp instructions. > > I don't see

Re: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-09 Thread will schmidt via Gcc-patches
On Fri, 2021-04-09 at 10:42 -0400, Michael Meissner wrote: > Add IEEE 128-bit min/max support on PowerPC. > > This patch has been posted various times in the past. My memory is the last > time I changed the patch, I addressed the concerns posted at that time. Since > then the patch seems to

[PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC

2021-04-09 Thread Michael Meissner via Gcc-patches
Add IEEE 128-bit min/max support on PowerPC. This patch has been posted various times in the past. My memory is the last time I changed the patch, I addressed the concerns posted at that time. Since then the patch seems to have gone into a limbo state. This patch adds the support for the IEEE