[libcxx-commits] [PATCH] D148405: [libc++] Remove the chrono include from algorithm
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Sat Apr 15 05:08:53 PDT 2023
Mordante requested changes to this revision.
Mordante added a comment.
This revision now requires changes to proceed.
Interesting that it fails with modules. In general we keep these transitive includes to avoid possibly breaking user code that depends on these transitive includes. (Note it's a bug in the user's code to depend on these includes, but this is not easily spotted.) However when it block us we are free to remove them. So no objection to the removal.
I expect this to fail in the CI since the transitive includes are not updated. Let me know if you need help with this.
This change needs to be mentioned in our release notes (docs/ReleaseNotes.rst) since it may affect users.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148405/new/
https://reviews.llvm.org/D148405
More information about the libcxx-commits
mailing list