[libcxx-commits] [PATCH] D100216: [libc++] Split a few things out of <memory>
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Fri Apr 9 10:07:26 PDT 2021
ldionne added a comment.
In D100216#2680139 <https://reviews.llvm.org/D100216#2680139>, @Quuxplusone wrote:
> As usual, I oppose this kind of patch unless it includes some "benefit" (e.g., simplifying the dependency graph). Right now this is just complexifying the graph.
The benefit is that `<memory>` is gradually becoming smaller. Locally, where I have more patches applied, I've started discovering things like helpers that are utterly misplaced (e.g. a helper that lives in `<memory>` but has no business being there, since it's only used in e.g. `<algorithm>`). Overall, this is going to make it easier to navigate and understand the library.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100216/new/
https://reviews.llvm.org/D100216
More information about the libcxx-commits
mailing list