[Bug target/65576] New: ICE in gcc.c-torture/compile/pr33855.c

[Bug target/65576] New: ICE in gcc.c-torture/compile/pr33855.c

[msebor at gcc dot gnu.org]

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

            Bug ID: 65576
           Summary: ICE in gcc.c-torture/compile/pr33855.c
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: msebor at gcc dot gnu.org

Compiling the gcc.c-torture/compile/pr33855.c test with yesterday's trunk
(r221640) on powerpc64 (a POWER7 processor) fails with the ICE below.  Adding
#include <complex.h> doesn't help.  I don't see this ICE on powerpc64le.

$ gcc -O2 -c /src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c: In function
‘foo’:
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c:21:11: warning:
implicit declaration of function ‘cabsl’ [-Wimplicit-function-declaration]
       if (cabsl(vsorc.vqd[indx]) < 1.e-20)
           ^
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c:21:11: warning:
incompatible implicit declaration of built-in function ‘cabsl’
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c:21:11: note:
include ‘<complex.h>’ or provide a declaration of ‘cabsl’
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c:27:1: error:
unrecognizable insn:
 }
 ^
(insn 130 129 44 5 (set (reg:DF 10 10)
        (mem/u/c:DF (symbol_ref/u:DI ("*.LC5") [flags 0x82]) [8  S8 A64]))
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c:20 -1
     (nil))
/src/gcc-trunk/gcc/testsuite/gcc.c-torture/compile/pr33855.c:27:1: internal
compiler error: in extract_insn, at recog.c:2343
0x107a403b _fatal_insn(char const*, rtx_def const*, char const*, int, char
const*)
    /src/gcc-trunk/gcc/rtl-error.c:110
0x107a408f _fatal_insn_not_found(rtx_def const*, char const*, int, char const*)
    /src/gcc-trunk/gcc/rtl-error.c:118
0x10769a27 extract_insn(rtx_insn*)
    /src/gcc-trunk/gcc/recog.c:2343
0x10769ae7 extract_insn_cached(rtx_insn*)
    /src/gcc-trunk/gcc/recog.c:2234
0x1048f677 cleanup_subreg_operands(rtx_insn*)
    /src/gcc-trunk/gcc/final.c:3137
0x107a15d7 reload(rtx_insn*, int)
    /src/gcc-trunk/gcc/reload1.c:1278
0x1062ec17 do_reload
    /src/gcc-trunk/gcc/ira.c:5430
0x1062ec17 execute
    /src/gcc-trunk/gcc/ira.c:5589
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See <http://gcc.gnu.org/bugs.html> for instructions.
>From gcc-bugs-return-481753-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Thu Mar 26 02:45:12 2015
Return-Path: <gcc-bugs-return-481753-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 14733 invoked by alias); 26 Mar 2015 02:45:11 -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 14632 invoked by uid 55); 26 Mar 2015 02:45:06 -0000
From: "jvdelisle at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug fortran/65541] [5 Regression] namelist regression
Date: Thu, 26 Mar 2015 04:12:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: fortran
X-Bugzilla-Version: 5.0
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: jvdelisle at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: P4
X-Bugzilla-Assigned-To: jvdelisle at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-65541-4-jh5L0Qpvpd@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-65541-4@http.gcc.gnu.org/bugzilla/>
References: <bug-65541-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: 2015-03/txt/msg02897.txt.bz2
Content-length: 512

https://gcc.gnu.org/bugzilla/show_bug.cgi?ide541

--- Comment #6 from Jerry DeLisle <jvdelisle at gcc dot gnu.org> ---
Author: jvdelisle
Date: Thu Mar 26 02:44:34 2015
New Revision: 221682

URL: https://gcc.gnu.org/viewcvs?rev"1682&root=gcc&view=rev
Log:
2015-03-25 Jerry DeLisle  <jvdelisle@gcc.gnu.org>

    PR libgfortran/65541
    * io/write.c (nml_write_obj): Convert '+' to '%' before emitting
    object names in namelists.

Modified:
    trunk/libgfortran/ChangeLog
    trunk/libgfortran/io/write.c

[Bug target/65576] ICE in gcc.c-torture/compile/pr33855.c

[msebor at gcc dot gnu.org]

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

--- Comment #1 from Martin Sebor <msebor at gcc dot gnu.org> ---
git bisect pinpointed the following as the commit that introduced the ICE:

