[Bug target/59844] New: Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[Bug target/59844] New: Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[meissner at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

            Bug ID: 59844
           Summary: Powerpc64le cannot bootstrap with -O3/-mcpu=power8
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: major
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: meissner at gcc dot gnu.org

Several of the direct move insns (added in ISA 2.07/power8) are guarded by
tests for WORDS_BIG_ENDIAN, which generates errors when the compiler is
bootstrapped with -O3 and -mcpu=power8.  In addition, the direct-move vector
tests will fail.

These guards were added to make sure when the little endian work was done, that
we used the correct word order for little endian as compared to big endian. 
Since gen_highpart/gen_lowpart is used for creating the SUBREG's, it is safe to
remove the big endian guard test.

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[meissner at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

Michael Meissner <meissner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|                            |powerpc64le-linux
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2014-01-16
               Host|                            |powerpc64le-linux
           Assignee|unassigned at gcc dot gnu.org      |meissner at gcc dot gnu.org
     Ever confirmed|0                           |1
              Build|                            |powerpc64le-linux

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[meissner at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

--- Comment #1 from Michael Meissner <meissner at gcc dot gnu.org> ---
Created attachment 31858
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31858&action=edit
Proposed patch to fix the problem

Patch tested by Bill Schmitd, and passes the regression testing.

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[meissner at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

--- Comment #2 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Thu Jan 16 17:08:52 2014
New Revision: 206668

URL: http://gcc.gnu.org/viewcvs?rev=206668&root=gcc&view=rev
Log:
2014-01-16  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/59844
    * config/rs6000/rs6000.md (reload_vsx_from_gprsf): Add little
    endian support, remove tests for WORDS_BIG_ENDIAN.
    (p8_mfvsrd_3_<mode>): Likewise.
    (reload_gpr_from_vsx<mode>): Likewise.
    (reload_gpr_from_vsxsf): Likewise.
    (p8_mfvsrd_4_disf): Likewise.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/rs6000/rs6000.md

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[meissner at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

Michael Meissner <meissner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Michael Meissner <meissner at gcc dot gnu.org> ---
Committed as subversion revision 206668.

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[meissner at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

--- Comment #4 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Thu Jan 16 17:35:41 2014
New Revision: 206670

URL: http://gcc.gnu.org/viewcvs?rev=206670&root=gcc&view=rev
Log:
2014-01-16  Michael Meissner  <meissner@linux.vnet.ibm.com>

    Back port from mainline
    2014-01-16  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/59844
    * config/rs6000/rs6000.md (reload_vsx_from_gprsf): Add little
    endian support, remove tests for WORDS_BIG_ENDIAN.
    (p8_mfvsrd_3_<mode>): Likewise.
    (reload_gpr_from_vsx<mode>): Likewise.
    (reload_gpr_from_vsxsf): Likewise.
    (p8_mfvsrd_4_disf): Likewise.


Modified:
    branches/ibm/gcc-4_8-branch/gcc/ChangeLog.ibm
    branches/ibm/gcc-4_8-branch/gcc/config/rs6000/rs6000.md

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[amodra at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

Alan Modra <amodra at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |anton at samba dot org

--- Comment #5 from Alan Modra <amodra at gmail dot com> ---
*** Bug 59814 has been marked as a duplicate of this bug. ***

[Bug target/59844] Powerpc64le cannot bootstrap with -O3/-mcpu=power8

[wschmidt at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59844

--- Comment #6 from Bill Schmidt <wschmidt at gcc dot gnu.org> ---
Author: wschmidt
Date: Fri Apr  4 14:32:32 2014
New Revision: 209105

URL: http://gcc.gnu.org/viewcvs?rev=209105&root=gcc&view=rev
Log:
[gcc]

2014-04-04  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

    Back port from mainline
    2014-01-16  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/59844
    * config/rs6000/rs6000.md (reload_vsx_from_gprsf): Add little
    endian support, remove tests for WORDS_BIG_ENDIAN.
    (p8_mfvsrd_3_<mode>): Likewise.
    (reload_gpr_from_vsx<mode>): Likewise.
    (reload_gpr_from_vsxsf): Likewise.
    (p8_mfvsrd_4_disf): Likewise.

[gcc/testsuite]

2014-04-04  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

    Backport from mainline
    2013-10-23  Pat Haugen  <pthaugen@us.ibm.com>

    * gcc.target/powerpc/direct-move.h: Fix header for executable tests.


Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog
    branches/gcc-4_8-branch/gcc/config/rs6000/rs6000.md
    branches/gcc-4_8-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_8-branch/gcc/testsuite/gcc.target/powerpc/direct-move.h