Hi!

Given this:

On 2016-11-10T18:44:52+0800, Chung-Lin Tang <chunglin_t...@mentor.com> wrote:
> --- tree.c    (revision 241809)
> +++ tree.c    (working copy)
> @@ -327,7 +327,7 @@ unsigned const char omp_clause_num_ops[] =

> -  1, /* OMP_CLAUSE_TILE  */
> +  3, /* OMP_CLAUSE_TILE  */

... for this:

> --- tree.h    (revision 241809)
> +++ tree.h    (working copy)

>  #define OMP_CLAUSE_TILE_LIST(NODE) \
>    OMP_CLAUSE_OPERAND (OMP_CLAUSE_SUBCODE_CHECK (NODE, OMP_CLAUSE_TILE), 0)
> +#define OMP_CLAUSE_TILE_ITERVAR(NODE) \
> +  OMP_CLAUSE_OPERAND (OMP_CLAUSE_SUBCODE_CHECK (NODE, OMP_CLAUSE_TILE), 1)
> +#define OMP_CLAUSE_TILE_COUNT(NODE) \
> +  OMP_CLAUSE_OPERAND (OMP_CLAUSE_SUBCODE_CHECK (NODE, OMP_CLAUSE_TILE), 2)

..., we also need to "Fix 'OMP_CLAUSE_TILE' operands handling in
'gcc/tree.c:walk_tree_1'".  In
commit 92dc5d844a2088db79bc4521be3ecb4e2f284444 pushed to master branch,
cherry-picked in commit e6880aa976f962ecf78d20b58f7815b585791647 into
releases/gcc-11 branch, in
commit 82631dd97a3762e59bf5b9623f3c8c999aba7d80 into releases/gcc-10
branch, in commit 1514a668b96a9b66539646ec3d2a6ef9c6f39fb2 into
releases/gcc-9 branch, see attached.


Grüße
 Thomas


-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 
München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas 
Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht 
München, HRB 106955
>From 92dc5d844a2088db79bc4521be3ecb4e2f284444 Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <tho...@codesourcery.com>
Date: Fri, 27 Aug 2021 07:49:35 +0200
Subject: [PATCH] Fix 'OMP_CLAUSE_TILE' operands handling in
 'gcc/tree.c:walk_tree_1'

In r245300 (commit 02889d23ee3b02854dff203dd87b9a25e30b61b4)
"OpenACC tile clause support" that one had changed to three operands,
similar to 'OMP_CLAUSE_COLLAPSE'.

There is no (existing) test case where this seems to matter (likewise
for 'OMP_CLAUSE_COLLAPSE'), but it's good to be consistent.

	gcc/
	* tree.c (walk_tree_1) <OMP_CLAUSE_TILE>: Handle three operands.
---
 gcc/tree.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/tree.c b/gcc/tree.c
index cba3bca41b3..4c7e03b0f25 100644
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -11166,7 +11166,6 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	case OMP_CLAUSE_BIND:
 	case OMP_CLAUSE_AUTO:
 	case OMP_CLAUSE_SEQ:
-	case OMP_CLAUSE_TILE:
 	case OMP_CLAUSE__SIMT_:
 	case OMP_CLAUSE_IF_PRESENT:
 	case OMP_CLAUSE_FINALIZE:
@@ -11179,6 +11178,7 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	  WALK_SUBTREE_TAIL (OMP_CLAUSE_CHAIN (*tp));
 
 	case OMP_CLAUSE_COLLAPSE:
