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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
