[PATCH] D28220: provide Win32 native threading

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 4 19:45:21 PST 2017


compnerd marked an inline comment as done.
compnerd added inline comments.


================
Comment at: include/__threading_support:458
+                               __libcpp_mutex_reference&& __m,
+                               timespec* __ts)
+{
----------------
halyavin wrote:
> In posix, pthread_cond_timedwait uses absolute time but we use relative time here.
Good catch.  I guess we have to convert the timespec to a relative time before using it.  Slightly annoying, but thankfully, we can fallback on chrono :-).


Repository:
  rL LLVM

https://reviews.llvm.org/D28220





More information about the cfe-commits mailing list