procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

re PR c++/31748 (bad diagnostic for invalid private clause)

Browse source 
PR c++/31748
	* semantics.c (finish_omp_clauses): Use %qD instead of %qE for
	DECL_P in not a variable and appears more than once error messages.

	* g++.dg/gomp/pr31748.C: New test.

From-SVN: r126201
This commit is contained in:
Jakub Jelinek 2007-07-02 14:59:49 +02:00 • committed by Jakub Jelinek
parent a2daf82cc2
commit 76dc15d453
4 changed files with 24 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
gcc/cp/ChangeLog
View file

@ -1,3 +1,9 @@
2007-07-02 Jakub Jelinek <jakub@redhat.com>
PR c++/31748
* semantics.c (finish_omp_clauses): Use %qD instead of %qE for
DECL_P in not a variable and appears more than once error messages.
2007-07-01 Ollie Wild <aaw@google.com>
* name-lookup.c (ambiguous_decl): Fix case when new->value is hidden.

7
gcc/cp/semantics.c
View file

@ -3377,14 +3377,17 @@ finish_omp_clauses (tree clauses)
{
if (processing_template_decl)
break;
error ("%qE is not a variable in clause %qs", t, name);
if (DECL_P (t))
error ("%qD is not a variable in clause %qs", t, name);
else
error ("%qE is not a variable in clause %qs", t, name);
remove = true;
}
else if (bitmap_bit_p (&generic_head, DECL_UID (t))
|| bitmap_bit_p (&firstprivate_head, DECL_UID (t))
|| bitmap_bit_p (&lastprivate_head, DECL_UID (t)))
{
error ("%qE appears more than once in data clauses", t);
error ("%qD appears more than once in data clauses", t);
remove = true;
}
else

3
gcc/testsuite/ChangeLog
View file

@ -3,6 +3,9 @@
* g++.dg/opt/nrv12.C: New test.
* gcc.target/i386/nrv1.c: New test.
PR c++/31748
* g++.dg/gomp/pr31748.C: New test.
2007-07-02 Ira Rosen <irar@il.ibm.com>
PR tree-optimization/32230

10
gcc/testsuite/g++.dg/gomp/pr31748.C Normal file
View file

@ -0,0 +1,10 @@
// PR c++/31748
struct A;
void
foo ()
{
#pragma omp parallel private(A) // { dg-error "struct A.*is not a variable" }
;
}