[Bug c++/53594] New: [C++11Spurious -Wuninitialized warning for member with NSDMI

[Bug c++/53594] New: [C++11Spurious -Wuninitialized warning for member with NSDMI

[redi at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

             Bug #: 53594
           Summary: [C++11Spurious -Wuninitialized warning for member with
                    NSDMI
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: redi@gcc.gnu.org


template<class T, int Min>
class Range
{
    const T min = Min;
};

Range<int, 0> r;

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[redi at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic
            Summary|[C++11Spurious              |[C++11] Spurious
                   |-Wuninitialized warning for |-Wuninitialized warning for
                   |member with NSDMI           |member with NSDMI

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> 2012-06-06 16:52:00 UTC ---
$ g++-4.7 -std=c++11 c.cc -c  -Wuninitialized -c
c.cc: In instantiation of 'class Range<int, 0>':
c.cc:7:15:   required from here
c.cc:4:19: warning: non-static const member 'const int Range<int, 0>::min' in
class without a constructor [-Wuninitialized]

The test in class.c:check_bases_and_members doesn't appear to look for NSDMIs

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[redi at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> 2012-06-06 16:56:12 UTC ---
Reduced:

class Range
{
    const int min = 1;
};

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[pinskia at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vincenzo.innocente at cern
                   |                            |dot ch

--- Comment #3 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-06-11 07:57:31 UTC ---
*** Bug 53629 has been marked as a duplicate of this bug. ***

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[pinskia at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-06-11
     Ever Confirmed|0                           |1

--- Comment #4 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-06-11 07:59:02 UTC ---
Confirmed.

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|unassigned at gcc dot       |jakub at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-06-11 08:57:57 UTC ---
Created attachment 27602
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27602
gcc48-pr53594.patch

Untested fix.

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[vincenzo.innocente at cern dot ch]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

--- Comment #6 from vincenzo Innocente <vincenzo.innocente at cern dot ch> 2012-06-11 09:28:49 UTC ---
the patch compiles and fixes my test case,
I've not tested any possible side effects

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-06-25 06:48:09 UTC ---
Author: jakub
Date: Mon Jun 25 06:48:04 2012
New Revision: 188925

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188925
Log:
    PR c++/53594
    * class.c (check_bases_and_members): Avoid -Wuninitialized
    diagnostics for non-static const members or references if they
    use NSDMI.

    * g++.dg/cpp0x/nsdmi7.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/cpp0x/nsdmi7.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/class.c
    trunk/gcc/testsuite/ChangeLog

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-06-25 06:54:12 UTC ---
Author: jakub
Date: Mon Jun 25 06:54:08 2012
New Revision: 188926

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188926
Log:
    PR c++/53594
    * class.c (check_bases_and_members): Avoid -Wuninitialized
    diagnostics for non-static const members or references if they
    use NSDMI.

    * g++.dg/cpp0x/nsdmi7.C: New test.

Added:
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/cpp0x/nsdmi7.C
Modified:
    branches/gcc-4_7-branch/gcc/cp/ChangeLog
    branches/gcc-4_7-branch/gcc/cp/class.c
    branches/gcc-4_7-branch/gcc/testsuite/ChangeLog

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-06-25 06:54:12 UTC ---
Author: jakub
Date: Mon Jun 25 06:54:08 2012
New Revision: 188926

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188926
Log:
    PR c++/53594
    * class.c (check_bases_and_members): Avoid -Wuninitialized
    diagnostics for non-static const members or references if they
    use NSDMI.

    * g++.dg/cpp0x/nsdmi7.C: New test.

Added:
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/cpp0x/nsdmi7.C
Modified:
    branches/gcc-4_7-branch/gcc/cp/ChangeLog
    branches/gcc-4_7-branch/gcc/cp/class.c
    branches/gcc-4_7-branch/gcc/testsuite/ChangeLog

--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-06-25 06:55:09 UTC ---
Fixed.

[Bug c++/53594] [C++11] Spurious -Wuninitialized warning for member with NSDMI

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53594

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-06-25 06:55:09 UTC ---
Fixed.