public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0
@ 2013-09-12  9:46 schwab@linux-m68k.org
  2013-09-12  9:47 ` [Bug middle-end/58402] " schwab@linux-m68k.org
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: schwab@linux-m68k.org @ 2013-09-12  9:46 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 58402
           Summary: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c
                    scan-tree-dump-times optimized "if" 0
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
          Assignee: unassigned at gcc dot gnu.org
          Reporter: schwab@linux-m68k.org
                CC: rguenther at suse dot de
            Blocks: 58377
            Target: m68k-*-*

This fails since r202496:

$ rm -f phi-opt-1.c.* && gcc/xgcc -Bgcc/
../gcc/testsuite/gcc.dg/tree-ssa/phi-opt-1.c -fno-diagnostics-show-caret
-fdiagnostics-color=never -O1 -fdump-tree-optimized -S -o phi-opt-1.s && grep
if phi-opt-1.c.*.optimized 
  if (c_2(D) == 0)
  if (a_3(D) == 0)


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug middle-end/58402] [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0
  2013-09-12  9:46 [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0 schwab@linux-m68k.org
@ 2013-09-12  9:47 ` schwab@linux-m68k.org
  2013-09-12  9:54 ` rguenth at gcc dot gnu.org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: schwab@linux-m68k.org @ 2013-09-12  9:47 UTC (permalink / raw)
  To: gcc-bugs

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

Andreas Schwab <schwab@linux-m68k.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.9.0


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug middle-end/58402] [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0
  2013-09-12  9:46 [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0 schwab@linux-m68k.org
  2013-09-12  9:47 ` [Bug middle-end/58402] " schwab@linux-m68k.org
@ 2013-09-12  9:54 ` rguenth at gcc dot gnu.org
  2013-09-12 10:14 ` rguenth at gcc dot gnu.org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-12  9:54 UTC (permalink / raw)
  To: gcc-bugs

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

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

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

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Hm, and I thought I tested that it wasn't caused by my patch by reverting and
re-checking .... somehow I must have messed up.


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug middle-end/58402] [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0
  2013-09-12  9:46 [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0 schwab@linux-m68k.org
  2013-09-12  9:47 ` [Bug middle-end/58402] " schwab@linux-m68k.org
  2013-09-12  9:54 ` rguenth at gcc dot gnu.org
@ 2013-09-12 10:14 ` rguenth at gcc dot gnu.org
  2013-09-12 13:19 ` rguenth at gcc dot gnu.org
  2013-09-12 13:21 ` rguenth at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-12 10:14 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Ok, so the testcase requires that

 1) critical edges are split (before PRE which isn't run at -O1)
 2) the next CFG cleanup (after copyprop) will end up removing the
    forwarders in a way building up a tree-argument PHI
 3) PHI-OPT executes for the third time, this time with the exact
    right CFG / PHI node present

my patch obfuscates the CFG gain by splitting edges and no CFG cleanup
running inbetween that and the late PHI-OPT pass.


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug middle-end/58402] [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0
  2013-09-12  9:46 [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0 schwab@linux-m68k.org
                   ` (2 preceding siblings ...)
  2013-09-12 10:14 ` rguenth at gcc dot gnu.org
@ 2013-09-12 13:19 ` rguenth at gcc dot gnu.org
  2013-09-12 13:21 ` rguenth at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-12 13:19 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Thu Sep 12 13:19:21 2013
New Revision: 202524

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

    PR tree-optimization/58402
    * passes.def: Move pass_late_warn_uninitialized later.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/passes.def


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug middle-end/58402] [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0
  2013-09-12  9:46 [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0 schwab@linux-m68k.org
                   ` (3 preceding siblings ...)
  2013-09-12 13:19 ` rguenth at gcc dot gnu.org
@ 2013-09-12 13:21 ` rguenth at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: rguenth at gcc dot gnu.org @ 2013-09-12 13:21 UTC (permalink / raw)
  To: gcc-bugs

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

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

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

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


^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-09-12 13:21 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-12  9:46 [Bug middle-end/58402] New: [4.9 regression] FAIL: gcc.dg/tree-ssa/phi-opt-1.c scan-tree-dump-times optimized "if" 0 schwab@linux-m68k.org
2013-09-12  9:47 ` [Bug middle-end/58402] " schwab@linux-m68k.org
2013-09-12  9:54 ` rguenth at gcc dot gnu.org
2013-09-12 10:14 ` rguenth at gcc dot gnu.org
2013-09-12 13:19 ` rguenth at gcc dot gnu.org
2013-09-12 13:21 ` 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).