Re: [PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Laurynas Biveinis
2011/8/16 Andreas Schwab : >        * ggc.h (ggc_alloc_rtvec_sized): Use ggc_alloc_zone_rtvec_def >        instead of ggc_alloc_zone_vec_rtvec_def. Thanks again, -- Laurynas

Re: [PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Andreas Schwab
Laurynas Biveinis writes: > The patch to adjust the definition is obvious/pre-approved if anyone's > inclined, or I'll just note in my TODO to fix this the next time I > commit something. Since I had it already readily tested on x86_64-linux, I've checked this in. Andreas. 2011-08-16 Andreas

Re: [PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Laurynas Biveinis
2011/8/16 Richard Guenther : >> ggc_alloc_zone_vec_rtvec_def is for allocating an array of rtvec_def, >> but you want a single (variable sized) rtvec_def, so >> ggc_alloc_zone_rtvec_def is the correct function to call. > > I'm not so sure about that given the more strongly typing of our allocators.

Re: [PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Richard Guenther
On Tue, Aug 16, 2011 at 1:17 PM, Andreas Schwab wrote: > Liang Wang writes: > >>  #define ggc_alloc_rtvec_sized(NELT)                                     \ >> -    (ggc_alloc_zone_vec_rtvec_def (sizeof (rtx),                        \ >> -                                   sizeof (struct rtvec_def

Re: [PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Andreas Schwab
Liang Wang writes: > #define ggc_alloc_rtvec_sized(NELT) \ > -(ggc_alloc_zone_vec_rtvec_def (sizeof (rtx),\ > - sizeof (struct rtvec_def) + ((NELT) - 1), > \ > +(ggc_alloc_zone_vec_rtvec_def (1

Re: [PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Richard Guenther
On Tue, Aug 16, 2011 at 11:35 AM, Liang Wang wrote: > Current implementation of ggc_alloc_rtvec_resized allocates more > spaces for rtvec.  This patch uses original formula to compute size > for rtvec.  Bootstrap on x86_64 successfully. > > OK for trunk? > > By the way, I don't have write access t

[PATCH ggc] fix ggc_alloc_rtvec_resized

2011-08-16 Thread Liang Wang
Current implementation of ggc_alloc_rtvec_resized allocates more spaces for rtvec. This patch uses original formula to compute size for rtvec. Bootstrap on x86_64 successfully. OK for trunk? By the way, I don't have write access to SVN repository yet. Could you please help commit it after appr