+	case OMP_CLAUSE_TILE:
 	  {
 	    int i;
 	    for (i = 0; i < 3; i++)
-- 
2.30.2

>From e6880aa976f962ecf78d20b58f7815b585791647 Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <tho...@codesourcery.com>
Date: Fri, 27 Aug 2021 07:49:35 +0200
Subject: [PATCH] Fix 'OMP_CLAUSE_TILE' operands handling in
 'gcc/tree.c:walk_tree_1'

In r245300 (commit 02889d23ee3b02854dff203dd87b9a25e30b61b4)
"OpenACC tile clause support" that one had changed to three operands,
similar to 'OMP_CLAUSE_COLLAPSE'.

There is no (existing) test case where this seems to matter (likewise
for 'OMP_CLAUSE_COLLAPSE'), but it's good to be consistent.

	gcc/
	* tree.c (walk_tree_1) <OMP_CLAUSE_TILE>: Handle three operands.

(cherry picked from commit 92dc5d844a2088db79bc4521be3ecb4e2f284444)
---
 gcc/tree.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/tree.c b/gcc/tree.c
index e0183b73e31..8bc81d66821 100644
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -12299,7 +12299,6 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	case OMP_CLAUSE_BIND:
 	case OMP_CLAUSE_AUTO:
 	case OMP_CLAUSE_SEQ:
-	case OMP_CLAUSE_TILE:
 	case OMP_CLAUSE__SIMT_:
 	case OMP_CLAUSE_IF_PRESENT:
 	case OMP_CLAUSE_FINALIZE:
@@ -12311,6 +12310,7 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	  WALK_SUBTREE_TAIL (OMP_CLAUSE_CHAIN (*tp));
 
 	case OMP_CLAUSE_COLLAPSE:
+	case OMP_CLAUSE_TILE:
 	  {
 	    int i;
 	    for (i = 0; i < 3; i++)
-- 
2.30.2

>From 82631dd97a3762e59bf5b9623f3c8c999aba7d80 Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <tho...@codesourcery.com>
Date: Fri, 27 Aug 2021 07:49:35 +0200
Subject: [PATCH] Fix 'OMP_CLAUSE_TILE' operands handling in
 'gcc/tree.c:walk_tree_1'

In r245300 (commit 02889d23ee3b02854dff203dd87b9a25e30b61b4)
"OpenACC tile clause support" that one had changed to three operands,
similar to 'OMP_CLAUSE_COLLAPSE'.

There is no (existing) test case where this seems to matter (likewise
for 'OMP_CLAUSE_COLLAPSE'), but it's good to be consistent.

	gcc/
	* tree.c (walk_tree_1) <OMP_CLAUSE_TILE>: Handle three operands.

(cherry picked from commit 92dc5d844a2088db79bc4521be3ecb4e2f284444)
---
 gcc/tree.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/tree.c b/gcc/tree.c
index b43bc809823..d82c308f14c 100644
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -12191,7 +12191,6 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	case OMP_CLAUSE_BIND:
 	case OMP_CLAUSE_AUTO:
 	case OMP_CLAUSE_SEQ:
-	case OMP_CLAUSE_TILE:
 	case OMP_CLAUSE__SIMT_:
 	case OMP_CLAUSE_IF_PRESENT:
 	case OMP_CLAUSE_FINALIZE:
@@ -12203,6 +12202,7 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	  WALK_SUBTREE_TAIL (OMP_CLAUSE_CHAIN (*tp));
 
 	case OMP_CLAUSE_COLLAPSE:
+	case OMP_CLAUSE_TILE:
 	  {
 	    int i;
 	    for (i = 0; i < 3; i++)
-- 
2.30.2

>From 1514a668b96a9b66539646ec3d2a6ef9c6f39fb2 Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <tho...@codesourcery.com>
Date: Fri, 27 Aug 2021 07:49:35 +0200
Subject: [PATCH] Fix 'OMP_CLAUSE_TILE' operands handling in
 'gcc/tree.c:walk_tree_1'

In r245300 (commit 02889d23ee3b02854dff203dd87b9a25e30b61b4)
"OpenACC tile clause support" that one had changed to three operands,
similar to 'OMP_CLAUSE_COLLAPSE'.

There is no (existing) test case where this seems to matter (likewise
for 'OMP_CLAUSE_COLLAPSE'), but it's good to be consistent.

	gcc/
	* tree.c (walk_tree_1) <OMP_CLAUSE_TILE>: Handle three operands.

(cherry picked from commit 92dc5d844a2088db79bc4521be3ecb4e2f284444)
---
 gcc/tree.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/tree.c b/gcc/tree.c
index be311754393..99f4318306a 100644
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -12340,7 +12340,6 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	case OMP_CLAUSE_DEFAULTMAP:
 	case OMP_CLAUSE_AUTO:
 	case OMP_CLAUSE_SEQ:
-	case OMP_CLAUSE_TILE:
 	case OMP_CLAUSE__SIMT_:
 	case OMP_CLAUSE_IF_PRESENT:
 	case OMP_CLAUSE_FINALIZE:
@@ -12352,6 +12351,7 @@ walk_tree_1 (tree *tp, walk_tree_fn func, void *data,
 	  WALK_SUBTREE_TAIL (OMP_CLAUSE_CHAIN (*tp));
 
 	case OMP_CLAUSE_COLLAPSE:
+	case OMP_CLAUSE_TILE:
 	  {
 	    int i;
 	    for (i = 0; i < 3; i++)
-- 
2.30.2

Reply via email to