[PATCH] D15862: A possible direction for fixing https://llvm.org/bugs/show_bug.cgi?id=25973.

[Marshall Clow via cfe-commits]

mclow.lists added inline comments.

================
Comment at: include/iterator:1418
@@ +1417,3 @@
+template <class _Iter>
+struct __libcpp_is_trivial_iterator<move_iterator<_Iter> >
+	: public _LIBCPP_BOOL_CONSTANT(__libcpp_is_trivial_iterator<_Iter>::value) {};
----------------
EricWF wrote:
> Should this trait handle `const` and `volatile` like most other traits do?
I don't think so; a const iterator cannot be incremented or decremented, so it's pretty useless.  Note that pointers to const (i.e, `__libcpp_is_trivial_iterator<move_iterator<const char *>>::value`) is already true. 


http://reviews.llvm.org/D15862