Loop clean up (2)

Richard Henderson rth@cygnus.com
Mon Jan 24 14:28:00 GMT 2000


On Mon, Jan 24, 2000 at 09:07:07PM +1300, Michael Hayes wrote:
>  > Isn't loop->info is a void*?  IIRC, you need a cast here for K&R.
> 
> OK, how about the following patch which kills two bird with one stone.
> I've renamed the info field in the loop structure to aux and
> have introduced an accessor macro LOOP_INFO.

Ok.

>  > Eh?  Shouldn't this be (value & 2 ? 2 : 1)?  I'm guessing we only
>  > test for >1 though.  Perhaps just the comment should be changed.
> 
> I don't know.  I haven't touched this bit of code.

I know.  Neither problem I found was in code you touched.  ;-)

> Can I commit it?

Yes.  I guess I should have been more explicit.


r~


More information about the Gcc-patches mailing list