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

[Louis Dionne via Phabricator via cfe-commits]

ldionne added a comment.

In https://reviews.llvm.org/D49774#1175543, @BillyONeal wrote:

> 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.


We could also just provide `file_clock` "early" in C++17. Strictly speaking that may make our implementation non-conforming, but IDK how big of a deal this would be?


Repository:
  rCXX libc++

https://reviews.llvm.org/D49774