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

Nathan Wilson via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 13 16:57:59 PST 2016


On Wed, Jan 13, 2016 at 6:48 PM, Richard Smith <richard at metafoo.co.uk>
wrote:

> On Wed, Jan 13, 2016 at 3:41 PM, Arthur O'Dwyer via cfe-commits <
> cfe-commits at lists.llvm.org> wrote:
>
>> Following Louis' suggestion, how about __pack_nth?
>>
>
> Maybe just __pack_element, to mirror its intended use to implement things
> like tuple_element? (I'm not completely happy about using this general name
> for something that only works for packs of types, but given that this
> template produces a type, it's probably the best we can do.)
>

That seems fine to me. I do feel like a name for something else will be
shoehorned in later as well though.

Louis - what do you think?


>
>
>> On Wed, Jan 13, 2016 at 3:16 PM, Nathan Wilson via cfe-commits <
>> cfe-commits at lists.llvm.org> wrote:
>>
>>>
>>>
>>> On Wed, Jan 13, 2016 at 4:52 PM, Richard Smith <richard at metafoo.co.uk>
>>> wrote:
>>>
>>>> On Wed, Jan 13, 2016 at 2:31 PM, Nathan Wilson <nwilson20 at gmail.com>
>>>> wrote:
>>>>
>>>>> nwilson added a comment.
>>>>>
>>>>> 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`?
>>>>
>>>>
>>>> Conversely, std::nth_element doesn't do indexing, and
>>>> std::tuple_element does.
>>>>
>>>
>>> Yeah, I was trying to combine them, but maybe that's misleading.
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>>>
>>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160113/ca860e60/attachment-0001.html>


More information about the cfe-commits mailing list