[llvm-commits] PATCH: Add VariadicFunction class template to LLVM's ADT

[Chandler Carruth]

Thanks for the review. I've committed the first version in r146729 so that
fixes can happen in-tree. Almost all of your and David's feedback both here
and on IRC has been addressed. Only one issue remains, and if you have
ideas about how to solve it, I'll happily jump on that one.

On Thu, Dec 15, 2011 at 10:19 AM, Douglas Gregor <dgregor at apple.com> wrote:

> Just one comment:
>
> +  VariadicFunction() {}
> +
>
> This is not trivial, which is unfortunate. Please just leave out the
> default constructor so we'll get the implicitly-generated trivial default
> constructor. (The same goes for all of the VariadicFunction* class
> templates).
>

This sounds great, but it breaks declaring these as 'const'. It's really
not clear to me why, but I may be having a dense moment here. Do you think
its worth requiring these to be non-const to make the constructor trivial?
Any other ideas of how to do this? I agree trivial would be much better
here.

Also, I agree with David about the use of ArrayRef.
>

Yea, I have no idea why I didn't fix that first thing. Thanks to you both
for the prodding there, it makes the interface much nicer.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20111216/9c64ba96/attachment.html>