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

Aaron Ballman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 06:07:14 PST 2018


aaron.ballman added a comment.

In D54883#1307853 <https://reviews.llvm.org/D54883#1307853>, @krytarowski wrote:

> In D54883#1307787 <https://reviews.llvm.org/D54883#1307787>, @aaron.ballman wrote:
>
> > Do our minimum system requirements for building LLVM include requiring POSIX 2008 conformance (perhaps indirectly, through the minimum toolchain requirements)?
>
>
> There are almost no OSes having conformance with UNIX/POSIX. UNIXv6 (or similar) is Solaris only.


If our existing system requirements aren't sufficient to practically guarantee these fields are available, I'd prefer to keep the autodetect code and rely on that instead.


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