[libcxx-commits] [PATCH] D122995: [NFC][libc++] Modularize chrono's calendar.
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Apr 5 08:40:23 PDT 2022
Mordante added a comment.
In D122995#3427701 <https://reviews.llvm.org/D122995#3427701>, @ldionne wrote:
> LGTM assuming you did not drop any transitive includes in the process. If you did, please keep them in `<chrono>` with a `TODO` to remove them. We can do it in bulk with a release note when we're ready.
No all includes where used, some have been removed from `calendar.h`, but the headers I moved them are included by `<chrono>`.
> Also, please poke CI just to make sure :)
I did, also made sure I removed the obsolete `.verify` files.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122995/new/
https://reviews.llvm.org/D122995
More information about the libcxx-commits
mailing list