From: James Norris <jnorris@codesourcery.com>
To: <gcc-patches@gcc.gnu.org>
Subject: [PATCH] Fix PR64748
Date: Mon, 16 Feb 2015 18:26:00 -0000 [thread overview]
Message-ID: <54E23658.6060105@codesourcery.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 155 bytes --]
This fixes the validation of the argument to the deviceptr clause.
Bootstrapped and regtested on x86_64-unknown-linux-gnu.
OK to commit to trunk?
Jim
[-- Attachment #2: deviceptr.patch --]
[-- Type: text/x-patch, Size: 6109 bytes --]
diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c
index ceb9e1a..9f0d7af 100644
--- a/gcc/c/c-parser.c
+++ b/gcc/c/c-parser.c
@@ -10334,11 +10334,11 @@ c_parser_oacc_data_clause_deviceptr (c_parser *parser, tree list)
c_parser_omp_var_list_parens() should construct a list of
locations to go along with the var list. */
- if (TREE_CODE (v) != VAR_DECL)
- error_at (loc, "%qD is not a variable", v);
- else if (TREE_TYPE (v) == error_mark_node)
+ if (TREE_TYPE (v) == error_mark_node)
;
- else if (!POINTER_TYPE_P (TREE_TYPE (v)))
+ else if ((TREE_CODE (v) != VAR_DECL ||
+ TREE_CODE (v) != PARM_DECL) &&
+ !POINTER_TYPE_P (TREE_TYPE (v)))
error_at (loc, "%qD is not a pointer variable", v);
tree u = build_omp_clause (loc, OMP_CLAUSE_MAP);
diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c
index 57dfbcc..37b4712 100644
--- a/gcc/cp/parser.c
+++ b/gcc/cp/parser.c
@@ -27988,11 +27988,11 @@ cp_parser_oacc_data_clause_deviceptr (cp_parser *parser, tree list)
c_parser_omp_var_list_parens should construct a list of
locations to go along with the var list. */
- if (TREE_CODE (v) != VAR_DECL)
- error_at (loc, "%qD is not a variable", v);
- else if (TREE_TYPE (v) == error_mark_node)
+ if (TREE_TYPE (v) == error_mark_node)
;
- else if (!POINTER_TYPE_P (TREE_TYPE (v)))
+ else if ((TREE_CODE (v) != VAR_DECL ||
+ TREE_CODE (v) != PARM_DECL) &&
+ !POINTER_TYPE_P (TREE_TYPE (v)))
error_at (loc, "%qD is not a pointer variable", v);
tree u = build_omp_clause (loc, OMP_CLAUSE_MAP);
diff --git a/gcc/testsuite/c-c++-common/goacc/deviceptr-1.c b/gcc/testsuite/c-c++-common/goacc/deviceptr-1.c
index 546fa82..5ec7540 100644
--- a/gcc/testsuite/c-c++-common/goacc/deviceptr-1.c
+++ b/gcc/testsuite/c-c++-common/goacc/deviceptr-1.c
@@ -8,27 +8,29 @@ fun1 (void)
#pragma acc kernels deviceptr(u[0:4]) /* { dg-error "expected '\\\)' before '\\\[' token" } */
;
-#pragma acc data deviceptr(fun1) /* { dg-error "'fun1' is not a variable" } */
+#pragma acc data deviceptr(fun1) /* { dg-error "'fun1' is not a pointer variable" } */
+ /* { dg-error "'fun1' is not a variable in 'map' clause" "fun1 is not a varialbe in map clause" { target *-*-* } 11 } */
;
#pragma acc parallel deviceptr(fun1[2:5])
- /* { dg-error "'fun1' is not a variable" "not a variable" { target *-*-* } 13 } */
- /* { dg-error "expected '\\\)' before '\\\[' token" "array" { target *-*-* } 13 } */
+ /* { dg-error "'fun1' is not a pointer variable" "not a pointer variable" { target *-*-* } 14 } */
+ /* { dg-error "expected '\\\)' before '\\\[' token" "array" { target *-*-* } 14 } */
+ /* { dg-error "'fun1' is not a variable in 'map' clause" "fun1 is not a varialbe in map clause" { target *-*-* } 14 } */
;
int i;
#pragma acc kernels deviceptr(i) /* { dg-error "'i' is not a pointer variable" } */
;
#pragma acc data deviceptr(i[0:4])
- /* { dg-error "'i' is not a pointer variable" "not a pointer variable" { target *-*-* } 21 } */
- /* { dg-error "expected '\\\)' before '\\\[' token" "array" { target *-*-* } 21 } */
+ /* { dg-error "'i' is not a pointer variable" "not a pointer variable" { target *-*-* } 23 } */
+ /* { dg-error "expected '\\\)' before '\\\[' token" "array" { target *-*-* } 23 } */
;
float fa[10];
#pragma acc parallel deviceptr(fa) /* { dg-error "'fa' is not a pointer variable" } */
;
#pragma acc kernels deviceptr(fa[1:5])
- /* { dg-error "'fa' is not a pointer variable" "not a pointer variable" { target *-*-* } 29 } */
- /* { dg-error "expected '\\\)' before '\\\[' token" "array" { target *-*-* } 29 } */
+ /* { dg-error "'fa' is not a pointer variable" "not a pointer variable" { target *-*-* } 31 } */
+ /* { dg-error "expected '\\\)' before '\\\[' token" "array" { target *-*-* } 31 } */
;
float *fp;
@@ -44,10 +46,11 @@ fun2 (void)
int i;
float *fp;
#pragma acc kernels deviceptr(fp,u,fun2,i,fp)
- /* { dg-error "'u' undeclared" "u undeclared" { target *-*-* } 46 } */
- /* { dg-error "'fun2' is not a variable" "fun2 not a variable" { target *-*-* } 46 } */
- /* { dg-error "'i' is not a pointer variable" "i not a pointer variable" { target *-*-* } 46 } */
- /* { dg-error "'fp' appears more than once in map clauses" "fp more than once" { target *-*-* } 46 } */
+ /* { dg-error "'u' undeclared" "u undeclared" { target *-*-* } 48 } */
+ /* { dg-error "'fun2' is not a pointer variable" "fun2 not a pointer variable" { target *-*-* } 48 } */
+ /* { dg-error "'i' is not a pointer variable" "i not a pointer variable" { target *-*-* } 48 } */
+ /* { dg-error "'fp' appears more than once in map clauses" "fp more than once" { target *-*-* } 48 } */
+ /* { dg-error "'fun2' is not a variable in 'map' clause" "fun2 is not a variable in map clause" { target *-*-* } 48 } */
;
}
diff --git a/libgomp/testsuite/libgomp.oacc-c-c++-common/deviceptr-1.c b/libgomp/testsuite/libgomp.oacc-c-c++-common/deviceptr-1.c
index e271a37..5443433 100644
--- a/libgomp/testsuite/libgomp.oacc-c-c++-common/deviceptr-1.c
+++ b/libgomp/testsuite/libgomp.oacc-c-c++-common/deviceptr-1.c
@@ -1,10 +1,26 @@
/* { dg-do run } */
+#include <string.h>
#include <stdlib.h>
+#include <openacc.h>
+
+void foo (int *b, const int N)
+{
+ int i;
+
+#pragma acc parallel loop deviceptr (b)
+ for (i = 0; i < N; i++)
+ b[i] = i;
+
+ return;
+}
int main (void)
{
void *a, *a_1, *a_2;
+ int *b;
+ const int N = 32;
+ int i;
#define A (void *) 0x123
a = A;
@@ -28,5 +44,35 @@ int main (void)
abort ();
#endif
+#if !ACC_MEM_SHARED
+ b = (int *) malloc (N * sizeof (int));
+
+ memset (b, 0, (N * sizeof (int)));
+
+#pragma acc data copy (b[0:N])
+ {
+#pragma acc parallel loop deviceptr (b)
+ for (i = 0; i < N; i++)
+ b[i] = i;
+ }
+
+ for (i = 0; i < N; i++)
+ {
+ if (b[i] != i)
+ abort ();
+ }
+
+#pragma acc data copy (b[0:N])
+ {
+ foo ((int *) acc_deviceptr (b), N);
+ }
+
+ for (i = 0; i < N; i++)
+ {
+ if (b[i] != i)
+ abort ();
+ }
+#endif
+
return 0;
}
next prev reply other threads:[~2015-02-16 18:26 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-01 21:25 [gomp4] Allow parameter declarations with deviceptr James Norris
2015-07-01 23:33 ` Cesar Philippidis
2015-02-16 18:26 ` James Norris [this message]
2015-03-10 13:37 ` [PATCH] Fix PR64748 James Norris
2015-03-10 18:48 ` Jeff Law
2015-03-13 15:24 ` Thomas Schwinge
2015-03-13 15:28 ` Jakub Jelinek
2015-03-13 15:20 ` [PATCH] Fix PR64748: OpenACC: "is not a variable" error with deviceptr() Thomas Schwinge
2015-07-07 14:01 ` [gomp4] Allow parameter declarations with deviceptr Thomas Schwinge
2016-02-01 19:41 [PATCH] Fix PR64748 James Norris
2016-02-01 20:03 ` Jakub Jelinek
2016-02-02 14:51 ` James Norris
2016-02-15 13:27 ` James Norris
2016-02-15 13:41 ` Jakub Jelinek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54E23658.6060105@codesourcery.com \
--to=jnorris@codesourcery.com \
--cc=gcc-patches@gcc.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).