[Bug target/81959] PowerPC __float128 optimization fails with integer PRE_INC addresses

[meissner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81959

Michael Meissner  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Michael Meissner  ---
Fixed in trunk and on the GCC 7 branch.  It was not an issue in the GCC 6
branch.

[Bug target/81959] PowerPC __float128 optimization fails with integer PRE_INC addresses

[meissner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81959

--- Comment #3 from Michael Meissner  ---
Author: meissner
Date: Mon Dec 11 18:54:55 2017
New Revision: 255545

URL: https://gcc.gnu.org/viewcvs?rev=255545=gcc=rev
Log:
[gcc]
2017-12-11  Michael Meissner  

Back port from trunk
2017-12-01  Michael Meissner  

PR target/81959
* config/rs6000/rs6000.c (rs6000_address_for_fpconvert): Check for
whether we can allocate pseudos before trying to fix an address.
* config/rs6000/rs6000.md (float_si2_hw): Make sure the
memory address is indexed or indirect.
(floatuns_si2_hw2): Likewise.

[gcct/testsuite]
2017-12-11  Michael Meissner  

Back port from trunk
2017-12-01  Michael Meissner  

PR target/81959
* gcc.target/powerpc/pr81959.c: New test.


Added:
branches/gcc-7-branch/gcc/testsuite/gcc.target/powerpc/pr81959.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/config/rs6000/rs6000.c
branches/gcc-7-branch/gcc/config/rs6000/rs6000.md
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug target/81959] PowerPC __float128 optimization fails with integer PRE_INC addresses

[meissner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81959

--- Comment #2 from Michael Meissner  ---
Author: meissner
Date: Fri Dec  1 23:52:20 2017
New Revision: 255341

URL: https://gcc.gnu.org/viewcvs?rev=255341=gcc=rev
Log:
[gcc]
2017-12-01  Michael Meissner  

PR target/81959
* config/rs6000/rs6000.c (rs6000_address_for_fpconvert): Check for
whether we can allocate pseudos before trying to fix an address.
* config/rs6000/rs6000.md (float_si2_hw): Make sure the
memory address is indexed or indirect.
(floatuns_si2_hw2): Likewise.

[gcct/testsuite]
2017-12-01  Michael Meissner  

PR target/81959
* gcc.target/powerpc/pr81959.c: New test.


Added:
trunk/gcc/testsuite/gcc.target/powerpc/pr81959.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/rs6000/rs6000.c
trunk/gcc/config/rs6000/rs6000.md
trunk/gcc/testsuite/ChangeLog

[Bug target/81959] PowerPC __float128 optimization fails with integer PRE_INC addresses

[meissner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81959

Michael Meissner  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2017-08-24
 Ever confirmed|0   |1

--- Comment #1 from Michael Meissner  ---
Using -mno-update or -mcpu=power8 'fixes' the bug.

It is probably related to the optimizations for PR 79038 that were added on
March 15th.

[Bug target/81959] PowerPC __float128 optimization fails with integer PRE_INC addresses

[meissner at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81959

Michael Meissner  changed:

   What|Removed |Added

 CC||dje at gcc dot gnu.org,
   ||meissner at gcc dot gnu.org,
   ||segher at gcc dot gnu.org,
   ||wschmidt at gcc dot gnu.org
   Assignee|unassigned at gcc dot gnu.org  |meissner at gcc dot 
gnu.org
   Target Milestone|--- |8.0