vsapsai added a comment. In https://reviews.llvm.org/D42755#994775, @Hahnfeld wrote:
> Can you please add a summary that describes which filesystem this problem can > be seen with? I think outside people can't access rdar tickets, so I can only > guess that it's related to APFS? Further guessing the filesystem supports > negative timestamps in general but not `file_time_type::min()` (which > probably is `-2 ** 63`)? Good suggestion for explaining the problem with `file_time_type::min()` in more detail. Also PR35990 <https://bugs.llvm.org/show_bug.cgi?id=35990> has some details on how different filesystems handle `file_time_type::min()`. As the change doesn't fix the PR, I don't think it is worth mentioning it in the commit message but it can be still useful in code review discussion. https://reviews.llvm.org/D42755 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
