[PATCH] D49956: [ADT] Add an early-increment iterator and range adaptor.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 30 20:08:41 PDT 2018


chandlerc added a comment.

We have many, many for loops still using iterators for this....

If you search for thinsg like `Foo &Var = *It++;` with different
identifiers you will find a bunch.

I think we would be unsuccessful at converting them to lambda accepting
algorithms. The range based for loop seems likely the most we would succeed
in threading through how people think about this.... I mean, if there is no
consensus for this iterator-like thing, then maybe we have to go w/
algorithms. My fear is that it means we will just go with iterator for
loops.


Repository:
  rL LLVM

https://reviews.llvm.org/D49956





More information about the llvm-commits mailing list