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