commit 2d5ee2b9035db2414ae3535db18550c1d90e52fd
Author: meissner <meissner@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Thu Mar 19 22:37:33 2015 +0000

    [gcc]
    2015-03-19  Michael Meissner  <meissner@linux.vnet.ibm.com>

        PR target/65240
        * config/rs6000/predicates.md (easy_fp_constant): Remove special
        -ffast-math handling that kept non-0 constants live in the RTL
        until reload.  Remove logic testing the number of instructions it
        took to create a constant in a GPR that was never used, due to a
        test for soft-float earlier.
        (memory_fp_constant): Delete, no longer used.

        * config/rs6000/rs6000.md (mov<MODE>_hardfloat): Remove
        alternatives for loading non-0 constants into GPRs for hard
        floating point that is no longer needed due to changes in
        easy_fp_constant.  Add support for loading 0.0 into GPRs.
        (mov<mode>_hardfloat32): Likewise.
        (mov<mode>_hardfloat64): Likewise.
        (mov<mode>_64bit_dm): Likewise.
        (movtd_64bit_nodm): Likewise.
        (pre-reload move FP constant define_split): Delete define_split,
        since it is no longer used.
        (extenddftf2_internal): Remove GHF constraints that are not valid
        for extenddftf2.

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

        PR target/65240
        * gcc/testsuite/g++.dg/pr65240.h: Add tests for PR 65240.
        * gcc/testsuite/g++.dg/pr65240-1.C: Likewise.
        * gcc/testsuite/g++.dg/pr65240-2.C: Likewise.
        * gcc/testsuite/g++.dg/pr65240-3.C: Likewise.
        * gcc/testsuite/g++.dg/pr65240-4.C: Likewise.



    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@221524
138bc75d-0d04-0410-961f-82ee72b054a4

[Bug target/65576] ICE in gcc.c-torture/compile/pr33855.c

[msebor at gcc dot gnu.org]

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

--- Comment #2 from Martin Sebor <msebor at gcc dot gnu.org> ---
*** Bug 65577 has been marked as a duplicate of this bug. ***

[Bug target/65576] ICE in gcc.c-torture/compile/pr33855.c

[msebor at gcc dot gnu.org]

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

Martin Sebor <msebor at gcc dot gnu.org> changed:

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

--- Comment #3 from Martin Sebor <msebor at gcc dot gnu.org> ---
Test still fails today (r221770).

[Bug target/65576] ICE in gcc.c-torture/compile/pr33855.c

[amodra at gmail dot com]

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

Alan Modra <amodra at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-04-01
                 CC|                            |amodra at gmail dot com
     Ever confirmed|0                           |1

--- Comment #4 from Alan Modra <amodra at gmail dot com> ---
Confirmed.  The failure occurs with -mcpu=power6x or lower.

[Bug target/65576] [5 Regression] ICE in gcc.c-torture/compile/pr33855.c

[meissner at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |meissner at gcc dot gnu.org

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

I haven't yet done a full bootstrap build/make check, but I believe this patch
fixes the problem.

The problem was for non-VSX systems, it still allowed 0.0 as the second
argument for the DF->TF expand insn, and reload tried to use it.  I rewrote it
to only require register for pre-VSX systems.

[Bug target/65576] [5 Regression] ICE in gcc.c-torture/compile/pr33855.c

[amodra at gmail dot com]

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

--- Comment #6 from Alan Modra <amodra at gmail dot com> ---
The patch does appear to fix the problem for non-power7, but introduces an ICE
on power7 for pr33855.c.

gcc.c-torture/compile/pr33855.c:27:1: error: unrecognizable insn:
 }
 ^
(insn 124 123 31 5 (set (mem:DF (plus:DI (reg/f:DI 31 31 [orig:161 D.2367 ]
[161])
                (const_int 24 [0x18])) [5 *_13+24 S8 A64])
        (const_double:DF 0.0 [0x0.0p+0]))
/src/gcc-virgin/gcc/testsuite/gcc.c-torture/compile/pr33855.c:20 -1
     (nil))

[Bug target/65576] [5 Regression] ICE in gcc.c-torture/compile/pr33855.c

[amodra at gmail dot com]

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

Alan Modra <amodra at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |https://gcc.gnu.org/ml/gcc-
                   |                            |patches/2015-04/msg00043.ht
                   |                            |ml
           Assignee|meissner at gcc dot gnu.org        |amodra at gmail dot com

--- Comment #7 from Alan Modra <amodra at gmail dot com> ---
Alternate patch posted

[Bug target/65576] [5 Regression] ICE in gcc.c-torture/compile/pr33855.c

[amodra at gcc dot gnu.org]

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

--- Comment #8 from Alan Modra <amodra at gcc dot gnu.org> ---
Author: amodra
Date: Fri Apr  3 23:16:26 2015
New Revision: 221862

URL: https://gcc.gnu.org/viewcvs?rev=221862&root=gcc&view=rev
Log:
    PR target/65576
    PR target/65240
    * config/rs6000/predicates.md (zero_reg_mem_operand): Exclude
    0.0 constant unless TARGET_VSX.
    * config/rs6000/rs6000.md (extenddftf2_internal): Remove last
    alternative.


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

[Bug target/65576] [5 Regression] ICE in gcc.c-torture/compile/pr33855.c

[amodra at gmail dot com]

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

Alan Modra <amodra at gmail dot com> changed:

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

--- Comment #9 from Alan Modra <amodra at gmail dot com> ---
Fixed