[libcxx-commits] [PATCH] D120436: [libc++] Remove <chrono> include from <future>

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Feb 23 15:03:48 PST 2022


philnik updated this revision to Diff 410950.
philnik added a comment.

- Fix Documentation


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120436

Files:
  libcxx/docs/ReleaseNotes.rst
  libcxx/include/future


Index: libcxx/include/future
===================================================================
--- libcxx/include/future
+++ libcxx/include/future
@@ -377,8 +377,6 @@
 #include <thread>
 #include <version>
 
-#include <chrono> // TODO: Remove unused header
-
 #if !defined(_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER)
 #  pragma GCC system_header
 #endif
Index: libcxx/docs/ReleaseNotes.rst
===================================================================
--- libcxx/docs/ReleaseNotes.rst
+++ libcxx/docs/ReleaseNotes.rst
@@ -57,6 +57,7 @@
   If, after updating libc++, you see compiler errors related to missing declarations in
   namespace ``std``, it might be because one of your source files now needs to
   ``#include <algorithm>`` and/or ``#include <chrono>``.
+
 ABI Changes
 -----------
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120436.410950.patch
Type: text/x-patch
Size: 794 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220223/d89e1cae/attachment.bin>


More information about the libcxx-commits mailing list