[PATCH] D20328: [libcxx] Externally threaded libc++ variant
Marshall Clow via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 7 09:06:06 PDT 2016
mclow.lists added a comment.
Some nits while reading this. More to come.
Also, I don't see how this can be retargeted "at runtime"; are you implying that someone can choose at program launch time what threading system to use?
How could that work given (say) a constexpr constructor for a mutex type?
================
Comment at: src/algorithm.cpp:99
@@ -90,1 +98,3 @@
+#ifndef _LIBCPP_HAS_NO_THREADS
+# undef __LOCK_LOCK
----------------
Why the undef here?
================
Comment at: src/mutex.cpp:277
@@ -261,1 +276,3 @@
+#ifndef _LIBCPP_HAS_NO_THREADS
+# undef __LOCK_INIT
----------------
Again; why undef at the end of a source file (not an include)?
For that matter, there's no need to use reserved identifiers here - `__LOCKXXX`
http://reviews.llvm.org/D20328
More information about the cfe-commits
mailing list