[Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure

[Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure

[hjl.tools at gmail dot com]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 3361 bytes --]

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

            Bug ID: 58521
           Summary: [4.9 Regression] bootstrap failure
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com
                CC: rguenth at gcc dot gnu.org

On Linux/i686, r202865 caused:

/export/project/git/gcc-regression/gcc/gcc/ira-color.c: In function ‘void
ira_color()’:
/export/project/git/gcc-regression/gcc/gcc/ira-color.c:4414:1: internal
compiler error: in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
 ira_color (void)
 ^
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.
make[3]: *** [ira-color.o] Error 1
make[3]: *** Waiting for unfinished jobs....
rm gcov.pod fsf-funding.pod cpp.pod gfdl.pod gcc.pod
make[3]: Leaving directory
`/export/project/git/gcc-regression-bootstrap/master/202865/bld/gcc'
make[2]: *** [all-stage3-gcc] Error 2
>From gcc-bugs-return-430482-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Tue Sep 24 19:03:28 2013
Return-Path: <gcc-bugs-return-430482-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 3938 invoked by alias); 24 Sep 2013 19:03:28 -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 3915 invoked by uid 48); 24 Sep 2013 19:03:26 -0000
From: "ctice at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug bootstrap/58441] [4.9 Regression] VTV headers are installed into the general include directory
Date: Tue, 24 Sep 2013 19:03:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: bootstrap
X-Bugzilla-Version: 4.9.0
X-Bugzilla-Keywords:
X-Bugzilla-Severity: minor
X-Bugzilla-Who: ctice at gcc dot gnu.org
X-Bugzilla-Status: UNCONFIRMED
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 4.9.0
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: attachments.created
Message-ID: <bug-58441-4-u0LMbx7DgO@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-58441-4@http.gcc.gnu.org/bugzilla/>
References: <bug-58441-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: 2013-09/txt/msg01722.txt.bz2
Content-length: 393

http://gcc.gnu.org/bugzilla/show_bug.cgi?idX441

--- Comment #5 from ctice at gcc dot gnu.org ---
Created attachment 30888
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id0888&actioníit
Patch to fix the problem

The Makefile was installing the headers in the wrong place.  This patch should
fix the problem.  I am currently testing this and will later submit the patch
for approval.

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure

[dominiq at lps dot ens.fr]

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

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
I have a different boostrap failure on x86_64-apple-darwin10:

/opt/gcc/build_w/./prev-gcc/xg++ -B/opt/gcc/build_w/./prev-gcc/
-B/opt/gcc/gcc4.9w/x86_64-apple-darwin10.8.0/bin/ -nostdinc++
-B/opt/gcc/build_w/prev-x86_64-apple-darwin10.8.0/libstdc++-v3/src/.libs
-B/opt/gcc/build_w/prev-x86_64-apple-darwin10.8.0/libstdc++-v3/libsupc++/.libs
-I/opt/gcc/build_w/prev-x86_64-apple-darwin10.8.0/libstdc++-v3/include/x86_64-apple-darwin10.8.0
-I/opt/gcc/build_w/prev-x86_64-apple-darwin10.8.0/libstdc++-v3/include
-I/opt/gcc/work/libstdc++-v3/libsupc++
-L/opt/gcc/build_w/prev-x86_64-apple-darwin10.8.0/libstdc++-v3/src/.libs
-L/opt/gcc/build_w/prev-x86_64-apple-darwin10.8.0/libstdc++-v3/libsupc++/.libs
-c   -g -O2 -mdynamic-no-pic -gtoggle -DIN_GCC   -fno-exceptions -fno-rtti
-fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings
-Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long
-Wno-variadic-macros -Wno-overlength-strings -Werror -fno-common 
-DHAVE_CONFIG_H -I. -I. -I../../work/gcc -I../../work/gcc/.
-I../../work/gcc/../include -I./../intl -I../../work/gcc/../libcpp/include
-I/opt/mp/include  -I../../work/gcc/../libdecnumber
-I../../work/gcc/../libdecnumber/dpd -I../libdecnumber
-I../../work/gcc/../libbacktrace -DCLOOG_INT_GMP  -I/opt/mp/include 
../../work/gcc/tree-ssa.c -o tree-ssa.o
../../work/gcc/tree-ssa-uninit.c: In function 'bool
is_use_properly_guarded(gimple, basic_block, gimple, unsigned int,
pointer_set_t*)':
../../work/gcc/tree-ssa-uninit.c:1907:1: internal compiler error: in
mem_ref_in_stmt, at tree-ssa-loop-im.c:677
 is_use_properly_guarded (gimple use_stmt,
 ^

../../work/gcc/tree-ssa-uninit.c:1907:1: internal compiler error: Abort trap
xg++: internal compiler error: Abort trap (program cc1plus)
make[3]: *** [tree-ssa-uninit.o] Abort trap
make[3]: *** Waiting for unfinished jobs....
rm fsf-funding.pod jcf-dump.pod grmic.pod gcov.pod gcj.pod cpp.pod gfortran.pod
gfdl.pod gij.pod gc-analyze.pod jv-convert.pod gcc.pod gcj-dbtool.pod
make[2]: *** [all-stage2-gcc] Error 2
make[1]: *** [stage2-bubble] Error 2
make: *** [all] Error 2

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure

[glisse at gcc dot gnu.org]

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

Marc Glisse <glisse at gcc dot gnu.org> changed:

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

--- Comment #2 from Marc Glisse <glisse at gcc dot gnu.org> ---
*** Bug 58524 has been marked as a duplicate of this bug. ***

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[burnus at gcc dot gnu.org]

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

--- Comment #3 from Tobias Burnus <burnus at gcc dot gnu.org> ---
As remarked in PR58524: For me, it failed with MALLOC_PERTURB_ set and didn't
without, which implies that some uninitialized memory is used.

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2013-09-25
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org
   Target Milestone|---                         |4.9.0
     Ever confirmed|0                           |1

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
So many people that can reproduce it but no preprocessed source ...

Anyway, mine (trying to reproduce it).

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[schwab@linux-m68k.org]

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

--- Comment #5 from Andreas Schwab <schwab@linux-m68k.org> ---
spawn /usr/local/gcc/gcc-20130925/Build/./gcc/xg++ -shared-libgcc
-B/usr/local/gcc/gcc-20130925/Build/./gcc -nostdinc++
-L/usr/local/gcc/gcc-20130925/Build/ia64-suse-linux/libstdc++-v3/src
-L/usr/local/gcc/gcc-20130925/Build/ia64-suse-linux/libstdc++-v3/src/.libs
-L/usr/local/gcc/gcc-20130925/Build/ia64-suse-linux/libstdc++-v3/libsupc++/.libs
-B/usr/ia64-suse-linux/bin/ -B/usr/ia64-suse-linux/lib/ -isystem
/usr/ia64-suse-linux/include -isystem /usr/ia64-suse-linux/sys-include
-B/usr/local/gcc/gcc-20130925/Build/ia64-suse-linux/./libstdc++-v3/src/.libs
-fdiagnostics-color=never -D_GLIBCXX_ASSERT -fmessage-length=0
-ffunction-sections -fdata-sections -O2 -g -D_GNU_SOURCE -O2 -g -D_GNU_SOURCE
-DLOCALEDIR="." -nostdinc++
-I/usr/local/gcc/gcc-20130925/Build/ia64-suse-linux/libstdc++-v3/include/ia64-suse-linux
-I/usr/local/gcc/gcc-20130925/Build/ia64-suse-linux/libstdc++-v3/include
-I/usr/local/gcc/gcc-20130925/libstdc++-v3/libsupc++
-I/usr/local/gcc/gcc-20130925/libstdc++-v3/include/backward
-I/usr/local/gcc/gcc-20130925/libstdc++-v3/testsuite/util
-I/usr/local/gcc/gcc-20130925/libstdc++-v3/testsuite -w -c -o testsuite_abi.o
/usr/local/gcc/gcc-20130925/libstdc++-v3/testsuite/util/testsuite_abi.cc
/usr/local/gcc/gcc-20130925/libstdc++-v3/testsuite/util/testsuite_abi.cc: In
function 'int compare_symbols(const char*, const char*, bool)':
/usr/local/gcc/gcc-20130925/libstdc++-v3/testsuite/util/testsuite_abi.cc:354:1:
internal compiler error: in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
 compare_symbols(const char* baseline_file, const char* test_file,
 ^
0x4000000001364bef mem_ref_in_stmt
    ../../gcc/tree-ssa-loop-im.c:677
0x4000000001364bef determine_max_movement
    ../../gcc/tree-ssa-loop-im.c:849
0x400000000136504f
invariantness_dom_walker::before_dom_children(basic_block_def*)
    ../../gcc/tree-ssa-loop-im.c:1172
0x400000000120c11f dom_walker::walk(basic_block_def*)
    ../../gcc/domwalk.c:176
0x400000000136888f tree_ssa_lim()
    ../../gcc/tree-ssa-loop-im.c:2629
0x4000000000e0830f tree_ssa_loop_im
    ../../gcc/tree-ssa-loop.c:145
0x4000000000e0830f execute
    ../../gcc/tree-ssa-loop.c:181

spawn /daten/aranym/gcc/gcc-20130925/Build/./gcc/xg++ -shared-libgcc
-B/daten/aranym/gcc/gcc-20130925/Build/./gcc -nostdinc++
-L/daten/aranym/gcc/gcc-20130925/Build/m68k-linux/libstdc++-v3/src
-L/daten/aranym/gcc/gcc-20130925/Build/m68k-linux/libstdc++-v3/src/.libs
-L/daten/aranym/gcc/gcc-20130925/Build/m68k-linux/libstdc++-v3/libsupc++/.libs
-B/daten/cross/m68k-linux/m68k-linux/bin/
-B/daten/cross/m68k-linux/m68k-linux/lib/ -isystem
/daten/cross/m68k-linux/m68k-linux/include -isystem
/daten/cross/m68k-linux/m68k-linux/sys-include
-B/daten/aranym/gcc/gcc-20130925/Build/m68k-linux/./libstdc++-v3/src/.libs
-fdiagnostics-color=never -D_GLIBCXX_ASSERT -fmessage-length=0
-ffunction-sections -fdata-sections -O2 -g -D_GNU_SOURCE -O2 -g -D_GNU_SOURCE
-DLOCALEDIR="." -nostdinc++
-I/daten/aranym/gcc/gcc-20130925/Build/m68k-linux/libstdc++-v3/include/m68k-linux
-I/daten/aranym/gcc/gcc-20130925/Build/m68k-linux/libstdc++-v3/include
-I/daten/aranym/gcc/gcc-20130925/libstdc++-v3/libsupc++
-I/daten/aranym/gcc/gcc-20130925/libstdc++-v3/include/backward
-I/daten/aranym/gcc/gcc-20130925/libstdc++-v3/testsuite/util
-I/daten/aranym/gcc/gcc-20130925/libstdc++-v3/testsuite -w -c -o
testsuite_abi.o
/daten/aranym/gcc/gcc-20130925/libstdc++-v3/testsuite/util/testsuite_abi.cc
/daten/aranym/gcc/gcc-20130925/libstdc++-v3/testsuite/util/testsuite_abi.cc: In
function 'int compare_symbols(const char*, const char*, bool)':
/daten/aranym/gcc/gcc-20130925/libstdc++-v3/testsuite/util/testsuite_abi.cc:354:1:
internal compiler error: in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
 compare_symbols(const char* baseline_file, const char* test_file,
 ^
0xdd6c71 mem_ref_in_stmt
    ../../gcc/tree-ssa-loop-im.c:677
0xdd6c71 determine_max_movement
    ../../gcc/tree-ssa-loop-im.c:849
0xdd718b invariantness_dom_walker::before_dom_children(basic_block_def*)
    ../../gcc/tree-ssa-loop-im.c:1172
0xd5f59a dom_walker::walk(basic_block_def*)
    ../../gcc/domwalk.c:176
0xdd8a74 tree_ssa_lim()
    ../../gcc/tree-ssa-loop-im.c:2629

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[rguenth at gcc dot gnu.org]

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

--- Comment #6 from Richard Biener <rguenth at gcc dot gnu.org> ---
Btw, the ICE hints at iterative_hash_expr and operand_equal_p being out-of-sync
after the change.  Yep:

        /* The type of the second operand is relevant, except for
           its top-level qualifiers.  */
        tree type = TYPE_MAIN_VARIANT (TREE_TYPE (TREE_OPERAND (t, 1)));

        val = iterative_hash_object (TYPE_HASH (type), val);

it's not required for correctness to hash this type.  See also the comment
in more generic code

          /* Don't hash the type, that can lead to having nodes which
             compare equal according to operand_equal_p, but which
             have different hash codes.  */

so this seems to be a recurring issue.

There shouldn't be many collisions if just hashing the offset constant,
so simply removing the MEM_REF special casing should work (TARGET_MEM_REF
isn't special-cased either).

So the above was without reproducing it yet (i586 bootstrap still ongoing).

Index: gcc/tree.c
===================================================================
--- gcc/tree.c  (revision 202862)
+++ gcc/tree.c  (working copy)
@@ -7298,21 +7280,6 @@ iterative_hash_expr (const_tree t, hashv
          }
        return val;
       }
-    case MEM_REF:
-      {
-       /* The type of the second operand is relevant, except for
-          its top-level qualifiers.  */
-       tree type = TYPE_MAIN_VARIANT (TREE_TYPE (TREE_OPERAND (t, 1)));
-
-       val = iterative_hash_object (TYPE_HASH (type), val);
-
-       /* We could use the standard hash computation from this point
-          on.  */
-       val = iterative_hash_object (code, val);
-       val = iterative_hash_expr (TREE_OPERAND (t, 1), val);
-       val = iterative_hash_expr (TREE_OPERAND (t, 0), val);
-       return val;
-      }
     case FUNCTION_DECL:
       /* When referring to a built-in FUNCTION_DECL, use the __builtin__ form.
         Otherwise nodes that compare equal according to operand_equal_p might

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[rguenth at gcc dot gnu.org]

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

--- Comment #7 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Wed Sep 25 09:51:13 2013
New Revision: 202889

URL: http://gcc.gnu.org/viewcvs?rev=202889&root=gcc&view=rev
Log:
2013-09-25  Richard Biener  <rguenther@suse.de>

    PR middle-end/58521
    * tree.c (iterative_hash_expr): Remove MEM_REF special handling.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/tree.c

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[rguenth at gcc dot gnu.org]

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

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

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

--- Comment #8 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.

[Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677

[congh at gcc dot gnu.org]

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

--- Comment #9 from congh at gcc dot gnu.org ---
Author: congh
Date: Thu Sep 26 01:36:49 2013
New Revision: 202932

URL: http://gcc.gnu.org/viewcvs?rev=202932&root=gcc&view=rev
Log:
2013-09-24  Cong Hou  <congh@google.com>

    Backport from mainline:
    2013-09-24  Richard Biener  <rguenther@suse.de>

    PR middle-end/58513
    * tree.c (reference_alias_ptr_type): Move ...
    * alias.c (reference_alias_ptr_type): ... here and implement
    in terms of the new reference_alias_ptr_type_1.
    (ref_all_alias_ptr_type_p): New helper.
    (get_deref_alias_set_1): Drop flag_strict_aliasing here,
    use ref_all_alias_ptr_type_p.
    (get_deref_alias_set): Add flag_strict_aliasing check here.
    (reference_alias_ptr_type_1): New function, split out from ...
    (get_alias_set): ... here.
    (alias_ptr_types_compatible_p): New function.
    * alias.h (reference_alias_ptr_type): Declare.
    (alias_ptr_types_compatible_p): Likewise.
    * tree.h (reference_alias_ptr_type): Remove.
    * fold-const.c (operand_equal_p): Use alias_ptr_types_compatible_p
    to compare MEM_REF alias types.

    2013-09-25  Richard Biener  <rguenther@suse.de>

    PR middle-end/58521
    * tree.c (iterative_hash_expr): Remove MEM_REF special handling.

2013-09-24  Cong Hou  <congh@google.com>

    Backport from mainline:
    2013-09-24  Richard Biener  <rguenther@suse.de>

    PR middle-end/58513
    * g++.dg/vect/pr58513.cc: New testcase.



Modified:
    branches/google/gcc-4_8/gcc/ChangeLog
    branches/google/gcc-4_8/gcc/alias.c
    branches/google/gcc-4_8/gcc/alias.h
    branches/google/gcc-4_8/gcc/fold-const.c
    branches/google/gcc-4_8/gcc/testsuite/ChangeLog
    branches/google/gcc-4_8/gcc/tree.c
    branches/google/gcc-4_8/gcc/tree.h