[Bug target/65138] New: [5 Regression] testsuite ICEs on powerpc64le

[Bug target/65138] New: [5 Regression] testsuite ICEs on powerpc64le

[trippels at gcc dot gnu.org]

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

            Bug ID: 65138
           Summary: [5 Regression] testsuite ICEs on powerpc64le
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: trippels at gcc dot gnu.org
              Host: powerpc64le-unknown-linux-gnu
            Target: powerpc64le-unknown-linux-gnu
             Build: powerpc64le-unknown-linux-gnu

Running the testsuite on the new gcc2-2-power8.osuosl.org powerpc64le machine
shows:

FAIL: gcc.c-torture/execute/20050604-1.c   -O2 -flto -fno-use-linker-plugin
-flto-partition=none  (internal compiler error)
FAIL: gcc.c-torture/execute/20050604-1.c   -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects  (internal compiler error)
FAIL: gcc.c-torture/execute/20060420-1.c   -O2 -flto -fno-use-linker-plugin
-flto-partition=none  (internal compiler error)
FAIL: gcc.c-torture/execute/20060420-1.c   -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects  (internal compiler error)
FAIL: c-c++-common/torture/vector-compare-1.c   -O2 -flto
-fno-use-linker-plugin -flto-partition=none  (internal compiler error)
FAIL: c-c++-common/torture/vector-compare-1.c   -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects  (internal compiler error)
FAIL: gcc.dg/torture/pr45678-1.c   -O2 -flto -fno-use-linker-plugin
-flto-partition=none  (internal compiler error)
FAIL: gcc.dg/torture/pr45678-2.c   -O2 -flto -fno-use-linker-plugin
-flto-partition=none  (internal compiler error)
FAIL: c-c++-common/torture/vector-compare-1.c   -O2 -flto
-fno-use-linker-plugin -flto-partition=none  (internal compiler error)
FAIL: c-c++-common/torture/vector-compare-1.c   -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects  (internal compiler error)

These all have the same backtrace:
c-c++-common/torture/vector-compare-1.c: In function ‘main’:
c-c++-common/torture/vector-compare-1.c:108:8: internal compiler error: in
convert_move, at expr.c:315
     f0 = (vector (4, float)){(float)argc, 1.,  2.,  10.};
        ^
0x1030bf03 convert_move(rtx_def*, rtx_def*, int)
        ../../gcc/gcc/expr.c:315
0x103130a7 store_expr_with_bounds(tree_node*, rtx_def*, int, bool, tree_node*)
        ../../gcc/gcc/expr.c:5459
0x1031b5e7 expand_assignment(tree_node*, tree_node*, bool)
        ../../gcc/gcc/expr.c:5154
0x101cb33f expand_gimple_stmt_1
        ../../gcc/gcc/cfgexpand.c:3385
0x101cb33f expand_gimple_stmt
        ../../gcc/gcc/cfgexpand.c:3481
0x101d2dd7 expand_gimple_basic_block
        ../../gcc/gcc/cfgexpand.c:5470
0x101d5387 execute
        ../../gcc/gcc/cfgexpand.c:6088
Please submit a full bug report,

FAIL: gcc.dg/torture/vector-1.c   -O2 -flto -fno-use-linker-plugin
-flto-partition=none  (internal compiler error)
FAIL: gcc.dg/torture/vector-2.c   -O2 -flto -fno-use-linker-plugin
-flto-partition=none  (internal compiler error)

These two have a different backtrace:
gcc.dg/torture/vector-2.c:15:16: internal compiler error: in
expand_expr_real_1, at expr.c:9666
  ((int*)&t)[0] = a;
                ^
0x103087c3 expand_expr_real_1(tree_node*, rtx_def*, machine_mode,
expand_modifier, rtx_def**, bool)
        ../../gcc/gcc/expr.c:9666
0x1031b8c3 expand_expr
        ../../gcc/gcc/expr.h:254
0x1031b8c3 expand_assignment(tree_node*, tree_node*, bool)
        ../../gcc/gcc/expr.c:4851
0x101cb33f expand_gimple_stmt_1
        ../../gcc/gcc/cfgexpand.c:3385
