public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () @ 2014-09-17 8:56 jakub at gcc dot gnu.org 2014-09-17 9:06 ` [Bug debug/63284] " jakub at gcc dot gnu.org ` (5 more replies) 0 siblings, 6 replies; 7+ messages in thread From: jakub at gcc dot gnu.org @ 2014-09-17 8:56 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 Bug ID: 63284 Summary: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () Product: gcc Version: 4.9.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: debug Assignee: jakub at gcc dot gnu.org Reporter: jakub at gcc dot gnu.org /* { dg-do compile } */ /* { dg-options "-O2 -fcompare-debug" } */ int a[10], *b, *d, c, f; int fn2 (void); void fn3 (void); void fn4 (int); static int fn1 (int x) { int e = a[0]; if (e) return 1; if (b) switch (x) { case 1: if (d) e = fn2 (); else fn3 (); break; case 0: if (d) { fn3 (); if (c) fn4 (1); } else fn4 (0); } return e; } void fn6 (void) { f = fn1 (0); } fails on the 4.9 branch, it happens to succeed on the trunk, but when one looks at *.optimized dumps differences, there are still undesirable differences in the order of predecessors of one bb (and thus also PHI argument order). ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug debug/63284] [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org @ 2014-09-17 9:06 ` jakub at gcc dot gnu.org 2014-09-17 9:07 ` jakub at gcc dot gnu.org ` (4 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: jakub at gcc dot gnu.org @ 2014-09-17 9:06 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 Jakub Jelinek <jakub at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |4.9.2 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug debug/63284] [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org 2014-09-17 9:06 ` [Bug debug/63284] " jakub at gcc dot gnu.org @ 2014-09-17 9:07 ` jakub at gcc dot gnu.org 2014-09-17 19:06 ` jakub at gcc dot gnu.org ` (3 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: jakub at gcc dot gnu.org @ 2014-09-17 9:07 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 --- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> --- Created attachment 33507 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33507&action=edit gcc49-pr63284.patch Untested fix. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug debug/63284] [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org 2014-09-17 9:06 ` [Bug debug/63284] " jakub at gcc dot gnu.org 2014-09-17 9:07 ` jakub at gcc dot gnu.org @ 2014-09-17 19:06 ` jakub at gcc dot gnu.org 2014-09-17 19:07 ` jakub at gcc dot gnu.org ` (2 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: jakub at gcc dot gnu.org @ 2014-09-17 19:06 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 --- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> --- Author: jakub Date: Wed Sep 17 19:05:49 2014 New Revision: 215331 URL: https://gcc.gnu.org/viewcvs?rev=215331&root=gcc&view=rev Log: PR debug/63284 * tree-cfgcleanup.c (fixup_noreturn_call): Don't split block if there are only debug stmts after the noreturn call, instead remove the debug stmts. * gcc.dg/pr63284.c: New test. Added: trunk/gcc/testsuite/gcc.dg/pr63284.c Modified: trunk/gcc/ChangeLog trunk/gcc/testsuite/ChangeLog trunk/gcc/tree-cfgcleanup.c ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug debug/63284] [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org ` (2 preceding siblings ...) 2014-09-17 19:06 ` jakub at gcc dot gnu.org @ 2014-09-17 19:07 ` jakub at gcc dot gnu.org 2014-09-17 19:08 ` jakub at gcc dot gnu.org 2014-09-22 10:26 ` rguenth at gcc dot gnu.org 5 siblings, 0 replies; 7+ messages in thread From: jakub at gcc dot gnu.org @ 2014-09-17 19:07 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 --- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> --- Author: jakub Date: Wed Sep 17 19:07:04 2014 New Revision: 215332 URL: https://gcc.gnu.org/viewcvs?rev=215332&root=gcc&view=rev Log: PR debug/63284 * tree-cfgcleanup.c (fixup_noreturn_call): Don't split block if there are only debug stmts after the noreturn call, instead remove the debug stmts. * gcc.dg/pr63284.c: New test. Added: branches/gcc-4_9-branch/gcc/testsuite/gcc.dg/pr63284.c Modified: branches/gcc-4_9-branch/gcc/ChangeLog branches/gcc-4_9-branch/gcc/testsuite/ChangeLog branches/gcc-4_9-branch/gcc/tree-cfgcleanup.c ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug debug/63284] [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org ` (3 preceding siblings ...) 2014-09-17 19:07 ` jakub at gcc dot gnu.org @ 2014-09-17 19:08 ` jakub at gcc dot gnu.org 2014-09-22 10:26 ` rguenth at gcc dot gnu.org 5 siblings, 0 replies; 7+ messages in thread From: jakub at gcc dot gnu.org @ 2014-09-17 19:08 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 --- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> --- Author: jakub Date: Wed Sep 17 19:08:06 2014 New Revision: 215333 URL: https://gcc.gnu.org/viewcvs?rev=215333&root=gcc&view=rev Log: PR debug/63284 * tree-cfgcleanup.c (fixup_noreturn_call): Don't split block if there are only debug stmts after the noreturn call, instead remove the debug stmts. * gcc.dg/pr63284.c: New test. Added: branches/gcc-4_8-branch/gcc/testsuite/gcc.dg/pr63284.c Modified: branches/gcc-4_8-branch/gcc/ChangeLog branches/gcc-4_8-branch/gcc/testsuite/ChangeLog branches/gcc-4_8-branch/gcc/tree-cfgcleanup.c ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug debug/63284] [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org ` (4 preceding siblings ...) 2014-09-17 19:08 ` jakub at gcc dot gnu.org @ 2014-09-22 10:26 ` rguenth at gcc dot gnu.org 5 siblings, 0 replies; 7+ messages in thread From: rguenth at gcc dot gnu.org @ 2014-09-22 10:26 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63284 Richard Biener <rguenth at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> --- Fixed. ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-09-22 10:26 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-09-17 8:56 [Bug debug/63284] New: [4.9/5 Regression] -fcompare-debug issue due to redirection to __builtin_unreachable () jakub at gcc dot gnu.org 2014-09-17 9:06 ` [Bug debug/63284] " jakub at gcc dot gnu.org 2014-09-17 9:07 ` jakub at gcc dot gnu.org 2014-09-17 19:06 ` jakub at gcc dot gnu.org 2014-09-17 19:07 ` jakub at gcc dot gnu.org 2014-09-17 19:08 ` jakub at gcc dot gnu.org 2014-09-22 10:26 ` rguenth at gcc dot gnu.org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).