This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch for loop unroll bug
- To: grahams at rcp dot co dot uk
- Subject: Re: Patch for loop unroll bug
- From: Jeffrey A Law <law at upchuck dot cygnus dot com>
- Date: Thu, 13 May 1999 01:54:57 -0600
- cc: egcs-patches at egcs dot cygnus dot com
- Reply-To: law at cygnus dot com
In message <3735B304.4283@rcp.co.uk>you write:
> Here's a corrected patch and ChangeLog entry against
> the egcs-19990502 snapshot.
>
> Graham
>
> ChangeLog
>
> * loop.c maybe_eliminate_biv(): check regno against
> max_reg_before_loop to correct a --enable-checking
> failure when compiling gcc/f/target.c
>
Installed. Also note that your mailer seems to be converting tabs into
spaces in addition to wrapping long lines. This makes your patches
difficult to apply.
The ChangeLog entry needed some work too. A ChangeLog entry should only note
the change, not the reason behind the change. If you want/need to identify
why a change was made you should do so as comments in the code itself since
that's what people are more likely to examine in the future if something
goes wrong.
Thanks,
jeff