Re: [PATCH] Partially fix 51988: value_replacement in PHIOPT should handle even the cases where there are other PHIs even with non equal value

2012-03-06 Thread Richard Guenther
On Wed, Jan 25, 2012 at 6:36 AM, Andrew Pinski wrote: > Hi, >  value_replacement in PHIOPT currently works only when there is one > PHI (which is non virtual). > http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01195.html improves the > situation but we can improve it even more as replacing a PHI argu

[PATCH] Partially fix 51988: value_replacement in PHIOPT should handle even the cases where there are other PHIs even with non equal value

2012-01-24 Thread Andrew Pinski
Hi, value_replacement in PHIOPT currently works only when there is one PHI (which is non virtual). http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01195.html improves the situation but we can improve it even more as replacing a PHI argument with a SSA_NAME is almost always a benefit. This patch imp