diff options
author | Gabriel Dos Reis <gdr@integrable-solutions.net> | 2005-12-01 12:00:17 +0000 |
---|---|---|
committer | Gabriel Dos Reis <gdr@gcc.gnu.org> | 2005-12-01 12:00:17 +0000 |
commit | 5d352b2d0a626db647dc54d7b61f37545c8b2852 (patch) | |
tree | 5be02279e546c457b46e114f76f4d19311831990 | |
parent | 4288927194f09f28d86b57940c8bc3619ba589aa (diff) | |
download | gcc-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/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/c-common.c | 10 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 8 | ||||
-rw-r--r-- | gcc/testsuite/g++.dg/opt/pr7503-3.C | 8 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/lvalue-2.c | 12 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/lvalue1.c | 2 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/pr17730-1.c | 2 |
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" } */ |