Last active
October 17, 2016 14:40
-
-
Save antirez/9de0b349315361dd15921fccf61e7c33 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
-#if defined(__ATOMIC_RELAXED) && (!defined(__clang__) || __apple_build_version__ > 4210057) | |
+#if defined(__ATOMIC_RELAXED) && (!defined(__clang__) || !defined(__APPLE__) || __apple_build_version__ > 4210057) |
hmms... __ATOMIC_RELAXED is defined but 0. I wonder how it looks on gcc
...
Yep since is not a feature macro, but is just a constant used in the API. I assumed that, if the constant is defined in the header files, then it is supported, which is not the case... I should find a better way to test if the atomic primitives are supported.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Yes, adding !defined(APPLE) improves the Mac OS X targetting...
+1
FYI, a list of all defines on 10.8.5: