[Bug optimization/13965] New: [tree-ssa] fold takes into account branch target information at tree level

[Bug optimization/13965] New: [tree-ssa] fold takes into account branch target information at tree level

[dann at godzilla dot ics dot uci dot edu]

It seems that fold takes into account target branch cost. 


ssa-ccp-5.c is a copy testsuite/gcc.dg/tree-ssa/ssa-ccp-5.c

./cc1 -O2 -fdump-tree-optimized ssa-ccp-5.c

the .optimized dump for the second function in the file is:

test55 (x, y)
{
<bb 0>:
  return;

}

./cc1 -O2 -fdump-tree-optimized ssa-ccp-5.c -mtune=pentiumpro

test55 (x, y)
{
<bb 0>:
  if (x == 5 && y != 0) goto <L0>; else goto <L2>;

<L0>:;
  if (x + 22 != 27) goto <L1>; else goto <L2>;

<L1>:;
  link_error ();

<L2>:;
<L3>:;
  return;

}

Thanks to Andrew Pinski for diagnosing this. 

The effect of adding -mtune=pentiumpro to the testcase from PR8361 is:
without -mtune=pentiumpro  97575 assembly lines 
with -mtune=pentiumpro    103723 assembly lines 

This is probably an issue on the mainline too, but it's harder to show a
testcase there.

-- 
           Summary: [tree-ssa] fold takes into account branch target
                    information at tree level
           Product: gcc
           Version: tree-ssa
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dann at godzilla dot ics dot uci dot edu
                CC: gcc-bugs at gcc dot gnu dot org


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

[Bug optimization/13965] [tree-ssa] fold takes into account branch target information at tree level

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-02-01 21:14 -------
Confirmed.  I know that everyone agreed that this is the right thing to do.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
           Keywords|                            |pessimizes-code
   Last reconfirmed|0000-00-00 00:00:00         |2004-02-01 21:14:03
               date|                            |
   Target Milestone|---                         |tree-ssa


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

[Bug tree-optimization/13965] [tree-ssa] fold takes into account branch target information at tree level

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-05-13 13:56 -------
Mine, I am working on having the if statements folded at a later in the game.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
  BugsThisDependsOn|                            |15357
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


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

[Bug tree-optimization/13965] [tree-ssa] fold takes into account branch target information at tree level

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.5.0                       |---


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

[Bug tree-optimization/13965] [tree-ssa] fold takes into account branch target information at tree level

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-08-13 22:01 -------
Subject: Bug 13965

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	bdavis@gcc.gnu.org	2004-08-13 22:01:02

Modified files:
	libgfortran/io : open.c 
	libgfortran    : ChangeLog 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/gfortran.dg: pr16935.f90 

Log message:
	2004-08-13  Bud Davis  <bdavis9659@comcast.net>
	
	PR gfortran/13965
	* gfortran.fortran-torture/execute/open_rewind.f90: New test.
	* io/open.c (st_open): use flags instead of the unit structure.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libgfortran/io/open.c.diff?cvsroot=gcc&r1=1.3&r2=1.4
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gfortran.dg/pr16935.f90.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libgfortran/ChangeLog.diff?cvsroot=gcc&r1=1.59&r2=1.60
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.4134&r2=1.4135



-- 


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

[Bug tree-optimization/13965] [tree-ssa] fold takes into account branch target information at tree level

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-09-09 03:53 -------


*** This bug has been marked as a duplicate of 15911 ***

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


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

[Bug tree-optimization/13965] [tree-ssa] fold takes into account branch target information at tree level

[pinskia at gcc dot gnu.org]

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

Bug 13965 depends on bug 15357, which changed state.

Bug 15357 Summary: [tree-ssa] combing if statements
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=15357

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