* [PATCH] [testsuite] require pthread for openmp
@ 2023-05-24 5:19 Alexandre Oliva
2023-05-24 9:03 ` Richard Biener
0 siblings, 1 reply; 2+ messages in thread
From: Alexandre Oliva @ 2023-05-24 5:19 UTC (permalink / raw)
To: gcc-patches; +Cc: Rainer Orth, Mike Stump
Fix test that uses -fopenmp without declaring requirement for pthread
support.
Bootstrapped on x86_64-linux-gnu. Also tested on ppc- and x86-vx7r2
with gcc-12.
for gcc/testsuite/ChangeLog
* g++.dg/pr80481.C: Add explicit pthread requirement.
---
gcc/testsuite/g++.dg/pr80481.C | 2 ++
1 file changed, 2 insertions(+)
diff --git a/gcc/testsuite/g++.dg/pr80481.C b/gcc/testsuite/g++.dg/pr80481.C
index 78c463b8e3b58..3a8869914634f 100644
--- a/gcc/testsuite/g++.dg/pr80481.C
+++ b/gcc/testsuite/g++.dg/pr80481.C
@@ -1,4 +1,6 @@
// { dg-do compile { target { i?86-*-* x86_64-*-* } && { ! *-*-solaris* } } }
+// -fopenmp implies -pthread
+// { dg-require-effective-target pthread }
// { dg-options "-Ofast -funroll-loops -fopenmp -march=knl" }
// Disabling epilogues until we find a better way to deal with scans.
// { dg-additional-options "--param vect-epilogues-nomask=0" }
--
Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/
Free Software Activist GNU Toolchain Engineer
Disinformation flourishes because many people care deeply about injustice
but very few check the facts. Ask me about <https://stallmansupport.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] [testsuite] require pthread for openmp
2023-05-24 5:19 [PATCH] [testsuite] require pthread for openmp Alexandre Oliva
@ 2023-05-24 9:03 ` Richard Biener
0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2023-05-24 9:03 UTC (permalink / raw)
To: Alexandre Oliva; +Cc: gcc-patches, Rainer Orth, Mike Stump
On Wed, May 24, 2023 at 7:20 AM Alexandre Oliva via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
>
> Fix test that uses -fopenmp without declaring requirement for pthread
> support.
>
> Bootstrapped on x86_64-linux-gnu. Also tested on ppc- and x86-vx7r2
> with gcc-12.
OK
> for gcc/testsuite/ChangeLog
>
> * g++.dg/pr80481.C: Add explicit pthread requirement.
> ---
> gcc/testsuite/g++.dg/pr80481.C | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/gcc/testsuite/g++.dg/pr80481.C b/gcc/testsuite/g++.dg/pr80481.C
> index 78c463b8e3b58..3a8869914634f 100644
> --- a/gcc/testsuite/g++.dg/pr80481.C
> +++ b/gcc/testsuite/g++.dg/pr80481.C
> @@ -1,4 +1,6 @@
> // { dg-do compile { target { i?86-*-* x86_64-*-* } && { ! *-*-solaris* } } }
> +// -fopenmp implies -pthread
> +// { dg-require-effective-target pthread }
> // { dg-options "-Ofast -funroll-loops -fopenmp -march=knl" }
> // Disabling epilogues until we find a better way to deal with scans.
> // { dg-additional-options "--param vect-epilogues-nomask=0" }
>
> --
> Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/
> Free Software Activist GNU Toolchain Engineer
> Disinformation flourishes because many people care deeply about injustice
> but very few check the facts. Ask me about <https://stallmansupport.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-05-24 9:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-24 5:19 [PATCH] [testsuite] require pthread for openmp Alexandre Oliva
2023-05-24 9:03 ` Richard Biener
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).