Hi!
On Wed, 27 Jul 2016 17:17:49 +0200, I wrote:
> I found that for a lot of OpenACC (and potentially also OpenMP) clauses
> (in C/C++ front ends; didn't look at Fortran), we use wrong location
> information. The problem is [...]
Haven't been able yet to allocate the time to incorporate David's
(testsuite) and Jeff's (further cleanup) suggestions (thanks!); to
unblock other work, for now committed in r239519 to gomp-4_0-branch as
follows:
commit fbe4b73c962dc88dae4318474711cd2ea48341fe
Author: tschwinge <tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Wed Aug 17 00:54:39 2016 +0000
Use correct location information for OpenACC shape and simple clauses in
C/C++
gcc/c/
* c-parser.c (c_parser_oacc_shape_clause)
(c_parser_oacc_simple_clause): Add loc formal parameter. Adjust
all users.
gcc/cp/
* parser.c (cp_parser_oacc_shape_clause): Add loc formal
parameter. Adjust all users.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/branches/gomp-4_0-branch@239519
138bc75d-0d04-0410-961f-82ee72b054a4
---
gcc/c/ChangeLog.gomp | 6 ++++++
gcc/c/c-parser.c | 29 +++++++++++++++--------------
gcc/cp/ChangeLog.gomp | 5 +++++
gcc/cp/parser.c | 12 +++++++-----
4 files changed, 33 insertions(+), 19 deletions(-)
diff --git gcc/c/ChangeLog.gomp gcc/c/ChangeLog.gomp
index a0f6258..d9725b0 100644
--- gcc/c/ChangeLog.gomp
+++ gcc/c/ChangeLog.gomp
@@ -1,3 +1,9 @@
+2016-08-17 Thomas Schwinge <tho...@codesourcery.com>
+
+ * c-parser.c (c_parser_oacc_shape_clause)
+ (c_parser_oacc_simple_clause): Add loc formal parameter. Adjust
+ all users.
+
2016-08-04 Thomas Schwinge <tho...@codesourcery.com>
Backport trunk r239128:
diff --git gcc/c/c-parser.c gcc/c/c-parser.c
index 1293d56..2e38807 100644
--- gcc/c/c-parser.c
+++ gcc/c/c-parser.c
@@ -11595,12 +11595,12 @@ c_parser_omp_clause_num_workers (c_parser *parser,
tree list)
*/
static tree
-c_parser_oacc_shape_clause (c_parser *parser, omp_clause_code kind,
+c_parser_oacc_shape_clause (c_parser *parser, location_t loc,
+ omp_clause_code kind,
const char *str, tree list)
{
const char *id = "num";
tree ops[2] = { NULL_TREE, NULL_TREE }, c;
- location_t loc = c_parser_peek_token (parser)->location;
if (kind == OMP_CLAUSE_VECTOR)
id = "length";
@@ -11730,12 +11730,12 @@ c_parser_oacc_shape_clause (c_parser *parser,
omp_clause_code kind,
seq */
static tree
-c_parser_oacc_simple_clause (c_parser *parser, enum omp_clause_code code,
- tree list)
+c_parser_oacc_simple_clause (c_parser * /* parser */, location_t loc,
+ enum omp_clause_code code, tree list)
{
check_no_duplicate_clause (list, code, omp_clause_code_name[code]);
- tree c = build_omp_clause (c_parser_peek_token (parser)->location, code);
+ tree c = build_omp_clause (loc, code);
OMP_CLAUSE_CHAIN (c) = list;
return c;
@@ -13184,7 +13184,7 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
c_name = "async";
break;
case PRAGMA_OACC_CLAUSE_AUTO:
- clauses = c_parser_oacc_simple_clause (parser, OMP_CLAUSE_AUTO,
+ clauses = c_parser_oacc_simple_clause (parser, here, OMP_CLAUSE_AUTO,
clauses);
c_name = "auto";
break;
@@ -13244,7 +13244,7 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
break;
case PRAGMA_OACC_CLAUSE_GANG:
c_name = "gang";
- clauses = c_parser_oacc_shape_clause (parser, OMP_CLAUSE_GANG,
+ clauses = c_parser_oacc_shape_clause (parser, here, OMP_CLAUSE_GANG,
c_name, clauses);
break;
case PRAGMA_OACC_CLAUSE_HOST:
@@ -13256,8 +13256,9 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
c_name = "if";
break;
case PRAGMA_OACC_CLAUSE_INDEPENDENT:
- clauses = c_parser_oacc_simple_clause (parser, OMP_CLAUSE_INDEPENDENT,
- clauses);
+ clauses = c_parser_oacc_simple_clause (parser, here,
+ OMP_CLAUSE_INDEPENDENT,
+ clauses);
c_name = "independent";
break;
case PRAGMA_OACC_CLAUSE_LINK:
@@ -13265,8 +13266,8 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
c_name = "link";
break;
case PRAGMA_OACC_CLAUSE_NOHOST:
- clauses = c_parser_oacc_simple_clause (parser, OMP_CLAUSE_NOHOST,
- clauses);
+ clauses = c_parser_oacc_simple_clause (parser, here,
+ OMP_CLAUSE_NOHOST, clauses);
c_name = "nohost";
break;
case PRAGMA_OACC_CLAUSE_NUM_GANGS:
@@ -13306,7 +13307,7 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
c_name = "reduction";
break;
case PRAGMA_OACC_CLAUSE_SEQ:
- clauses = c_parser_oacc_simple_clause (parser, OMP_CLAUSE_SEQ,
+ clauses = c_parser_oacc_simple_clause (parser, here, OMP_CLAUSE_SEQ,
clauses);
c_name = "seq";
break;
@@ -13320,7 +13321,7 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
break;
case PRAGMA_OACC_CLAUSE_VECTOR:
c_name = "vector";
- clauses = c_parser_oacc_shape_clause (parser, OMP_CLAUSE_VECTOR,
+ clauses = c_parser_oacc_shape_clause (parser, here, OMP_CLAUSE_VECTOR,
c_name, clauses);
break;
case PRAGMA_OACC_CLAUSE_VECTOR_LENGTH:
@@ -13333,7 +13334,7 @@ c_parser_oacc_all_clauses (c_parser *parser,
omp_clause_mask mask,
break;
case PRAGMA_OACC_CLAUSE_WORKER:
c_name = "worker";
- clauses = c_parser_oacc_shape_clause (parser, OMP_CLAUSE_WORKER,
+ clauses = c_parser_oacc_shape_clause (parser, here, OMP_CLAUSE_WORKER,
c_name, clauses);
break;
default:
diff --git gcc/cp/ChangeLog.gomp gcc/cp/ChangeLog.gomp
index f8e9f3f..feb186a 100644
--- gcc/cp/ChangeLog.gomp
+++ gcc/cp/ChangeLog.gomp
@@ -1,3 +1,8 @@
+2016-08-17 Thomas Schwinge <tho...@codesourcery.com>
+
+ * parser.c (cp_parser_oacc_shape_clause): Add loc formal
+ parameter. Adjust all users.
+
2016-08-04 Thomas Schwinge <tho...@codesourcery.com>
Backport trunk r239128:
diff --git gcc/cp/parser.c gcc/cp/parser.c
index a16898d..d3d673b 100644
--- gcc/cp/parser.c
+++ gcc/cp/parser.c
@@ -30252,13 +30252,13 @@ cp_parser_oacc_single_int_clause (cp_parser *parser,
omp_clause_code code,
*/
static tree
-cp_parser_oacc_shape_clause (cp_parser *parser, omp_clause_code kind,
+cp_parser_oacc_shape_clause (cp_parser *parser, location_t loc,
+ omp_clause_code kind,
const char *str, tree list)
{
const char *id = "num";
cp_lexer *lexer = parser->lexer;
tree ops[2] = { NULL_TREE, NULL_TREE }, c;
- location_t loc = cp_lexer_peek_token (lexer)->location;
if (kind == OMP_CLAUSE_VECTOR)
id = "length";
@@ -32294,7 +32294,7 @@ cp_parser_oacc_all_clauses (cp_parser *parser,
omp_clause_mask mask,
break;
case PRAGMA_OACC_CLAUSE_GANG:
c_name = "gang";
- clauses = cp_parser_oacc_shape_clause (parser, OMP_CLAUSE_GANG,
+ clauses = cp_parser_oacc_shape_clause (parser, here, OMP_CLAUSE_GANG,
c_name, clauses);
break;
case PRAGMA_OACC_CLAUSE_HOST:
@@ -32377,7 +32377,8 @@ cp_parser_oacc_all_clauses (cp_parser *parser,
omp_clause_mask mask,
break;
case PRAGMA_OACC_CLAUSE_VECTOR:
c_name = "vector";
- clauses = cp_parser_oacc_shape_clause (parser, OMP_CLAUSE_VECTOR,
+ clauses = cp_parser_oacc_shape_clause (parser, here,
+ OMP_CLAUSE_VECTOR,
c_name, clauses);
break;
case PRAGMA_OACC_CLAUSE_VECTOR_LENGTH:
@@ -32392,7 +32393,8 @@ cp_parser_oacc_all_clauses (cp_parser *parser,
omp_clause_mask mask,
break;
case PRAGMA_OACC_CLAUSE_WORKER:
c_name = "worker";
- clauses = cp_parser_oacc_shape_clause (parser, OMP_CLAUSE_WORKER,
+ clauses = cp_parser_oacc_shape_clause (parser, here,
+ OMP_CLAUSE_WORKER,
c_name, clauses);
break;
default:
Grüße
Thomas
