[libcxx-commits] [PATCH] D133490: [libc++] Fix compilation error on platforms that don't implement std::tm

Joe Loser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Sep 8 07:54:43 PDT 2022


jloser added inline comments.


================
Comment at: libcxx/include/__chrono/convert_to_tm.h:15
 #include <__config>
 #include <ctime>
 
----------------
Since we're taking the `tm` type as a template argument, we never explicitly mention `std::tm` now.  Therefore, we should be able to remove this include, right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133490



More information about the libcxx-commits mailing list