[PATCH] D15421: [Feature] Add a builtin for indexing into parameter packs

Louis Dionne via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 13 14:36:46 PST 2016


ldionne added a comment.

In http://reviews.llvm.org/D15421#326209, @nwilson wrote:

> In http://reviews.llvm.org/D15421#326144, @rsmith wrote:
>
> > Bikeshedding on the name a bit... how about `__type_pack_element`?
>
>
> Hmm, I kind of felt like having `nth` in there implied we're indexing into something... What about `__nth_pack_element`?


If anything, I would probably start the name with `__pack`. This way, we could add other builtins that follow the same pattern, like `__pack_slice`, `__pack_reverse` and probably others.

Or if we don't want to start with `__pack`, then no problem, but we might want to settle on a prefix that will work for other builtins on packs as well.


http://reviews.llvm.org/D15421





More information about the cfe-commits mailing list