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

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 13 12:31:52 PST 2016


EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

Sorry, this LGTM. You should push it to 3.8 as well.


================
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?
It doesn't need to so long as "_Iter" is taken by value as it is in `<string>`.


http://reviews.llvm.org/D15862





More information about the cfe-commits mailing list