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

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 2 08:32:58 PST 2018


Certainly human time that you have to wait on unconditionally every single
time is more of a waste than that which you have to wait on only rarely
when someone makes a mistake and until it’s fixed.
On Sun, Dec 2, 2018 at 7:46 AM Nico Weber via Phabricator <
reviews at reviews.llvm.org> wrote:

> 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
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181202/9387fefe/attachment.html>


More information about the llvm-commits mailing list