[Bug tree-optimization/51988] value_replacement in PHIOPT should handle even the cases where there are other PHIs even with non equal value

["pinskia at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>]

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

--- Comment #8 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-03-09 09:27:35 UTC ---
Author: pinskia
Date: Fri Mar  9 09:27:29 2012
New Revision: 185131

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=185131
Log:
2012-03-09  Andrew Pinski  <apinski@cavium.com>

    PR middle-end/51988
    * tree-ssa-phiopt.c: Include tree-pretty-print.h for
    print_generic_expr.
    (tree_ssa_phiopt_worker): Go through all the PHIs for
    value_replacement instead of just the singleton one.
    (value_replacement): Change return type to int.  Return 0 instead of
    false.
    Allow the middle basic block to contain more than just the definings
    tatement. 
    Handle non empty middle basic blocks.
    * Makefile.in (tree-ssa-phiopt.o): Add tree-pretty-print.h.

2012-03-09  Andrew Pinski  <apinski@cavium.com>

    PR middle-end/51988
    * gcc.dg/tree-ssa/phi-opt-8.c: New testcase.
    * gcc.dg/tree-ssa/phi-opt-9.c: New testcase.


Added:
    trunk/gcc/testsuite/gcc.dg/tree-ssa/phi-opt-8.c
    trunk/gcc/testsuite/gcc.dg/tree-ssa/phi-opt-9.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/Makefile.in
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-ssa-phiopt.c