[Lldb-commits] [PATCH] Fixes http://reviews.llvm.org/rL230691

Zachary Turner zturner at google.com
Fri Feb 27 10:16:35 PST 2015


In http://reviews.llvm.org/D7933#131286, @clayborg wrote:

> I meant to say I like the other fix that uses pthread_condion_t and pthread_mutex_t since it won't spin like this fix will. Please use the other one.


Just to be clear, are you saying you don't want C++11 at all?  Or that you just want a condition variable instead of a spinlock?  Because if you just want a condition variable, then we should go with the C++11 and std::condition_variable as I suggested, since the test will work on platforms that don't have pthreads.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7933

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






More information about the lldb-commits mailing list