[PATCH] D16639: [libcxx] Limit catopen usage to unix-like OSes

[Ben Craig via cfe-commits]

bcraig updated this revision to Diff 46291.
bcraig added a comment.

CloudABI doesn't define __unix__, so stop blacklisting it explicitly and let the whitelist filter it out.


http://reviews.llvm.org/D16639

Files:
  include/__config
  include/locale

Index: include/locale
===================================================================
--- include/locale
+++ include/locale
@@ -199,6 +199,14 @@
 // has had a chance to bake for a bit
 #include <support/newlib/xlocale.h>
 #endif
+
+#if defined(__unix__) || (defined(__APPLE__) && defined(__MACH__))
+// Most unix variants have catopen.  These are the specific ones that don't.
+#if !defined(_WIN32) && !defined(__ANDROID__) && !defined(_NEWLIB_VERSION)
+#define _LIBCPP_HAS_CATOPEN 1
+#endif
+#endif
+
 #ifdef _LIBCPP_HAS_CATOPEN
 #include <nl_types.h>
 #endif
Index: include/__config
===================================================================
--- include/__config
+++ include/__config
@@ -713,11 +713,6 @@
 #define _LIBCPP_LOCALE__L_EXTENSIONS 1
 #endif
 
-#if !defined(_WIN32) && !defined(__ANDROID__) && !defined(_NEWLIB_VERSION) && \
-    !defined(__CloudABI__)
-#define _LIBCPP_HAS_CATOPEN 1
-#endif
-
 #ifdef __FreeBSD__
 #define _DECLARE_C99_LDBL_MATH 1
 #endif


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16639.46291.patch
Type: text/x-patch
Size: 982 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160128/e59eec19/attachment-0001.bin>