EricWF closed this revision.
EricWF added a comment.
r334069
Repository:
rCXX libc++
https://reviews.llvm.org/D46806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Quuxplusone updated this revision to Diff 150067.
Quuxplusone edited the summary of this revision.
Quuxplusone added a comment.
Also, `` doesn't need a full definition of
`std::tuple`; just the forward declaration in `<__tuple>` will suffice.
Repository:
rCXX libc++
https://reviews.llvm.org/
Quuxplusone added inline comments.
Comment at: include/experimental/memory_resource:99
// 8.5, memory.resource
-class _LIBCPP_TEMPLATE_VIS memory_resource
+class _LIBCPP_TYPE_VIS memory_resource
{
...although maybe I don't understand the rules here. For example
Quuxplusone updated this revision to Diff 147677.
Quuxplusone added a comment.
@EricWF: would you mind landing these two drive-by fixes while you're at it?
Repository:
rCXX libc++
https://reviews.llvm.org/D46806
Files:
include/__functional_base
include/experimental/memory_resource
Inde
Quuxplusone created this revision.
Quuxplusone added a reviewer: EricWF.
Herald added a subscriber: cfe-commits.
`__user_alloc_construct_impl` is used by , but
this `__user_alloc_construct` is never used.
Repository:
rCXX libc++
https://reviews.llvm.org/D46806
Files:
include/__functional_
