[PATCH] Implement shared_mutex re: N4508

Eric Fiselier eric at efcs.ca
Mon Jun 29 13:44:27 PDT 2015


Can apple take this patch since it adds new symbols to the dylib?


================
Comment at: include/shared_mutex:196
@@ +195,3 @@
+
+//     typedef __shared_mutex_base::native_handle_type native_handle_type;
+//     _LIBCPP_INLINE_VISIBILITY native_handle_type native_handle() { return __base::unlock_shared(); }
----------------
mclow.lists wrote:
> EricWF wrote:
> > Why are these commented out/not getting committed?
> In the standard, they are optional.
Why comment them out though? Either we should ship them or remove them.

http://reviews.llvm.org/D10480

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list