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: [PATCH]: Cleaning rtl loop walking, take 2


On Sat, 2005-03-05 at 16:57 -0500, Daniel Berlin wrote:
> This one was bootstrapped and regtested on i686-pc-linux-gnu.
> No regressions.
> 
> pre-approved by Roger Sayle on IRC.
> 
> Committed to mainline.
> 

Before i committed it, kazu pointed out that some of these loops
actually do weird things buried deep inside of them, that may delete
instructions.

Rather than play around with this and figure out which ones will work,
i'm just going to give up on this cleanup for now.  This was more of a 2
hour project done due to some prodding by stevenb when i mentioned that
whoever added the macro decided not to use it in old code.
I'm beginning to see why.

Hopefully, we won't accept changes to RTL that don't use the macro where
possible (i ran into at least have a dozen places where we write that
loop in different ways that are all equivalent)


--Dan


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