Re: RFA: PATCH to add variadic version of build_constructor

2013-05-08 Thread Jason Merrill
On 05/07/2013 04:41 AM, Richard Biener wrote: Eh, two NULL_TREE terminators are ugly ... callers know the number of elements, so maybe instead pass that number as argument? OK, sure. Can we overload build_constructor with a variadic variant? Thus, tree build_constructor (tree type, vec *val

Re: RFA: PATCH to add variadic version of build_constructor

2013-05-07 Thread Richard Biener
On Tue, May 7, 2013 at 5:20 AM, Jason Merrill wrote: > Most build_* functions have variadic versions, but build_constructor doesn't > as of yet, and it would be convenient for a patch I'm working on. I decided > to call it build_constructor_va, but am open to other naming ideas. > > Tested x86_64

RFA: PATCH to add variadic version of build_constructor

2013-05-06 Thread Jason Merrill
Most build_* functions have variadic versions, but build_constructor doesn't as of yet, and it would be convenient for a patch I'm working on. I decided to call it build_constructor_va, but am open to other naming ideas. Tested x86_64-pc-linux-gnu. OK for trunk? commit 7cab86001c233a8cac209e