EricWF added a comment. So the reason there are `atime` tests at all is due to the implementation details of `__last_write_time` here. Essentially we have to do extra work trying to maintain the access time.
I'm OK with this patches rename. `first` and `second` were certainly not clear. That being said, I'm not sure we should be removing the access time tests, they were intentional. Repository: rCXX libc++ https://reviews.llvm.org/D47557 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
