[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