[PATCH] D54883: [Support/FileSystem] Use st_atimespec only when __APPLE__ is defined

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 2 07:46:16 PST 2018


thakis added a comment.

In D54883#1315912 <https://reviews.llvm.org/D54883#1315912>, @mgorny wrote:

> Machine time is cheap, human time isn't. Saving time on CMake checks in no way justifies wasting somebody else's time when the hardcoded values mismatch at least once.


Isn't machine time that humans have to wait on human time too? 🤔


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54883/new/

https://reviews.llvm.org/D54883





More information about the llvm-commits mailing list