Re: [PATCH #2], PR target/81959, Fix ++int to _Float128 conversion on power9

2017-12-11 Thread Segher Boessenkool
On Mon, Dec 04, 2017 at 04:31:55PM -0500, Michael Meissner wrote: > On Fri, Dec 01, 2017 at 05:33:39PM -0600, Segher Boessenkool wrote: > > Okay for trunk. Further improvements welcome ;-) Thanks! > > Here is the patch for GCC 7 (the bug shows up in GCC 7). It is slightly > different due to the

Re: [PATCH #2], PR target/81959, Fix ++int to _Float128 conversion on power9

2017-12-04 Thread Michael Meissner
On Fri, Dec 01, 2017 at 05:33:39PM -0600, Segher Boessenkool wrote: > Okay for trunk. Further improvements welcome ;-) Thanks! Here is the patch for GCC 7 (the bug shows up in GCC 7). It is slightly different due to the surrounding lines in rs6000.c being different. Is it ok to apply? There w

Re: [PATCH #2], PR target/81959, Fix ++int to _Float128 conversion on power9

2017-12-01 Thread Michael Meissner
On Fri, Dec 01, 2017 at 05:33:39PM -0600, Segher Boessenkool wrote: > Hi! > > On Thu, Nov 30, 2017 at 04:52:44PM -0500, Michael Meissner wrote: > > No, then it tends to generate worse code if it is done before the first > > split > > pass (because it no longer keeps the address together). I've b

Re: [PATCH #2], PR target/81959, Fix ++int to _Float128 conversion on power9

2017-12-01 Thread Segher Boessenkool
Hi! On Thu, Nov 30, 2017 at 04:52:44PM -0500, Michael Meissner wrote: > No, then it tends to generate worse code if it is done before the first split > pass (because it no longer keeps the address together). I've been thinking > that in general, we should replace these calls with a new predicate

[PATCH #2], PR target/81959, Fix ++int to _Float128 conversion on power9

2017-11-30 Thread Michael Meissner
I submitted the original version of the patch back in August, and then I forgot about it. https://gcc.gnu.org/ml/gcc-patches/2017-08/msg01600.html Hi Mike, On Mon, Aug 28, 2017 at 02:50:02PM -0400, Michael Meissner wrote: > When I added the optimization for loading 32-bit values directly into the