dim added inline comments.

================
Comment at: include/__threading_support:43
 
+#if defined(__clang__) && __has_attribute(acquire_capability)
+#define _LIBCPP_THREAD_SAFETY_ATTRIBUTE(x) __attribute__((x))
----------------
I think the least intrusive way would be to add a `defined(__FreeBSD__)` here, 
as that is the only OS with thread annotations for pthread functions, as far as 
I know.

The alternative is to turn off the capability annotations for 
`__libcpp_mutex_t` again, and use the `no_thread_safety_analysis` annotation 
for the functions.

Any particular preference?


https://reviews.llvm.org/D28520



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

Reply via email to