This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: New free_reg_info function
- To: Bernd Schmidt <crux at pool dot informatik dot rwth-aachen dot de>
- Subject: Re: New free_reg_info function
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Wed, 28 Oct 1998 11:22:07 -0700
- cc: egcs-patches at cygnus dot com
- Reply-To: law at cygnus dot com
In message <Pine.GSO.4.02A.9810281018270.6834-100000@matlock.informatik.rwth-
aachen.de>you write:
> Today's CVS source calls allocate_reg_info with -1 as first argument to
> tell it to free all memory. Unfortunately, the first argument is of type
> size_t, which is unsigned at least on sparc-sun-solaris2.6, so this change
> caused a nasty machine lockup due to allocating a huge amount of memory.
>
> Bernd
>
> * regclass.c (renumber, regno_allocated): New static variables, moved
> out of allocate_reg_info.
> (allocate_reg_info): Move these two variables outside the function.
> Move code to free memory into new function free_reg_info.
> (free_reg_info): New function, broken out of allocate_reg_info.
> * toplev.c (compile_file): Call free_reg_info, not allocate_reg_info.
> * rtl.h (allocate_reg_info): Don't declare.
> (free_reg_info): Declare.
Sorry about the lossage. Thanks for the patch, I've installed it.
jeff