On Thu, Mar 12, 2020 at 01:58:20AM -0400, Jason Merrill wrote:
> > +      if (reuse && nelts < CONSTRUCTOR_NELTS (new_init))
> > +   {
> > +     vec<constructor_elt, va_gc> *v = NULL;
> > +     if (nelts)
> 
> vec_alloc does nothing if nelts is 0, so this test seems unnecessary. OK
> either way.

I wasn't sure, but now I've verified it doesn't do anything but v = NULL,
which is exactly what we want.  Dropped the " = NULL" and if (nelts)
and committed, thanks.
> 
> > +       vec_alloc (v, nelts);

        Jakub

Reply via email to