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

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 11:36:25 PST 2018


mgorny added a comment.

I agree with Aaron. The CMake feature is working correctly for all supported use cases, and buys us much more portability than the replacement. I'm yet to hear any actual advantage from not using CMake check.


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