[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