From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2062) id 393D03833017; Wed, 9 Jun 2021 20:46:14 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 393D03833017 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Thomas Rodgers To: gcc-cvs@gcc.gnu.org, libstdc++-cvs@gcc.gnu.org Subject: [gcc r12-1350] libstd++: Only support atomic_ref::wait tests which are always lockfree X-Act-Checkin: gcc X-Git-Author: Thomas Rodgers X-Git-Refname: refs/heads/master X-Git-Oldrev: d4faa36e7540c573c5dc17850bcd938d0900b2e9 X-Git-Newrev: eb3a3bb8ce4ce2420919d03a3e9f9a2a4297c4d3 Message-Id: <20210609204614.393D03833017@sourceware.org> Date: Wed, 9 Jun 2021 20:46:14 +0000 (GMT) X-BeenThere: libstdc++-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 09 Jun 2021 20:46:14 -0000 https://gcc.gnu.org/g:eb3a3bb8ce4ce2420919d03a3e9f9a2a4297c4d3 commit r12-1350-geb3a3bb8ce4ce2420919d03a3e9f9a2a4297c4d3 Author: Thomas Rodgers Date: Wed Jun 9 13:44:31 2021 -0700 libstd++: Only support atomic_ref::wait tests which are always lockfree Fixes a regression on arm32 targets. libstdc++-v3/ChangeLog: * testsuite/29_atomics/atomic_ref/wait_notify.cc: Guard test logic with constexpr check for is_always_lock_free. Diff: --- .../testsuite/29_atomics/atomic_ref/wait_notify.cc | 25 ++++++++++++---------- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/libstdc++-v3/testsuite/29_atomics/atomic_ref/wait_notify.cc b/libstdc++-v3/testsuite/29_atomics/atomic_ref/wait_notify.cc index 003b86c85d0..b75e27617f7 100644 --- a/libstdc++-v3/testsuite/29_atomics/atomic_ref/wait_notify.cc +++ b/libstdc++-v3/testsuite/29_atomics/atomic_ref/wait_notify.cc @@ -30,17 +30,20 @@ template void test (S va, S vb) { - S aa{ va }; - S bb{ vb }; - std::atomic_ref a{ aa }; - a.wait(bb); - std::thread t([&] - { - a.store(bb); - a.notify_one(); - }); - a.wait(aa); - t.join(); + if constexpr (std::atomic_ref::is_always_lock_free) + { + S aa{ va }; + S bb{ vb }; + std::atomic_ref a{ aa }; + a.wait(bb); + std::thread t([&] + { + a.store(bb); + a.notify_one(); + }); + a.wait(aa); + t.join(); + } } int