diff options
author | Jakub Jelinek <jakub@redhat.com> | 2017-10-06 17:50:50 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2017-10-06 17:50:50 +0200 |
commit | 7260f6f79cd3aea6580013d386ce0fd8696a6b19 (patch) | |
tree | d3aecc72ca1ca9d2daa372193c889de08de7511a | |
parent | 681f18d19b56527329fc80e5fdb8ef534978f705 (diff) | |
download | gcc-7260f6f79cd3aea6580013d386ce0fd8696a6b19.zip gcc-7260f6f79cd3aea6580013d386ce0fd8696a6b19.tar.gz gcc-7260f6f79cd3aea6580013d386ce0fd8696a6b19.tar.bz2 |
re PR c++/82299 (-Wuseless-cast errors on typed enums used in member data initializers in c++1z)
PR c++/82299
* decl.c (reshape_init): Suppress warn_useless_cast for direct enum
init.
* typeck.c (convert_for_assignment): Likewise.
* g++.dg/cpp0x/pr82299.C: New test.
From-SVN: r253495
-rw-r--r-- | gcc/cp/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/cp/decl.c | 5 | ||||
-rw-r--r-- | gcc/cp/typeck.c | 5 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 3 | ||||
-rw-r--r-- | gcc/testsuite/g++.dg/cpp0x/pr82299.C | 9 |
5 files changed, 25 insertions, 2 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index e41576d..ea0e8b6 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,5 +1,10 @@ 2017-10-06 Jakub Jelinek <jakub@redhat.com> + PR c++/82299 + * decl.c (reshape_init): Suppress warn_useless_cast for direct enum + init. + * typeck.c (convert_for_assignment): Likewise. + P0704R1 - fixing const-qualified pointers to members * typeck2.c (build_m_component_ref): For -std=c++2a allow pointer to const & qualified method on rvalue. diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index 7c68f68..6f36aa1 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -6047,7 +6047,10 @@ reshape_init (tree type, tree init, tsubst_flags_t complain) tree elt = CONSTRUCTOR_ELT (init, 0)->value; type = cv_unqualified (type); if (check_narrowing (ENUM_UNDERLYING_TYPE (type), elt, complain)) - return cp_build_c_cast (type, elt, tf_warning_or_error); + { + warning_sentinel w (warn_useless_cast); + return cp_build_c_cast (type, elt, tf_warning_or_error); + } else return error_mark_node; } diff --git a/gcc/cp/typeck.c b/gcc/cp/typeck.c index 326721e..c3310db 100644 --- a/gcc/cp/typeck.c +++ b/gcc/cp/typeck.c @@ -8527,7 +8527,10 @@ convert_for_assignment (tree type, tree rhs, { tree elt = CONSTRUCTOR_ELT (rhs, 0)->value; if (check_narrowing (ENUM_UNDERLYING_TYPE (type), elt, complain)) - rhs = cp_build_c_cast (type, elt, complain); + { + warning_sentinel w (warn_useless_cast); + rhs = cp_build_c_cast (type, elt, complain); + } else rhs = error_mark_node; } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 7432a3a..d7af2a9 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,8 @@ 2017-10-06 Jakub Jelinek <jakub@redhat.com> + PR c++/82299 + * g++.dg/cpp0x/pr82299.C: New test. + P0704R1 - fixing const-qualified pointers to members * g++.dg/cpp2a/ptrmem1.C: New test. diff --git a/gcc/testsuite/g++.dg/cpp0x/pr82299.C b/gcc/testsuite/g++.dg/cpp0x/pr82299.C new file mode 100644 index 0000000..27f4c5f --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp0x/pr82299.C @@ -0,0 +1,9 @@ +// PR c++/82299 +// { dg-do compile { target c++11 } } +// { dg-options "-Wuseless-cast" } + +enum Enum : char { A = 0, B = 1 }; + +struct S { + Enum e { Enum::A }; // { dg-bogus "useless cast to type" } +}; |