[libcxx-commits] [PATCH] D124123: [libc++] Remove <functional> includes

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Apr 23 01:07:50 PDT 2022


philnik added a comment.

@var-const What would you like to see in the description? The consensus was that we remove a few of the transitive includes in the libc++15 release cycle.
I don't think that Louis has to take a look here, since he approved of the direction. The entire reason this is in it's own patch is to be able to easily revert it just in case we break something in a really bad way.



================
Comment at: libcxx/docs/ReleaseNotes.rst:71
+- Some libc++ headers no longer transitively include all of
+    - ``<algorithm>``,
+    - ``<chrono>``,
----------------
var-const wrote:
> Optional: I think it's more common for list elements to end in `;` semicolons.
I think it makes more sense to remove them entirely.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124123/new/

https://reviews.llvm.org/D124123



More information about the libcxx-commits mailing list