[Bug tree-optimization/43655] [4.5/4.6 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[Bug tree-optimization/43655] [4.5/4.6 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org
      Known to work|                            |

--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-10 19:21:02 UTC ---
The problem is that since 4.5, the:
  /* There must be no VDEFs.  */
  if (gimple_vdef (stmt))
    return false;

  /* Without alias info we can't move around loads.  */
  if (gimple_references_memory_p (stmt) && !optimize)
    return false;
never return false if !optimize (in 4.4 gimple_references_memory_p was just
checking a bit in gimple stmt).
We know at this point that is_gimple_assign (stmt), what's the best way to
replace the latter check to rule out all memory loads and stores if !optimize?

[Bug tree-optimization/43655] [4.5/4.6 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2010.12.10 19:25:42
         AssignedTo|unassigned at gcc dot       |jakub at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-10 19:25:42 UTC ---
Created attachment 22708
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22708
gcc46-pr43655.patch

Untested fix.

[Bug tree-optimization/43655] [4.5/4.6 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.5.2                       |4.5.3

--- Comment #9 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-12-16 13:03:46 UTC ---
GCC 4.5.2 is being released, adjusting target milestone.

[Bug tree-optimization/43655] [4.5/4.6 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[jakub at gcc dot gnu.org]

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

--- Comment #10 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-16 21:44:05 UTC ---
Author: jakub
Date: Thu Dec 16 21:44:02 2010
New Revision: 167955

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167955
Log:
    PR tree-optimization/43655
    * tree-ssa-ter.c (is_replaceable_p): Don't use
    gimple_references_memory_p for -O0, instead check for load
    by looking at rhs.

    * g++.dg/opt/pr43655.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/opt/pr43655.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-ssa-ter.c

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |4.6.0
            Summary|[4.5/4.6 Regression]        |[4.5 Regression] -ftree-ter
                   |-ftree-ter causes FAIL:     |causes FAIL:
                   |g++.old-deja/g++.law/temps5 |g++.old-deja/g++.law/temps5
                   |.C execution test           |.C execution test

--- Comment #11 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-16 21:58:25 UTC ---
Fixed on the trunk so far.

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[jakub at gcc dot gnu.org]

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

--- Comment #12 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-01-16 20:21:44 UTC ---
Author: jakub
Date: Sun Jan 16 20:21:42 2011
New Revision: 168866

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168866
Log:
    Backport from mainline
    2010-12-16  Jakub Jelinek  <jakub@redhat.com>

    PR tree-optimization/43655
    * tree-ssa-ter.c (is_replaceable_p): Don't use
    gimple_references_memory_p for -O0, instead check for load
    by looking at rhs.

    * g++.dg/opt/pr43655.C: New test.

Added:
    branches/gcc-4_5-branch/gcc/testsuite/g++.dg/opt/pr43655.C
Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_5-branch/gcc/tree-ssa-ter.c

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

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

--- Comment #13 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-01-17 08:06:36 UTC ---
Fixed.

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2010-04-06 11:21 -------
GCC 4.5.0 is being released.  Deferring to 4.5.1.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.5.0                       |4.5.1


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

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2010-04-06 09:27 -------
IMHO the best is to remove the -ftree-ter flag.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |matz at gcc dot gnu dot org


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

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.5.0


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

[Bug tree-optimization/43655] [4.5 Regression] -ftree-ter causes FAIL: g++.old-deja/g++.law/temps5.C execution test

[zsojka at seznam dot cz]

------- Comment #1 from zsojka at seznam dot cz  2010-04-05 21:23 -------
Created an attachment (id=20317)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20317&action=view)
g++.old-deja/g++.law/temps5.C


-- 


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