Re: [PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-08-08 Thread Pat Haugen
On 08/05/2016 08:04 PM, Segher Boessenkool wrote: > On Fri, Aug 05, 2016 at 04:27:36PM -0500, Pat Haugen wrote: >> On 08/02/2016 03:15 PM, Segher Boessenkool wrote: >>> On Tue, Aug 02, 2016 at 03:03:42PM -0500, Pat Haugen wrote: On 07/29/2016 10:47 AM, Kelvin Nilsen wrote: > + "xsxexpdp %

Re: [PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-08-05 Thread Segher Boessenkool
On Fri, Aug 05, 2016 at 04:27:36PM -0500, Pat Haugen wrote: > On 08/02/2016 03:15 PM, Segher Boessenkool wrote: > > On Tue, Aug 02, 2016 at 03:03:42PM -0500, Pat Haugen wrote: > >> On 07/29/2016 10:47 AM, Kelvin Nilsen wrote: > >>> + "xsxexpdp %0,%x1" > >>> + [(set_attr "type" "fp")]) > >> > >> T

Re: [PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-08-05 Thread Pat Haugen
On 08/02/2016 03:15 PM, Segher Boessenkool wrote: > On Tue, Aug 02, 2016 at 03:03:42PM -0500, Pat Haugen wrote: >> On 07/29/2016 10:47 AM, Kelvin Nilsen wrote: >>> + "xsxexpdp %0,%x1" >>> + [(set_attr "type" "fp")]) >> >> Type should be 'integer'. > > It has VSX regs as input, integer is worse t

Re: [PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-08-02 Thread Segher Boessenkool
On Tue, Aug 02, 2016 at 03:03:42PM -0500, Pat Haugen wrote: > On 07/29/2016 10:47 AM, Kelvin Nilsen wrote: > > + "xsxexpdp %0,%x1" > > + [(set_attr "type" "fp")]) > > Type should be 'integer'. It has VSX regs as input, integer is worse than fpsimple here, I think? Or vecsimple, that seems bette

Re: [PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-08-02 Thread Pat Haugen
On 07/29/2016 10:47 AM, Kelvin Nilsen wrote: > + "xsxexpdp %0,%x1" > + [(set_attr "type" "fp")]) Type should be 'integer'. > + "xsxsigdp %0,%x1" > + [(set_attr "type" "fp")]) 'integer' > + "xsiexpdp %x0,%1,%2" > + [(set_attr "type" "fp")]) 'fpsimple' > + "xscmpexpdp %0,%x1,%x2" > + [(

Re: [PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-08-01 Thread Segher Boessenkool
Hi Kelvin, On Fri, Jul 29, 2016 at 09:47:55AM -0600, Kelvin Nilsen wrote: > This patch adds built-in support for the following fourteen new binary > floating point instructions introduced with the Power9 architecture: Some comments, mostly about whitespace: > --- gcc/doc/extend.texi (revi

[PATCH,rs6000] Add built-in function support Power9 binary floating point operations

2016-07-29 Thread Kelvin Nilsen
This patch adds built-in support for the following fourteen new binary floating point instructions introduced with the Power9 architecture: VSX Scalar Extract Exponent Double-Precision VSX Scalar Extract Significand Double-Precision VSX Scalar Insert Exponent Double-Precision VSX Scalar Compare