Re: [Patch] Fortran/OpenMP: Add support for OpenMP 5.1 masked construct (was: Re: [committed] openmp: Add support for OpenMP 5.1 masked construct)

2021-08-16 Thread Jakub Jelinek via Gcc-patches
On Fri, Aug 13, 2021 at 08:52:34PM +0200, Tobias Burnus wrote: > gcc/fortran/ChangeLog: > > * dump-parse-tree.c (show_omp_clauses): Handle 'filter' clause. > (show_omp_node, show_code_node): Handle (combined) omp masked construct. > * frontend-passes.c (gfc_code_walker): Likewise

Re: [committed] openmp: Add support for OpenMP 5.1 masked construct

2021-08-13 Thread Thomas Schwinge
Hi! On 2021-08-12T22:48:33+0200, Jakub Jelinek via Gcc-patches wrote: > The patch also contains something I should have done much earlier, ... and as its own commit... ;-| > for clauses that accept some integral expression where we only care > about the value, forces during gimplification tha

Re: [Patch] Fortran/OpenMP: Add support for OpenMP 5.1 masked construct (was: Re: [committed] openmp: Add support for OpenMP 5.1 masked construct)

2021-08-13 Thread Tobias Burnus
On 13.08.21 16:37, Tobias Burnus wrote: When converting the C/C++ runtime testcase to Fortran, I did run into a bug: https://gcc.gnu.org/PR101899 (see PR or testcase; related to 'omp taskloop'.) I am any more sure whether it is a bug or not or what is the bug (see PR) – however, for this patch

[Patch] Fortran/OpenMP: Add support for OpenMP 5.1 masked construct (was: Re: [committed] openmp: Add support for OpenMP 5.1 masked construct)

2021-08-13 Thread Tobias Burnus
Hi all, hi Jakub, On 12.08.21 22:48, Jakub Jelinek via Gcc-patches wrote: This construct has been introduced as a replacement for master construct, but unlike that construct is slightly more general, has an optional clause which allows to choose which thread will be the one running the region, i

[committed] openmp: Add support for OpenMP 5.1 masked construct

2021-08-12 Thread Jakub Jelinek via Gcc-patches
Hi! This construct has been introduced as a replacement for master construct, but unlike that construct is slightly more general, has an optional clause which allows to choose which thread will be the one running the region, it can be some other thread than the master (primary) thread with number