[PATCH] D45792: [ADT] Teach reverse() about filter_iterator ranges

Tim Shen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 19 14:03:25 PDT 2018


timshen added a comment.

My understanding is that, the reason why reverse_iterator<filter_iterator<I, F>> doesn't "just work" is because filter_iterator<I> is only a forward_iterator, while reverse_iterator expects a bidirectional_iterator. Can we make filter_iterator bidirectional instead?


https://reviews.llvm.org/D45792





More information about the llvm-commits mailing list