0x101cb33f expand_gimple_stmt
        ../../gcc/gcc/cfgexpand.c:3481
0x101d2dd7 expand_gimple_basic_block
        ../../gcc/gcc/cfgexpand.c:5470
0x101d5387 execute
        ../../gcc/gcc/cfgexpand.c:6088
Please submit a full bug report,
with preprocessed source if appropriate
>From gcc-bugs-return-477950-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Fri Feb 20 13:55:12 2015
Return-Path: <gcc-bugs-return-477950-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 6876 invoked by alias); 20 Feb 2015 13:55:12 -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 6791 invoked by uid 48); 20 Feb 2015 13:55:08 -0000
From: "jamborm at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug ipa/65028] [5 Regression] 450.soplex in SPEC CPU 2006 is miscompiled
Date: Fri, 20 Feb 2015 14:02:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: ipa
X-Bugzilla-Version: 5.0
X-Bugzilla-Keywords: wrong-code
X-Bugzilla-Severity: normal
X-Bugzilla-Who: jamborm at gcc dot gnu.org
X-Bugzilla-Status: RESOLVED
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: jamborm at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 5.0
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: bug_status resolution
Message-ID: <bug-65028-4-Qqelkpb4VW@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-65028-4@http.gcc.gnu.org/bugzilla/>
References: <bug-65028-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-02/txt/msg02282.txt.bz2
Content-length: 880

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

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

