diff options
| author | Marek Polacek <polacek@redhat.com> | 2021-11-19 14:22:10 -0500 |
|---|---|---|
| committer | Marek Polacek <polacek@redhat.com> | 2021-11-23 15:02:08 -0500 |
| commit | 4b1d3d8d732bea86c7b2aba46c2a437461020824 (patch) | |
| tree | 4c71c291334d11e645f9a42f24b90b7d572c36a1 /gcc/cp | |
| parent | 7b7318faf7987ae17806a8c84fbaccaf255e7cbf (diff) | |
| download | gcc-4b1d3d8d732bea86c7b2aba46c2a437461020824.zip gcc-4b1d3d8d732bea86c7b2aba46c2a437461020824.tar.gz gcc-4b1d3d8d732bea86c7b2aba46c2a437461020824.tar.bz2 | |
c++: -Wuninitialized for mem-inits and empty classes [PR19808]
This fixes a bogus -Wuninitialized warning: there's nothing to initialize
in empty classes, so don't add them into our uninitialized set.
PR c++/19808
gcc/cp/ChangeLog:
* init.c (emit_mem_initializers): Don't add is_really_empty_class
members into uninitialized.
gcc/testsuite/ChangeLog:
* g++.dg/warn/Wuninitialized-28.C: Make a class nonempty.
* g++.dg/warn/Wuninitialized-29.C: Likewise.
* g++.dg/warn/Wuninitialized-31.C: New test.
Diffstat (limited to 'gcc/cp')
| -rw-r--r-- | gcc/cp/init.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/gcc/cp/init.c b/gcc/cp/init.c index 975f2ed..2a4512e 100644 --- a/gcc/cp/init.c +++ b/gcc/cp/init.c @@ -1470,7 +1470,8 @@ emit_mem_initializers (tree mem_inits) for (tree f = next_initializable_field (TYPE_FIELDS (current_class_type)); f != NULL_TREE; f = next_initializable_field (DECL_CHAIN (f))) - if (!DECL_ARTIFICIAL (f)) + if (!DECL_ARTIFICIAL (f) + && !is_really_empty_class (TREE_TYPE (f), /*ignore_vptr*/false)) uninitialized.add (f); if (mem_inits |