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: [RFC][PATCH,reload] PR middle-end/42431, fix latent bug in gen_reload_chain_without_interm_reg_p()


On 03/02/10 03:23, Jakub Jelinek wrote:
On Tue, Mar 02, 2010 at 11:13:49AM +0100, Richard Guenther wrote:
Bootstrapped and regression tested on x86_64-linux-gnu.
! VEC_free (rtx_p , heap, substitute_stack);
Extra space after rtx_p.

! return result;

not freeing the stack should speed up further calls for the cost of
a very small leak.

Otherwise ok.
I guess it can be freed e.g. at the end of reload1.c (reload).

Jakub
Attached is the final patch (whitespace fixed and memory free'd at the end of reload) that I checked in this morning.

jeff


Attachment: PP
Description: Text document


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