[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #10 from Jakub Jelinek jakub at gcc dot gnu.org 2012-01-11 22:59:16 UTC --- Author: jakub Date: Wed Jan 11 22:59:12 2012 New Revision: 183111 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=183111 Log: PR bootstrap/51796 * combine.c (distribute_notes): If i3 is a noreturn call, allow old_size to be equal to args_size and make sure the noreturn call gets REG_ARGS_SIZE note. * expr.c (fixup_args_size_notes): Put REG_ARGS_SIZE notes on noreturn calls even when the delta is 0. * gcc.dg/pr51796.c: New test. Added: trunk/gcc/testsuite/gcc.dg/pr51796.c Modified: trunk/gcc/ChangeLog trunk/gcc/combine.c trunk/gcc/expr.c trunk/gcc/testsuite/ChangeLog
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||FIXED --- Comment #11 from Jakub Jelinek jakub at gcc dot gnu.org 2012-01-11 23:15:11 UTC --- Should be fixed now.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #9 from Mikael Pettersson mikpe at it dot uu.se 2012-01-10 09:10:07 UTC --- (In reply to comment #8) Ah, obviously s/REG_UNUSED/REG_NORETURN/g in the patch, sorry for that. With the above modification I can now build a cross successfully. I'll resume my native bootstrap now with it applied and see how it goes.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 Richard Guenther rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |4.7.0
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 Mikael Pettersson mikpe at it dot uu.se changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #1 from Mikael Pettersson mikpe at it dot uu.se 2012-01-09 12:44:35 UTC --- It's caused by r182924: Author: jakub Date: Thu Jan 5 20:47:16 2012 New Revision: 182924 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=182924 Log: PR debug/51762 * calls.c (emit_call_1): For noreturn calls force a REG_ARGS_SIZE note when !ACCUMULATE_OUTGOING_ARGS. m68k is !ACCUMULATE_OUTGOING_ARGS, so it triggers this change.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #2 from Mikael Pettersson mikpe at it dot uu.se 2012-01-09 13:19:23 UTC --- Created attachment 26279 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=26279 reduced test case
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #3 from Jakub Jelinek jakub at gcc dot gnu.org 2012-01-09 16:52:57 UTC --- I guess the assert should be adjusted, from gcc_assert (old_size != args_size); to gcc_assert (old_size != args_size || find_reg_note (i3, REG_NORETURN, NULL)); because we add the REG_ARGS_SIZE notes now even to noreturn calls where there is no difference from previous args size. Can't test it right now though.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #4 from Mikael Pettersson mikpe at it dot uu.se 2012-01-09 20:56:12 UTC --- The || find_reg_note (i3, REG_NORETURN, NULL) bit also failed because i3 has no notes at all. INSN_CODE (i3) == 437, which seems to correspond to *call from m68k.md.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #5 from Jakub Jelinek jakub at gcc dot gnu.org 2012-01-09 23:24:16 UTC --- Created attachment 26288 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=26288 gcc47-pr51796.patch Ah, REG_UNUSED might be still in next_note chain, not processed yet by distribute_notes. Completely (not even compile time) tested patch.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #6 from Andreas Schwab sch...@linux-m68k.org 2012-01-09 23:52:41 UTC --- Still fails in gcc_assert (n).
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 Zdenek Sojka zsojka at seznam dot cz changed: What|Removed |Added CC||zsojka at seznam dot cz --- Comment #7 from Zdenek Sojka zsojka at seznam dot cz 2012-01-10 05:45:55 UTC --- I hit this (probably) ICE with more obscure flags, with a x86_64-unknown-linux-gnu target: $ gcc -Os -fno-omit-frame-pointer -fno-tree-dominator-opts -fno-tree-fre -fno-tree-pre testsuite/gcc.c-torture/compile/pr39941.c testsuite/gcc.c-torture/compile/pr39941.c: In function 'bsd_boot': testsuite/gcc.c-torture/compile/pr39941.c:11:1: internal compiler error: in distribute_notes, at combine.c:13285 Please submit a full bug report, with preprocessed source if appropriate. See http://gcc.gnu.org/bugs.html for instructions.
[Bug bootstrap/51796] [4.7 regression] internal compiler error: in distribute_notes, at combine.c:13285 for libgomp/alloc.c on m68k-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51796 --- Comment #8 from Jakub Jelinek jakub at gcc dot gnu.org 2012-01-10 07:47:55 UTC --- Ah, obviously s/REG_UNUSED/REG_NORETURN/g in the patch, sorry for that.