rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land.
LGTM, but a minor simplification seems possible. ================ Comment at: include/__tuple:32 template <class _Tp> -class _LIBCPP_TYPE_VIS_ONLY tuple_size<const _Tp> - : public __tuple_size_base_type<_Tp>::type {}; +class _LIBCPP_TYPE_VIS_ONLY tuple_size<__enable_if_tuple_size_imp<const _Tp, decltype(tuple_size<typename remove_cv<_Tp>::type>::value)>> + : public integral_constant<size_t, tuple_size<_Tp>::value> {}; ---------------- remove_cv looks redundant here, deduction already stripped the cv qualifiers. https://reviews.llvm.org/D28222 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits