Add note for location of other convenience functions.

[aaron.ballman]

Can you add some unit tests?

include/llvm/ADT/iterator_range.h

67

function -> functions
stl-like containers -> containers

[majnemer]

include/llvm/ADT/iterator_range.h

75–87

Doesn't reverse_range duplicate the reverse we already have in STLExtras.h?

[marsupial]

include/llvm/ADT/iterator_range.h

75–87

Thats all I really wanted...Should I add a comment it's there?

[chandlerc]

include/llvm/ADT/iterator_range.h

75–87

Personally, I think we should move the range-based utilities out of STLExtras.h and into a header that is more likely to be found. Either this one or something quite clearly providing range-based algos and adapters.

[marsupial]

include/llvm/ADT/iterator_range.h

75–87

Doing that right and making sure includes are updated properly seems like a lengthy process (which I personally don't have time for right now).

The hint seems adequate to those trying to glean where these may be located, but if it's not wanted I can close this out all together.

include/llvm/ADT/iterator_range.h

 template <typename T> iterator_range<T> make_range(std::pair<T, T> p) {
   return iterator_range<T>(std::move(p.first), std::move(p.second));
 }
 
 template<typename T>
 iterator_range<decltype(begin(std::declval<T>()))> drop_begin(T &&t, int n) {
   return make_range(std::next(begin(t), n), end(t));
 }
+
+/// Other convenience functions in STLExtras.h.
+
 }
 
 #endif