jroelofs added a comment.

LGTM, with one small nit (and once http://reviews.llvm.org/D13407 lands):


================
Comment at: src/locale.cpp:1176
@@ -1171,3 +1175,3 @@
 }
-#endif // __GLIBC__ || __EMSCRIPTEN__ || __NETBSD__
+#endif // __GLIBC__ || __NETBSD__ || || __EMSCRIPTEN__ || 
defined(_LIBCXX_LIBC_IS_MUSL)
 
----------------
    s/|| ||/||/


http://reviews.llvm.org/D13673



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to