* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
@ 2013-09-24 19:23 ` dominiq at lps dot ens.fr
2013-09-25 6:39 ` glisse at gcc dot gnu.org
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: dominiq at lps dot ens.fr @ 2013-09-24 19:23 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
2013-09-24 19:23 ` [Bug bootstrap/58521] " dominiq at lps dot ens.fr
@ 2013-09-25 6:39 ` glisse at gcc dot gnu.org
2013-09-25 7:15 ` [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
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: glisse at gcc dot gnu.org @ 2013-09-25 6:39 UTC (permalink / raw)
To: gcc-bugs
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. ***
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
2013-09-24 19:23 ` [Bug bootstrap/58521] " dominiq at lps dot ens.fr
2013-09-25 6:39 ` glisse at gcc dot gnu.org
@ 2013-09-25 7:15 ` burnus at gcc dot gnu.org
2013-09-25 7:55 ` rguenth at gcc dot gnu.org
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: burnus at gcc dot gnu.org @ 2013-09-25 7:15 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
` (2 preceding siblings ...)
2013-09-25 7:15 ` [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
@ 2013-09-25 7:55 ` rguenth at gcc dot gnu.org
2013-09-25 7:59 ` schwab@linux-m68k.org
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-25 7:55 UTC (permalink / raw)
To: gcc-bugs
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).
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
` (3 preceding siblings ...)
2013-09-25 7:55 ` rguenth at gcc dot gnu.org
@ 2013-09-25 7:59 ` schwab@linux-m68k.org
2013-09-25 8:03 ` rguenth at gcc dot gnu.org
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: schwab@linux-m68k.org @ 2013-09-25 7:59 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
` (4 preceding siblings ...)
2013-09-25 7:59 ` schwab@linux-m68k.org
@ 2013-09-25 8:03 ` rguenth at gcc dot gnu.org
2013-09-25 9:51 ` rguenth at gcc dot gnu.org
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-25 8:03 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
` (5 preceding siblings ...)
2013-09-25 8:03 ` rguenth at gcc dot gnu.org
@ 2013-09-25 9:51 ` rguenth at gcc dot gnu.org
2013-09-25 9:52 ` rguenth at gcc dot gnu.org
2013-09-26 1:37 ` congh at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-25 9:51 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
` (6 preceding siblings ...)
2013-09-25 9:51 ` rguenth at gcc dot gnu.org
@ 2013-09-25 9:52 ` rguenth at gcc dot gnu.org
2013-09-26 1:37 ` congh at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-25 9:52 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug bootstrap/58521] [4.9 Regression] bootstrap failure: ICE in mem_ref_in_stmt, at tree-ssa-loop-im.c:677
2013-09-24 18:51 [Bug bootstrap/58521] New: [4.9 Regression] bootstrap failure hjl.tools at gmail dot com
` (7 preceding siblings ...)
2013-09-25 9:52 ` rguenth at gcc dot gnu.org
@ 2013-09-26 1:37 ` congh at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: congh at gcc dot gnu.org @ 2013-09-26 1:37 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 10+ messages in thread