]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
libstdc++: Ignore _GLIBCXX_USE_POSIX_SEMAPHORE if not supported [PR116992]
authorJonathan Wakely <jwakely@redhat.com>
Mon, 7 Oct 2024 09:22:24 +0000 (10:22 +0100)
committerJonathan Wakely <redi@gcc.gnu.org>
Wed, 9 Oct 2024 12:41:06 +0000 (13:41 +0100)
If _GLIBCXX_HAVE_POSIX_SEMAPHRE is undefined then users get an error
when defining _GLIBCXX_USE_POSIX_SEMAPHORE. We can just ignore it
instead (and warn them it's being ignored).

This fixes a testsuite failure on hppa64-hp-hpux11.11 (and probably some
other targets):

FAIL: 30_threads/semaphore/platform_try_acquire_for.cc  -std=gnu++20 (test for excess errors)
Excess errors:
semaphore:49: error: '__semaphore_impl' has not been declared

libstdc++-v3/ChangeLog:

PR libstdc++/116992
* include/bits/semaphore_base.h (_GLIBCXX_USE_POSIX_SEMAPHORE):
Undefine and issue a warning if POSIX sem_t is not supported.
* testsuite/30_threads/semaphore/platform_try_acquire_for.cc:
Prune new warning.

libstdc++-v3/include/bits/semaphore_base.h
libstdc++-v3/testsuite/30_threads/semaphore/platform_try_acquire_for.cc

index 9d73b37e60a161fd647bcc8dc8d90dbf0eae785b..dd16d2c92498a4e05960ca84bcfb723700911ffd 100644 (file)
@@ -45,6 +45,9 @@
 # include <cerrno>     // errno, EINTR, EAGAIN etc.
 # include <limits.h>   // SEM_VALUE_MAX
 # include <semaphore.h>        // sem_t, sem_init, sem_wait, sem_post etc.
+#elif defined(_GLIBCXX_USE_POSIX_SEMAPHORE)
+# warning "POSIX semaphore not available, ignoring _GLIBCXX_USE_POSIX_SEMAPHORE"
+# undef _GLIBCXX_USE_POSIX_SEMAPHORE
 #endif
 
 namespace std _GLIBCXX_VISIBILITY(default)
index bf6cd142bf01e5d457ce5db6a8e339128cfd3984..6d90564ea8acb22b5c7c10f0be8fe9e5453fecaf 100644 (file)
@@ -5,3 +5,5 @@
 // { dg-add-options libatomic }
 
 #include "try_acquire_for.cc"
+
+// { dg-prune-output "ignoring _GLIBCXX_USE_POSIX_SEMAPHORE" }