This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PING: Fwd: Re: [patch] implement Cilk Plus simd loops on trunk
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: Aldy Hernandez <aldyh at redhat dot com>, Jason Merrill <jason at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, "Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>
- Date: Mon, 17 Mar 2014 17:54:58 +0100
- Subject: Re: PING: Fwd: Re: [patch] implement Cilk Plus simd loops on trunk
- Authentication-results: sourceware.org; auth=none
- References: <52740D59 dot 50104 at redhat dot com> <5284FF25 dot 7020401 at redhat dot com> <20131114170553 dot GD27813 at tucnak dot zalov dot cz> <52850AB6 dot 2070408 at redhat dot com> <20131114180147 dot GE27813 at tucnak dot zalov dot cz> <5285790E dot 9050000 at redhat dot com> <20131115073757 dot GA892 at tucnak dot redhat dot com> <52863CB4 dot 50900 at redhat dot com> <528695CD dot 5080705 at redhat dot com> <87bnxhlpyb dot fsf at kepler dot schwinge dot homeip dot net>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Mar 07, 2014 at 09:21:48PM +0100, Thomas Schwinge wrote:
> Maybe it's just too late on a Friday evening, but I don't understand this
> change, part of r204863. GF_OMP_FOR_KIND_FOR has the value zero;
> shouldn't this comparison have remained unchanged? Is the following
> (untested) patch OK for trunk? Does this need a test case?
>
> commit f3c7834ecbedc50e04223d24b1b671fc8a62c169
> Author: Thomas Schwinge <thomas@codesourcery.com>
> Date: Fri Mar 7 21:11:43 2014 +0100
>
> Restore check for OpenMP for construct.
>
> gcc/
> * omp-low.c (lower_rec_input_clauses) <build_omp_barrier>: Restore
> check for GF_OMP_FOR_KIND_FOR.
Ok for trunk, sorry for the delay.
> diff --git gcc/omp-low.c gcc/omp-low.c
> index 4dc3956..713a4ae 100644
> --- gcc/omp-low.c
> +++ gcc/omp-low.c
> @@ -3915,7 +3915,7 @@ lower_rec_input_clauses (tree clauses, gimple_seq *ilist, gimple_seq *dlist,
> /* Don't add any barrier for #pragma omp simd or
> #pragma omp distribute. */
> if (gimple_code (ctx->stmt) != GIMPLE_OMP_FOR
> - || gimple_omp_for_kind (ctx->stmt) & GF_OMP_FOR_KIND_FOR)
> + || gimple_omp_for_kind (ctx->stmt) == GF_OMP_FOR_KIND_FOR)
> gimple_seq_add_stmt (ilist, build_omp_barrier (NULL_TREE));
> }
>
Jakub