This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
Re: On purpose ???
- To: Theodore Papadopoulo <Theodore dot Papadopoulo at sophia dot inria dot fr>
- Subject: Re: On purpose ???
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Sat, 31 Oct 1998 01:07:51 -0700
- cc: egcs-bugs at cygnus dot com
- Reply-To: law at cygnus dot com
In message <199808101712.TAA18081@mururoa.inria.fr>you write:
>
> While browsing the warning reports of two different builds (
> sparc-sun-solaris2.6 and ) I noticed the following message:
>
> 1 unsigned value < 0 is always 0
>
> This happens in gcc/regclass.c at line 1786 in the allocate_reg_info
> function:
>
> /* Free up all storage allocated */
> if (num_regs < 0) ----> num_regs is unsigned
>
> This looks wrong. It seems to have been introduced after the change
> (Thu Jul 2 14:16:11 1998 Michael Meissner <meissner@cygnus.com>:
> see the full entry at the end of the mail).
>
> Now, AFAICT it seems that allocate_reg_info is never called with a
> negative parameter so it's not really a problem and we could just zap
> this piece of code (it looks that the storage is never freed ?? Maybe
> it's not worth bothering).
>
> All this is not really important, but still I thought that it was
> better to mention it.
This "problem" has been fixed in the development tree. The code to deallocate
the reg info has been moved into its own routine and is called at the end of
the compilation unit.
jeff