aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabriel Dos Reis <gdr@integrable-solutions.net>2005-12-01 12:00:17 +0000
committerGabriel Dos Reis <gdr@gcc.gnu.org>2005-12-01 12:00:17 +0000
commit5d352b2d0a626db647dc54d7b61f37545c8b2852 (patch)
tree5be02279e546c457b46e114f76f4d19311831990
parent4288927194f09f28d86b57940c8bc3619ba589aa (diff)
downloadgcc-5d352b2d0a626db647dc54d7b61f37545c8b2852.zip
gcc-5d352b2d0a626db647dc54d7b61f37545c8b2852.tar.gz
gcc-5d352b2d0a626db647dc54d7b61f37545c8b2852.tar.bz2
re PR c++/13384 (error: non-lvalue in assignment - message a little misleading for C++)
PR c/13384 * c-common.c (lvalue_error): Fix wording. testsuite/ PR c/13384 * gcc.dg/pr17730-1.c: Adjust. * gcc.dg/lvalue1.c (main): Likewise. * gcc.dg/lvalue-2.c: Likewise. * g++.dg/pr7503-3.C From-SVN: r107816
-rw-r--r--gcc/ChangeLog5
-rw-r--r--gcc/c-common.c10
-rw-r--r--gcc/testsuite/ChangeLog8
-rw-r--r--gcc/testsuite/g++.dg/opt/pr7503-3.C8
-rw-r--r--gcc/testsuite/gcc.dg/lvalue-2.c12
-rw-r--r--gcc/testsuite/gcc.dg/lvalue1.c2
-rw-r--r--gcc/testsuite/gcc.dg/pr17730-1.c2
7 files changed, 30 insertions, 17 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 1cc0251..e0715c4 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2005-12-01 Gabriel Dos Reis <gdr@integrable-solutions.net>
+
+ PR c/13384
+ * c-common.c (lvalue_error): Fix wording.
+
2005-12-01 Richard Guenther <rguenther@suse.de>
* tree-ssa-forwprop.c (forward_propagate_addr_expr): Fix
diff --git a/gcc/c-common.c b/gcc/c-common.c
index 43c147b..e617094 100644
--- a/gcc/c-common.c
+++ b/gcc/c-common.c
@@ -5990,19 +5990,19 @@ lvalue_error (enum lvalue_use use)
switch (use)
{
case lv_assign:
- error ("invalid lvalue in assignment");
+ error ("lvalue required as left operand of assignment");
break;
case lv_increment:
- error ("invalid lvalue in increment");
+ error ("lvalue required as increment operand");
break;
case lv_decrement:
- error ("invalid lvalue in decrement");
+ error ("lvalue required as decrement operand");
break;
case lv_addressof:
- error ("invalid lvalue in unary %<&%>");
+ error ("lvalue required as unary %<&%> operand");
break;
case lv_asm:
- error ("invalid lvalue in asm statement");
+ error ("lvalue required in asm statement");
break;
default:
gcc_unreachable ();
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 71f0c9b..a7f9194 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,11 @@
+2005-12-01 Gabriel Dos Reis <gdr@integrable-solutions.net>
+
+ PR c/13384
+ * gcc.dg/pr17730-1.c: Adjust.
+ * gcc.dg/lvalue1.c (main): Likewise.
+ * gcc.dg/lvalue-2.c: Likewise.
+ * g++.dg/pr7503-3.C
+
2005-12-01 Geoffrey Keating <geoffk@apple.com>
* gcc.target/powerpc/pr18096-1.c: Be more liberal about possible
diff --git a/gcc/testsuite/g++.dg/opt/pr7503-3.C b/gcc/testsuite/g++.dg/opt/pr7503-3.C
index 4e8de0f..dc71d20 100644
--- a/gcc/testsuite/g++.dg/opt/pr7503-3.C
+++ b/gcc/testsuite/g++.dg/opt/pr7503-3.C
@@ -6,21 +6,21 @@ extern int A, B;
void test1()
{
- (A++ <? B) = 0; // { dg-error "lvalue in assignment" }
+ (A++ <? B) = 0; // { dg-error "lvalue required" }
}
void test2()
{
- (A <? B++) = 0; // { dg-error "lvalue in assignment" }
+ (A <? B++) = 0; // { dg-error "lvalue required" }
}
void test3()
{
- (A++ >? B) = 0; // { dg-error "lvalue in assignment" }
+ (A++ >? B) = 0; // { dg-error "lvalue required" }
}
void test4()
{
- (A >? B++) = 0; // { dg-error "lvalue in assignment" }
+ (A >? B++) = 0; // { dg-error "lvalue required" }
}
diff --git a/gcc/testsuite/gcc.dg/lvalue-2.c b/gcc/testsuite/gcc.dg/lvalue-2.c
index 5ad648e..add3e78 100644
--- a/gcc/testsuite/gcc.dg/lvalue-2.c
+++ b/gcc/testsuite/gcc.dg/lvalue-2.c
@@ -9,12 +9,12 @@ int a, b;
void
f0 (void)
{
- (a+b) = 1; /* { dg-error "error: invalid lvalue in assignment" } */
- (a+b)++; /* { dg-error "error: invalid lvalue in increment" } */
- ++(a+b); /* { dg-error "error: invalid lvalue in increment" } */
- (a+b)--; /* { dg-error "error: invalid lvalue in decrement" } */
- --(a+b); /* { dg-error "error: invalid lvalue in decrement" } */
- &(a+b); /* { dg-error "error: invalid lvalue in unary '&'" } */
+ (a+b) = 1; /* { dg-error "lvalue required as left operand of assignment" } */
+ (a+b)++; /* { dg-error "lvalue required as increment operand" } */
+ ++(a+b); /* { dg-error "lvalue required as increment operand" } */
+ (a+b)--; /* { dg-error "lvalue required as decrement operand" } */
+ --(a+b); /* { dg-error "lvalue required as decrement operand" } */
+ &(a+b); /* { dg-error "lvalue required as unary '&' operand" } */
}
const int c;
diff --git a/gcc/testsuite/gcc.dg/lvalue1.c b/gcc/testsuite/gcc.dg/lvalue1.c
index 7e21608..67b0130 100644
--- a/gcc/testsuite/gcc.dg/lvalue1.c
+++ b/gcc/testsuite/gcc.dg/lvalue1.c
@@ -4,6 +4,6 @@
int main()
{
int i;
- +i = 1; /* { dg-error "invalid lvalue in assignment" } */
+ +i = 1; /* { dg-error "lvalue required as left operand of assignment" } */
return 0;
}
diff --git a/gcc/testsuite/gcc.dg/pr17730-1.c b/gcc/testsuite/gcc.dg/pr17730-1.c
index 0fede2f..fe9be68 100644
--- a/gcc/testsuite/gcc.dg/pr17730-1.c
+++ b/gcc/testsuite/gcc.dg/pr17730-1.c
@@ -2,4 +2,4 @@
/* { dg-do compile } */
/* { dg-options "" } */
-char *p = &'C'; /* { dg-error "error: invalid lvalue in unary '&'" } */
+char *p = &'C'; /* { dg-error "error: lvalue required as unary '&' operand" } */