[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