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