Re: [PATCH 3/4] PowerPC: Add power10 xsmaxcqp/xsmincqp support

2020-09-10 Thread Segher Boessenkool
Hi! On Wed, Aug 26, 2020 at 10:45:26PM -0400, Michael Meissner wrote: > * config/rs6000/rs6000.md (FSCALAR): New mode iterator for floating > point scalars. We have the long-established SFDF for a very similar thing. So maybe just call this SFDFTF or SDTF or something?

Re: [PATCH 3/4] PowerPC: Add power10 xsmaxcqp/xsmincqp support

2020-09-10 Thread Segher Boessenkool
On Fri, Aug 28, 2020 at 12:09:48AM -0400, Michael Meissner wrote: > On Thu, Aug 27, 2020 at 03:47:19PM -0500, will schmidt wrote: > > > (Fm): New mode attribute for floating point scalars. > > > > Mixed feels on mixed case, but I defer. :-) > > It is similar to other mode attributes (Ff, Fv)

Re: [PATCH 3/4] PowerPC: Add power10 xsmaxcqp/xsmincqp support

2020-08-27 Thread Michael Meissner via Gcc-patches
On Thu, Aug 27, 2020 at 03:47:19PM -0500, will schmidt wrote: > > (Fm): New mode attribute for floating point scalars. > > Mixed feels on mixed case, but I defer. :-) It is similar to other mode attributes (Ff, Fv) used for setting constraints based on the mode. -- Michael Meissner, IBM

Re: [PATCH 3/4] PowerPC: Add power10 xsmaxcqp/xsmincqp support

2020-08-27 Thread will schmidt via Gcc-patches
On Wed, 2020-08-26 at 22:45 -0400, Michael Meissner via Gcc-patches wrote: > PowerPC: Add power10 xsmaxcqp/xsmincqp support. > > This patch adds support for the ISA 3.1 (power10) IEEE 128-bit "C" minimum and > maximum functions. Because of the NaN differences, the built-in functions > will >

[PATCH 3/4] PowerPC: Add power10 xsmaxcqp/xsmincqp support

2020-08-26 Thread Michael Meissner via Gcc-patches
PowerPC: Add power10 xsmaxcqp/xsmincqp support. This patch adds support for the ISA 3.1 (power10) IEEE 128-bit "C" minimum and maximum functions. Because of the NaN differences, the built-in functions will only generate these instructions if -ffast-math is used until the conditional move support