[libcxx-commits] [PATCH] D132284: [libc++] Reduces the number of transitive includes.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Sep 2 08:56:35 PDT 2022


Mordante added inline comments.


================
Comment at: libcxx/include/algorithm:1902
 
-#ifndef _LIBCPP_REMOVE_TRANSITIVE_INCLUDES
-#  include <chrono>
+#if !defined(_LIBCPP_REMOVE_TRANSITIVE_INCLUDES) && _LIBCPP_STD_VER <= 17
+#  include <chrono> // IGNORE-CYCLE due to <format>
----------------
Mordante wrote:
> During the live code review we discussed to move these transitive includes to the end of the file. That will be done in a followup patch.
See D133212


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132284



More information about the libcxx-commits mailing list