This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PTR-PLUS merge into the mainline


Hi,

On Thu, 5 Jul 2007, Richard Guenther wrote:

> If there is anything to fix, then all those variants should produce
> the same code, not just foo3 and foo4.  So for these cases we should
> make sure that value-numbering sees them as computing the same value
> and extend combine to choose the instructions with the least cost.
> 
> Changing fold isn't a real fix.  It's a workaround for a specific
> testcase.

What do you suggest now specifically?
combine isn't the problem here, at the time we reach RTL this should 
already be done. Your patch only catches specific cases and pessimises 
others.

bye, Roman


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]