[PATCH] D47557: Filesystem tests: un-confuse write time

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 31 10:44:41 PDT 2018


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





More information about the cfe-commits mailing list