[Bug target/64505] New: Powerpc compiler generates insn not found for -m32 -mpowerpc64

[Bug target/64505] New: Powerpc compiler generates insn not found for -m32 -mpowerpc64

[meissner at gcc dot gnu.org]

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

            Bug ID: 64505
           Summary: Powerpc compiler generates insn not found for -m32
                    -mpowerpc64
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: meissner at gcc dot gnu.org
          Reporter: meissner at gcc dot gnu.org
              Host: powerpc64-unknown-linux-gnu
            Target: powerpc64-unknown-linux-gnu
             Build: powerpc64-unknown-linux-gnu

Created attachment 34384
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34384&action=edit
File that shows the problem

Compiling the attached file will generate an insn not found error message if it
is compiled with the following options: -std=c99 -fno-strict-aliasing -Wall
-m32 -mpowerpc64 foo.c -S -O2.

The problem is inside of rs6000_secondary_reload, there is code for 64-bit and
32-bit. The reload insn that is returned is looking for a 64-bit scratch
register, but the combination of switches -m32 -mpowerpc64 it should be
generating a 32-bit scratch register.

[Bug target/64505] Powerpc compiler generates insn not found for -m32 -mpowerpc64

[meissner at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-01-05
     Ever confirmed|0                           |1

[Bug target/64505] Powerpc compiler generates insn not found for -m32 -mpowerpc64

[meissner at gcc dot gnu.org]

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

--- Comment #1 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Tue Jan  6 20:29:54 2015
New Revision: 219264

URL: https://gcc.gnu.org/viewcvs?rev=219264&root=gcc&view=rev
Log:
[gcc]
2015-01-06  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/64505
    * config/rs6000/rs6000.c (rs6000_secondary_reload): Return the
    correct reload handler if -m32 -mpowerpc64 is used.

[gcc/testsuite]
2015-01-06  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/64505
    * gcc.target/powerpc/pr64505.c: New file to test -m32 -mpowerpc64
    fix is correct.


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

[Bug target/64505] Powerpc compiler generates insn not found for -m32 -mpowerpc64

[meissner at gcc dot gnu.org]

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

--- Comment #3 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Tue Jan 13 17:47:29 2015
New Revision: 219547

URL: https://gcc.gnu.org/viewcvs?rev=219547&root=gcc&view=rev
Log:
2015-01-12  Michael Meissner  <meissner@linux.vnet.ibm.com>

    Backport from trunk
    2015-01-06  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/64505
    * config/rs6000/rs6000.c (rs6000_secondary_reload): Return the
    correct reload handler if -m32 -mpowerpc64 is used.

    * testsuite/gcc.target/powerpc/pr64505.c: New file to test -m32
    -mpowerpc64 fix is correct.


Added:
    branches/ibm/gcc-4_8-branch/gcc/testsuite/gcc.target/powerpc/pr64505.c
      - copied unchanged from r219481,
branches/gcc-4_9-branch/gcc/testsuite/gcc.target/powerpc/pr64505.c
Modified:
    branches/ibm/gcc-4_8-branch/gcc/ChangeLog.ibm
    branches/ibm/gcc-4_8-branch/gcc/config/rs6000/rs6000.c

[Bug target/64505] Powerpc compiler generates insn not found for -m32 -mpowerpc64

[segher at gcc dot gnu.org]

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

Segher Boessenkool <segher at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |segher at gcc dot gnu.org

--- Comment #4 from Segher Boessenkool <segher at gcc dot gnu.org> ---
So, fixed?

[Bug target/64505] Powerpc compiler generates insn not found for -m32 -mpowerpc64

[bergner at gcc dot gnu.org]

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

Peter Bergner <bergner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34384|0                           |1
        is obsolete|                            |
                 CC|                            |bergner at gcc dot gnu.org

--- Comment #5 from Peter Bergner <bergner at gcc dot gnu.org> ---
Created attachment 34595
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34595&action=edit
Minimized test case

Here is a minimized test case that triggers the same error.

[Bug target/64505] Powerpc compiler generates insn not found for -m32 -mpowerpc64

[bergner at gcc dot gnu.org]

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

Peter Bergner <bergner at gcc dot gnu.org> changed:

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

--- Comment #6 from Peter Bergner <bergner at gcc dot gnu.org> ---
(In reply to Segher Boessenkool from comment #4)
> So, fixed?

Yes, this is fixed.