[PATCH] D30136: [compiler-rt][builtins][WIP] Add _IsOSVersionAtLeast, to be used by ObjC's @available

Alex Lorenz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 20 09:19:01 PST 2017


arphaman added a comment.

Thanks for taking your time to work on this!

We've discussed the best approach internally and came to the conclusion that `Gestalt` should be used to determine the version of the OS even though it's officially deprecated. I'm sorry if my previous comments in which I dismissed `Gestalt` made you do extra work, I wasn't aware of some nuances back then. Can you please rewrite this function to use `Gestalt`?


https://reviews.llvm.org/D30136





More information about the llvm-commits mailing list