Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-08 Thread Michael Meissner
On Thu, Feb 08, 2018 at 06:10:31PM -0500, Hans-Peter Nilsson wrote: > On Wed, 7 Feb 2018, Segher Boessenkool wrote: > > Hi Mike, > > > > On Tue, Feb 06, 2018 at 04:34:08PM -0500, Michael Meissner wrote: > > > Here is the patch reworked. It bootstraps on both little/big endian > > > power8, > > >

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-08 Thread Hans-Peter Nilsson
On Wed, 7 Feb 2018, Segher Boessenkool wrote: > Hi Mike, > > On Tue, Feb 06, 2018 at 04:34:08PM -0500, Michael Meissner wrote: > > Here is the patch reworked. It bootstraps on both little/big endian power8, > > and all of the tests run. Can I install this into trunk now, and into GCC 7 > > after

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-07 Thread Segher Boessenkool
Hi Mike, On Tue, Feb 06, 2018 at 04:34:08PM -0500, Michael Meissner wrote: > Here is the patch reworked. It bootstraps on both little/big endian power8, > and all of the tests run. Can I install this into trunk now, and into GCC 7 > after a soak period (along with the previous patch)? > +;; If

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-06 Thread Michael Meissner
On Tue, Feb 06, 2018 at 11:15:21AM -0600, Segher Boessenkool wrote: > Hi! > > On Mon, Feb 05, 2018 at 05:57:51PM -0500, Michael Meissner wrote: > > > We have > > > > > > (define_code_attr su [(sign_extend "s") > > > (zero_extend "u") > > >

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-06 Thread Michael Meissner
On Tue, Feb 06, 2018 at 11:15:21AM -0600, Segher Boessenkool wrote: > Hi! > > On Mon, Feb 05, 2018 at 05:57:51PM -0500, Michael Meissner wrote: > > > We have > > > > > > (define_code_attr su [(sign_extend "s") > > > (zero_extend "u") > > >

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-06 Thread Segher Boessenkool
Hi! On Mon, Feb 05, 2018 at 05:57:51PM -0500, Michael Meissner wrote: > > We have > > > > (define_code_attr su [(sign_extend "s") > > (zero_extend "u") > > (fix "s") > > (unsigned_fix "s") > >

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-05 Thread Michael Meissner
On Mon, Feb 05, 2018 at 08:01:32AM -0600, Segher Boessenkool wrote: > On Mon, Feb 05, 2018 at 07:54:58AM -0500, Michael Meissner wrote: > > On Mon, Feb 05, 2018 at 05:57:25AM -0600, Segher Boessenkool wrote: > > > On Thu, Feb 01, 2018 at 02:31:17PM -0500, Michael Meissner wrote: > > > > This patch

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-05 Thread Segher Boessenkool
On Mon, Feb 05, 2018 at 07:54:58AM -0500, Michael Meissner wrote: > On Mon, Feb 05, 2018 at 05:57:25AM -0600, Segher Boessenkool wrote: > > On Thu, Feb 01, 2018 at 02:31:17PM -0500, Michael Meissner wrote: > > > This patch fixes the optimization regression that occurred on GCC 7 where > > >

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-05 Thread Michael Meissner
On Mon, Feb 05, 2018 at 05:57:25AM -0600, Segher Boessenkool wrote: > On Thu, Feb 01, 2018 at 02:31:17PM -0500, Michael Meissner wrote: > > This patch fixes the optimization regression that occurred on GCC 7 where > > conversions from the various floating point types to small integers would at > >

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-05 Thread Segher Boessenkool
On Thu, Feb 01, 2018 at 02:31:17PM -0500, Michael Meissner wrote: > This patch fixes the optimization regression that occurred on GCC 7 where > conversions from the various floating point types to small integers would at > times generate a store and a load. [ snip big explanation; thanks for