Re: [PATCH] Improve PHI-OPT when there are multiple phis

2012-03-12 Thread Richard Guenther
On Sat, Mar 10, 2012 at 8:55 AM, Andrew Pinski wrote: > Woops I forgot the patch. Ok. Thanks, Richard. > Thanks, > Andrew Pinski > > On Fri, Mar 9, 2012 at 11:45 AM, Andrew Pinski > wrote: >> On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther >> wrote: >>> On Tue, Jan 24, 2012 at 7:34 AM, Andr

Re: [PATCH] Improve PHI-OPT when there are multiple phis

2012-03-09 Thread Andrew Pinski
Woops I forgot the patch. Thanks, Andrew Pinski On Fri, Mar 9, 2012 at 11:45 AM, Andrew Pinski wrote: > On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther > wrote: >> On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski >> wrote: >>> Hi, >>>  Right now PHI-OPT does try to handle the case where we hav

Re: [PATCH] Improve PHI-OPT when there are multiple phis

2012-03-09 Thread Andrew Pinski
On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther wrote: > On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski > wrote: >> Hi, >>  Right now PHI-OPT does try to handle the case where we have multiple >> PHIs but the other PHIs have the same value for the edges we care >> about. >> This fixes the issue

Re: [PATCH] Improve PHI-OPT when there are multiple phis

2012-01-24 Thread Richard Guenther
On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski wrote: > Hi, >  Right now PHI-OPT does try to handle the case where we have multiple > PHIs but the other PHIs have the same value for the edges we care > about. > This fixes the issue and allows PHI-OPT to handle a few more cases and > it removes the

[PATCH] Improve PHI-OPT when there are multiple phis

2012-01-23 Thread Andrew Pinski
Hi, Right now PHI-OPT does try to handle the case where we have multiple PHIs but the other PHIs have the same value for the edges we care about. This fixes the issue and allows PHI-OPT to handle a few more cases and it removes the TODO in the comments. OK For 4.8? Bootstrapped and tested on x86