This is the mail archive of the gcc-patches@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: Possible fix for 30% performance regression in PR 33928


On Wed, May 6, 2009 at 6:15 AM, Bradley Lucier <lucier@math.purdue.edu> wrote:
>
> On May 5, 2009, at 11:41 PM, Bradley Lucier wrote:
>
>> cse.c hasn't changed since this revision, so this will apply cleanly.
>
> This is totally wrong, sorry. ?I'll look at testing it.

And in any case, just adding back those functions doesn't explain why
your code gets better. Blindly adding stuff back without understanding
why that helps isn't going to get us anywhere closer to simplifying
CSE.

There was a reason to remove them: fwprop should do the same
transformations already.  Maybe we can use your patch to figure out
what transformation is missing.

Ciao!
Steven


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