[PATCH] D134076: RFC - [ADT] Ranges pipe syntax support + SFINAE

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 19 15:44:39 PDT 2022


dblaikie added a comment.

In D134076#3800519 <https://reviews.llvm.org/D134076#3800519>, @jplayer-nv wrote:

> In D134076#3797962 <https://reviews.llvm.org/D134076#3797962>, @dblaikie wrote:
>
>> My gut reaction is that this is quite a lot of API surface area to add without current or near-future planned usage.
>>
>> Any chance you're interested in proposing this as part of a patch series to at least migrate some of the existing iterator-adaptor or custom algorithms to this usage? (@kazu - maybe this'd be of some interest to you too?)
>
> I'd be happy to break this down into bite-sized chunks.  Is discourse the right place to iterate on the details of such a patch series?  I can start with the ability to default construct and copy-assign existing iterators which carry a callable.

Yeah, Discourse is probably the place to go to build broader consensus about this novel direction.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134076/new/

https://reviews.llvm.org/D134076



More information about the llvm-commits mailing list