--- Comment #60 from Martin Jambor <jamborm at gcc dot gnu.org> ---
(In reply to Jan Hubicka from comment #57)
> Martin, I just tested firefox with -march=native -flto -O3 and it still
> segfaults
> for me at movaps, so it seems that there is still some issue with the code :(
> (it happens with or withtout the propagation patch I posted)
>

Please have a look at my comments of the patch.  If that still does
not help, open a new bug and I will have a look.  This PR has had more
than it's fair share of comments already :-)

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |5.0

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Only with -flto?  huh.

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #3 from Michael Meissner <meissner at gcc dot gnu.org> ---
I just built a toolchain on gcc2-power8.osuosl.org with the configuration line:

/home/meissner/fsf-src/trunk/configure
--prefix=/home/meissner/fsf-install-ppcle64/trunk
--enable-languages=c,c++,ada,fortran,java,objc,obj-c++ --with-long-double-128
--enable-secureplt --enable-__cxa_atexit --disable-libsanitizer
--with-cpu=power8

And after doing a make check, install, and check, I did not see any of these
tests generate errors with -flto -fuse-linker-plugin -flto-partition=none.

What was the exact command line you used to configure the compiler, and what
was your path?

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[trippels at gcc dot gnu.org]

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

--- Comment #4 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
(In reply to Michael Meissner from comment #3)
> I just built a toolchain on gcc2-power8.osuosl.org with the configuration
> line:
> 
> /home/meissner/fsf-src/trunk/configure
> --prefix=/home/meissner/fsf-install-ppcle64/trunk
> --enable-languages=c,c++,ada,fortran,java,objc,obj-c++
> --with-long-double-128 --enable-secureplt --enable-__cxa_atexit
> --disable-libsanitizer --with-cpu=power8
> 
> And after doing a make check, install, and check, I did not see any of these
> tests generate errors with -flto -fuse-linker-plugin -flto-partition=none.
> 
> What was the exact command line you used to configure the compiler, and what
> was your path?

It doesn't happen --with-cpu=power8. So please leave that out. I was using:

../gcc/configure --disable-bootstrap --disable-libstdcxx-pch --disable-libvtv
--disable-libitm --disable-libcilkrts --disable-libssp --disable-libgomp
--disable-werror --disable-multilib --enable-languages=c,c++

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #5 from Michael Meissner <meissner at gcc dot gnu.org> ---
Ok, after figuring out that I was building on gcc2-power8.osuosl.org (the old
big endian system that is going away), I checked out the current revision
(221024) and rebuilt a tool chain on gcc2-2-power8.osuosl.org.  I now see the
error if I build without the --with-cpu=power8 option (the options I used were
--prefix=<xxx> and --disable-multilib).

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-02-26
     Ever confirmed|0                           |1

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[dje at gcc dot gnu.org]

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

--- Comment #7 from David Edelsohn <dje at gcc dot gnu.org> ---
Allowing users to specify -mcpu=powerpc64le is all well and good, but if
TARGET_DEFAULT is set, it should not be gratuitously overridden in
rs6000_option_override.  The logic is not correct and defaults should not be
inherited from the processor name.

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #8 from Michael Meissner <meissner at gcc dot gnu.org> ---
Created attachment 34925
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34925&action=edit
Revised patch #2

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34923|0                           |1
        is obsolete|                            |
  Attachment #34925|0                           |1
        is obsolete|                            |

--- Comment #9 from Michael Meissner <meissner at gcc dot gnu.org> ---
Created attachment 34926
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34926&action=edit
Revised patch #3

This patch changes the logic to use TARGET_DEFAULT.  If TARGET_DEFAULT is 0, it
falls back to the defaults for powerpc, powerpc64, or powerpc64le.

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

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

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #10 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Tue Mar  3 00:59:56 2015
New Revision: 221132

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

    PR 65138/target
    * config/rs6000/rs6000-cpus.def (powerpc64le): Add new generic
    processor type for 64-bit little endian PowerPC.

    * config/rs6000/rs6000.c (rs6000_option_override_internal): If
    -mdebug=reg, print TARGET_DEFAULT.  Fix logic to use
    TARGET_DEFAULT if there is no default cpu.  Fix -mdebug=reg
    printing built-in mask so it does not pass NULL pointers.

    * doc/invoke.texi (IBM RS/6000 and PowerPC options): Document
    -mcpu=powerpc64le.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/rs6000/rs6000-cpus.def
    trunk/gcc/config/rs6000/rs6000.c
    trunk/gcc/doc/invoke.texi

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #11 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Tue Mar  3 17:04:01 2015
New Revision: 221155

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

    PR 65138/target
    * config/rs6000/rs6000-tables.opt: Regenerate table.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/rs6000/rs6000-tables.opt

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[aldyh at gcc dot gnu.org]

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

Aldy Hernandez <aldyh at gcc dot gnu.org> changed:

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

--- Comment #12 from Aldy Hernandez <aldyh at gcc dot gnu.org> ---
Is this fixed; can we close it?

[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #13 from Michael Meissner <meissner at gcc dot gnu.org> ---
I'm preparing the patches for the back ports to GCC 4.8/4.9.  Once those are
committed, I will close the PR.

[Bug target/65138] testsuite ICEs on powerpc64le

[aldyh at gcc dot gnu.org]

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

Aldy Hernandez <aldyh at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[5 Regression] testsuite    |testsuite ICEs on
                   |ICEs on powerpc64le         |powerpc64le

--- Comment #14 from Aldy Hernandez <aldyh at gcc dot gnu.org> ---
Changing summary to reflect that this is no longer a GCC5 regression, as it has
been fixed on mainline.

[Bug target/65138] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #15 from Michael Meissner <meissner at gcc dot gnu.org> ---
Created attachment 34943
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34943&action=edit
Backport of patches to gcc 4.9

[Bug target/65138] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #16 from Michael Meissner <meissner at gcc dot gnu.org> ---
Created attachment 34957
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34957&action=edit
Backport of patches to gcc 4.8

Here is the backport of the patches to gcc 4.8.

[Bug target/65138] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #17 from Michael Meissner <meissner at gcc dot gnu.org> ---
Just an additional comment.  Jakub asked whether the PowerPC needed the
additional target attribute support that the x86 added as part of PR61925.  I
looked at those patches, and at present those are not needed.  Those patches
are needed on the x86 because the x86 saves memory by not creating all of the
built-in functions at compiler start time.  Instead it only builds the
built-ins for the current switches.  If the user uses the target
attribute/pragma support to add new options, these built-in functions would be
created.

At the present time, the PowerPC does not create the built-in functions on the
fly, so it doesn't have to change the attributes when creating said functions. 
If we ever mirror the x86 behavior and not create all 1,167 built-in functions,
we would need similar patches.

[Bug target/65138] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #18 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Thu Mar  5 20:31:10 2015
New Revision: 221224

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

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

    PR 65138/target
    * config/rs6000/rs6000-cpus.def (powerpc64le): Add new generic
    processor type for 64-bit little endian PowerPC.

    * config/rs6000/rs6000.c (rs6000_option_override_internal): If
    -mdebug=reg, print TARGET_DEFAULT.  Fix logic to use
    TARGET_DEFAULT if there is no default cpu.  Fix -mdebug=reg
    printing built-in mask so it does not pass NULL pointers.

    * config/rs6000/rs6000-tables.opt: Regenerate.

    * doc/invoke.texi (IBM RS/6000 and PowerPC options): Document
    -mcpu=powerpc64le.

    Backport from trunk
    2015-01-19  David Edelsohn  <dje.gcc@gmail.com>

    * config/rs6000/default64.h: Include rs6000-cpus.def.
    (TARGET_DEFAULT) [LITTLE_ENDIAN]: Use ISA 2.7 (POWER8).
    (TARGET_DEFAULT) [BIG_ENDIAN]: Use POWER4.
    * config/rs6000/driver-rs6000.c (detect_processor_aix): Add POWER7
    and POWER8.
    * config/rs6000/linux64.h (PROCESSOR_DEFAULT64): Always default to
    POWER8.
    * config/rs6000/rs6000.c (rs6000_file_start): Emit .machine
    pseudo-op to specify assembler dialect.


Modified:
    branches/gcc-4_9-branch/gcc/ChangeLog
    branches/gcc-4_9-branch/gcc/config/rs6000/default64.h
    branches/gcc-4_9-branch/gcc/config/rs6000/driver-rs6000.c
    branches/gcc-4_9-branch/gcc/config/rs6000/linux64.h
    branches/gcc-4_9-branch/gcc/config/rs6000/rs6000-cpus.def
    branches/gcc-4_9-branch/gcc/config/rs6000/rs6000-tables.opt
    branches/gcc-4_9-branch/gcc/config/rs6000/rs6000.c
    branches/gcc-4_9-branch/gcc/doc/invoke.texi

[Bug target/65138] testsuite ICEs on powerpc64le

[meissner at gcc dot gnu.org]

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

--- Comment #19 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Thu Mar  5 21:26:25 2015
New Revision: 221225

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

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

    PR 65138/target
    * config/rs6000/rs6000-cpus.def (powerpc64le): Add new generic
    processor type for 64-bit little endian PowerPC.

    * config/rs6000/rs6000.c (rs6000_option_override_internal): If
    -mdebug=reg, print TARGET_DEFAULT.  Fix logic to use
    TARGET_DEFAULT if there is no default cpu.  Fix -mdebug=reg
    printing built-in mask so it does not pass NULL pointers.

    * config/rs6000/rs6000-tables.opt: Regenerate.

    * doc/invoke.texi (IBM RS/6000 and PowerPC options): Document
    -mcpu=powerpc64le.

    Backport from trunk
    2015-01-19  David Edelsohn  <dje.gcc@gmail.com>

    * config/rs6000/default64.h: Include rs6000-cpus.def.
    (TARGET_DEFAULT) [LITTLE_ENDIAN]: Use ISA 2.7 (POWER8).
    (TARGET_DEFAULT) [BIG_ENDIAN]: Use POWER4.
    * config/rs6000/driver-rs6000.c (detect_processor_aix): Add POWER7
    and POWER8.
    * config/rs6000/linux64.h (PROCESSOR_DEFAULT64): Always default to
    POWER8.
    * config/rs6000/rs6000.c (rs6000_file_start): Emit .machine
    pseudo-op to specify assembler dialect.


Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog
    branches/gcc-4_8-branch/gcc/config/rs6000/default64.h
    branches/gcc-4_8-branch/gcc/config/rs6000/driver-rs6000.c
    branches/gcc-4_8-branch/gcc/config/rs6000/linux64.h
    branches/gcc-4_8-branch/gcc/config/rs6000/rs6000-cpus.def
    branches/gcc-4_8-branch/gcc/config/rs6000/rs6000-tables.opt
    branches/gcc-4_8-branch/gcc/config/rs6000/rs6000.c
    branches/gcc-4_8-branch/gcc/doc/invoke.texi