I will commit the attached patch in a while – unless last-minute
comments or other issues show up.
On 12.08.21 13:31, Jakub Jelinek wrote:
LGTM, some nits below.
Maybe in tree-core.h do:
- OMP_CLAUSE_PROC_BIND_MASTER = 2,
+ OMP_CLAUSE_PROC_BIND_PRIMARY = 2,
+ OMP_CLAUSE_PROC_BIND_MASTER = OMP
On 12.08.21 14:13, Hans-Peter Nilsson via Fortran wrote:
I'd call it obvious, so i dare to approve it.
OK.
thanks!
Thanks, but not coming from a testsuite or fortran
maintainer I'm not sure I can actually rely on that.
OTOH, damn the torpedoes. Committed.
If it helps: A post-commit LGTM from
> From: Bernhard Reutner-Fischer
> Date: Thu, 12 Aug 2021 09:03:50 +0200
> On Thu, 12 Aug 2021 00:09:21 +0200
> Hans-Peter Nilsson via Fortran wrote:
>
> > I had a file-path to sources with the substring "new" in it,
> > and (only) this test regressed compared to results from
> > another build
On Thu, Aug 12, 2021 at 12:52:17PM +0200, Tobias Burnus wrote:
> gcc/c/ChangeLog:
>
> * c-parser.c (c_parser_omp_clause_proc_bind): Accept
> 'primary' as alias for 'master'.
>
> gcc/cp/ChangeLog:
>
> * parser.c (cp_parser_omp_clause_proc_bind): Accept
> 'primary' as alias
The attached patch adds another (very) low-hanging fruit of OpenMP 5.1
to GCC, given that we already have one OpenMP 5.1 feature and another
one also related to 'master'/'masked' construct might be added soon.
OK?
Tobias
-
Siemens Electronic Design Automation GmbH; Anschrift: Ar
On Thu, 12 Aug 2021 00:09:21 +0200
Hans-Peter Nilsson via Fortran wrote:
> I had a file-path to sources with the substring "new" in it,
> and (only) this test regressed compared to results from
> another build without "new" in the name.
>
> The test does
> ! { dg-final { scan-tree-dump-times "n
