Re: [PATCH] D11781: Refactored pthread usage in libcxx

2016-05-10 Thread Fulvio Esposito via cfe-commits
espositofulvio added a comment. In http://reviews.llvm.org/D11781#423520, @rmaprath wrote: > @espositofulvio: Thanks for the patch! :) > > Committed as r268734. Glad to see you land the patch! Great work :) Repository: rL LLVM http://reviews.llvm.org/D11781

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2016-04-16 Thread Fulvio Esposito via cfe-commits
espositofulvio added a comment. In http://reviews.llvm.org/D11781#400968, @rmaprath wrote: > Hi, could I know the status of this? I'd like to push this forward. > > @espositofulvio: Are you working on this? (just checking since this has gone > stale for a while). @EricWF: I can create a

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-09-05 Thread Fulvio Esposito via cfe-commits
espositofulvio updated the summary for this revision. espositofulvio updated this revision to Diff 34104. espositofulvio added a comment. - Addressed possible ABI breaks - Reverted to not using a __config_file as @jroelofs has two separate patch for that Repository: rL LLVM

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-08-11 Thread Fulvio Esposito via cfe-commits
espositofulvio added inline comments. Comment at: include/__config:742 @@ +741,3 @@ +#ifndef _LIBCPP_HAS_NO_THREADS +# if defined(__FreeBSD__) || \ +defined(__NetBSD__) || \ jroelofs wrote: jroelofs wrote: espositofulvio wrote: theraven wrote:

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-08-11 Thread Fulvio Esposito via cfe-commits
espositofulvio added inline comments. Comment at: include/__config:742 @@ +741,3 @@ +#ifndef _LIBCPP_HAS_NO_THREADS +# if defined(__FreeBSD__) || \ +defined(__NetBSD__) || \ theraven wrote: espositofulvio wrote: jroelofs wrote: espositofulvio wrote:

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-08-10 Thread Fulvio Esposito via cfe-commits
espositofulvio updated this revision to Diff 31716. espositofulvio added a comment. Added __CloudABI__ to the list of platform using pthread Repository: rL LLVM http://reviews.llvm.org/D11781 Files: include/__config include/__mutex_base include/mutex

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-08-10 Thread Fulvio Esposito via cfe-commits
espositofulvio marked 5 inline comments as done. espositofulvio added a comment. Repository: rL LLVM http://reviews.llvm.org/D11781 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-08-10 Thread Fulvio Esposito via cfe-commits
espositofulvio added inline comments. Comment at: include/__config:742 @@ +741,3 @@ +#ifndef _LIBCPP_HAS_NO_THREADS +# if defined(__FreeBSD__) || \ +defined(__NetBSD__) || \ jroelofs wrote: jroelofs wrote: @espositofulvio: @ed meant this: ```

Re: [PATCH] D11781: Refactored pthread usage in libcxx

2015-08-10 Thread Fulvio Esposito via cfe-commits
espositofulvio added inline comments. Comment at: include/__config:742 @@ +741,3 @@ +#ifndef _LIBCPP_HAS_NO_THREADS +# if defined(__FreeBSD__) || \ +defined(__NetBSD__) || \ jroelofs wrote: espositofulvio wrote: jroelofs wrote: jroelofs wrote: