I forgot to mention that the dse patch fixes the problem earlier on so we now do the optimization pre-reload. We still have an extra store but that is recorded as another bug I filed.

Sent from my iPhone

On Mar 29, 2008, at 1:55, "pinskia at gcc dot gnu dot org" <[EMAIL PROTECTED] > wrote:



------- Comment #3 from pinskia at gcc dot gnu dot org 2008-03-29 08:55 ------- Hmm, I wonder how important this is now after the DSE patch for PR 33927 which basically does the same thing and it also runs after reload. I really don't want to make cselib any slower than it is already and post reload cse is really to me a hack for reload (or really RA) not doing its job so I don't want to
slow down post reload cse.

I am going to close this as won't fix as the reasons mentioned about.


--

pinskia at gcc dot gnu dot org changed:

          What    |Removed                     |Added
--- --- ----------------------------------------------------------------------
            Status|ASSIGNED                    |RESOLVED
        Resolution|                            |WONTFIX


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

Reply via email to