[Bug rtl-optimization/60876] New: 2014-04-17 change to mqchmode.h breaks PowerPC

[Bug rtl-optimization/60876] New: 2014-04-17 change to mqchmode.h breaks PowerPC

[meissner at gcc dot gnu.org]

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

            Bug ID: 60876
           Summary: 2014-04-17 change to mqchmode.h breaks PowerPC
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: rtl-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: meissner at gcc dot gnu.org

The change on 2014-04-17 to machmode.h had a side effect of breaking the
PowerPC builds, due to the fact that the PowerPC code in rs6000.c used integers
in two places where GET_MODE_SIZE was used, and this now causes the following
error:

/home/meissner/fsf-src/trunk/gcc/config/rs6000/rs6000.c: In function ‘void
rs6000_setup_reg_addr_masks()’:
/home/meissner/fsf-src/trunk/gcc/config/rs6000/rs6000.c:2366: error: invalid
conversion from ‘ssize_t’ to ‘machine_mode’
/home/meissner/fsf-src/trunk/gcc/config/rs6000/rs6000.c:2370: error: invalid
conversion from ‘ssize_t’ to ‘machine_mode’
/home/meissner/fsf-src/trunk/gcc/config/rs6000/rs6000.c: In function ‘void
rs6000_init_hard_regno_mode_ok(bool)’:
/home/meissner/fsf-src/trunk/gcc/config/rs6000/rs6000.c:2826: error: invalid
conversion from ‘ssize_t’ to ‘machine_mode’
>From gcc-bugs-return-449295-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Thu Apr 17 21:03:07 2014
Return-Path: <gcc-bugs-return-449295-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 32165 invoked by alias); 17 Apr 2014 21:03:07 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 32128 invoked by uid 48); 17 Apr 2014 21:03:04 -0000
From: "meissner at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug rtl-optimization/60876] 2014-04-17 change to mqchmode.h breaks PowerPC
Date: Thu, 17 Apr 2014 21:03:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: rtl-optimization
X-Bugzilla-Version: 4.9.0
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: meissner at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: meissner at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: priority bug_status cf_reconfirmed_on cc assigned_to everconfirmed
Message-ID: <bug-60876-4-851hqrWwFd@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-60876-4@http.gcc.gnu.org/bugzilla/>
References: <bug-60876-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-04/txt/msg01315.txt.bz2
Content-length: 692

http://gcc.gnu.org/bugzilla/show_bug.cgi?id`876

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2014-04-17
                 CC|                            |dje at gcc dot gnu.org,
                   |                            |jakub at redhat dot com
           Assignee|unassigned at gcc dot gnu.org      |meissner at gcc dot gnu.org
     Ever confirmed|0                           |1

[Bug target/60876] 2014-04-17 change to mqchmode.h breaks PowerPC

[meissner at gcc dot gnu.org]

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

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

I'm testing the patch right now.

[Bug target/60876] 2014-04-17 change to machmode.h breaks PowerPC

[meissner at gcc dot gnu.org]

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

--- Comment #2 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Thu Apr 17 21:50:58 2014
New Revision: 209498

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

    PR target/60876
    * config/rs6000/rs6000.c (rs6000_setup_reg_addr_masks): Make sure
    GET_MODE_SIZE gets passed an enum machine_mode type and not
    integer.
    (rs6000_init_hard_regno_mode_ok): Likewise.


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

[Bug target/60876] 2014-04-17 change to machmode.h breaks PowerPC

[meissner at gcc dot gnu.org]

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

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> ---
Fixed in subversion id 209498.