[PATCH] D49774: [libc++] Use __int128_t to represent file_time_type.

Billy Robert O'Neal III via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 25 11:39:34 PDT 2018


BillyONeal added a comment.

In https://reviews.llvm.org/D49774#1175131, @mclow.lists wrote:

> Another problem (that Eric and I discussed last night) is that filesystem is part of C++17, and `file_clock` is C++20. So we need a solution for C++17 as well.


It seems like we need to fix C++20 to allow that to be a typedef to a type in std::filesystem or that will be ABI breaking for MSVC++. IMO we should fix the spec to allow that rather than make libc++ jump through insane hoops.


Repository:
  rCXX libc++

https://reviews.llvm.org/D49774





More information about the cfe-commits mailing list