Re: [PATCH] Add C++2a synchronization support

2020-08-03 Thread Jonathan Wakely via Gcc-patches
On 03/08/20 15:09 +0100, Jonathan Wakely wrote: On 05/06/20 17:29 -0700, Thomas Rodgers wrote: diff --git a/libstdc++-v3/include/bits/semaphore_base.h b/libstdc++-v3/include/bits/semaphore_base.h new file mode 100644 index 000..f0c4235d91c --- /dev/null +++ b/libstdc++-v3/include/bits/s

Re: [PATCH] Add C++2a synchronization support

2020-08-03 Thread Jonathan Wakely via Gcc-patches
On 05/06/20 17:29 -0700, Thomas Rodgers wrote: diff --git a/libstdc++-v3/include/bits/semaphore_base.h b/libstdc++-v3/include/bits/semaphore_base.h new file mode 100644 index 000..f0c4235d91c --- /dev/null +++ b/libstdc++-v3/include/bits/semaphore_base.h @@ -0,0 +1,272 @@ +// -*- C++ -*-

Re: [PATCH] Add C++2a synchronization support

2020-07-08 Thread Jonathan Wakely via Gcc-patches
On 05/06/20 17:29 -0700, Thomas Rodgers wrote: Add support for - atomic wait/notify_one/notify_all counting_semaphore binary_semaphore latch * include/Makefile.am (bits_headers): Add new header. * include/Makefile.in: Regenerate. * include/bits/

Re: [PATCH] Add C++2a synchronization support

2020-05-11 Thread Thomas Rodgers via Gcc-patches
I *think* I have addressed everything in the attached patch. commit 24a989d2bf2158bdbe2511310d0583d0c6226f71 Author: Thomas Rodgers Date: Mon Apr 6 17:58:47 2020 -0700 Add C++2a synchronization support Add support for - atomic wait/notify_one/notify_all coun

Re: [PATCH] Add C++2a synchronization support

2020-05-11 Thread Jonathan Wakely via Gcc-patches
On 11/05/20 08:43 -0700, Thomas Rodgers wrote: Jonathan Wakely writes: On 09/05/20 17:01 -0700, Thomas Rodgers via Libstdc++ wrote: +#include shouldn't be here (it adds runtime cost, as well as compile-time). Oversight, not removed after debugging it. Can't this just be __old i

Re: [PATCH] Add C++2a synchronization support

2020-05-11 Thread Thomas Rodgers via Gcc-patches
Jonathan Wakely writes: > On 09/05/20 17:01 -0700, Thomas Rodgers via Libstdc++ wrote: >>+#include > > shouldn't be here (it adds runtime cost, as well as > compile-time). > Oversight, not removed after debugging it. > > Can't this just be __old instead of *std::__addressof(__old) ? > Co

Re: [PATCH] Add C++2a synchronization support

2020-05-11 Thread Jonathan Wakely via Gcc-patches
On 09/05/20 17:01 -0700, Thomas Rodgers via Libstdc++ wrote: * Note, this patch supersedes my previous atomic wait and semaphore patches. Add support for - atomic wait/notify_one/notify_all counting_semaphore binary_semaphore latch * include/Makefile.am (bits_