[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