On Mon, Jul 13, 2015 at 02:43:46PM +0100, Richard Sandiford wrote:
> Nice clean-up :-)
> 
> tbsaunde+...@tbsaunde.org writes:
> > @@ -2248,11 +2249,14 @@ process_define_subst (void)
> >  static void
> >  rtx_handle_directive (int lineno, const char *rtx_name)
> >  {
> > -  rtx queue, x;
> > +  auto_vec<rtx, 10> subrtxs;
> > +  if (!read_rtx (rtx_name, &subrtxs))
> > +    return;
> 
> Very minor, but many iterators are over powers of 2, so it seems unlikely
> that 10 would be a good stack/heap cut-off point.  How about 16 or 32
> instead?
> 
> OK with that change, thanks.

changed to 32 and committed, thanks!

Trev

> 
> Richard
> 

Reply via email to