Hi!

On Wed, 26 Feb 2014 19:32:35 +0400, Ilmir Usmanov <i.usma...@samsung.com> wrote:
> This patch is pretty obvious. Currently ASYNC clause cannot have 
> integer-expression-list. Patch fixes this.
> 
> OK for gomp4 branch?

> From df76a29ebf869687209d7a606e243624cc136dbc Mon Sep 17 00:00:00 2001
> From: Ilmir Usmanov <i.usma...@samsung.com>
> Date: Wed, 26 Feb 2014 19:04:47 +0400
> Subject: [PATCH 4/5] Fix ASYNC
> 
> ---
>  gcc/gimplify.c  | 4 ++--
>  gcc/omp-low.c   | 4 ++--
>  gcc/tree-core.h | 6 +++---
>  gcc/tree.c      | 6 +++---
>  4 files changed, 10 insertions(+), 10 deletions(-)

Sure, with ChangeLog added, and the following changes folded in, I think:

> --- a/gcc/gimplify.c
> +++ b/gcc/gimplify.c
> @@ -6545,11 +6545,11 @@ gimplify_adjust_omp_clauses (tree *list_p)
>       case OMP_CLAUSE_OACC_DEVICE:
>       case OMP_CLAUSE_DEVICE_RESIDENT:
>       case OMP_CLAUSE_USE_DEVICE:
> +     case OMP_CLAUSE_ASYNC:
>       case OMP_CLAUSE_GANG:
>       case OMP_CLAUSE_WAIT:

GANG/ASYNC/WAIT, for consistency?

> --- a/gcc/tree.c
> +++ b/gcc/tree.c
> @@ -263,6 +263,7 @@ unsigned const char omp_clause_num_ops[] =
>    1, /* OMP_CLAUSE_OACC_DEVICE  */
>    1, /* OMP_CLAUSE_DEVICE_RESIDENT  */
>    1, /* OMP_CLAUSE_USE_DEVICE  */
> +  1, /* OMP_CLAUSE_ASYNC  */
>    1, /* OMP_CLAUSE_GANG  */
>    1, /* OMP_CLAUSE_WAIT  */

Likewise.

For consistency, also move OMP_CLAUSE_ASYNC_EXPR in gcc/tree.h, and also
adapt OMP_CLAUSE_ASYNC in gcc/tree-pretty-print.c:dump_omp_clause?


Grüße,
 Thomas

Attachment: pgpQJ0AeLLvJc.pgp
Description: PGP signature

Reply via email to