[Bug c++/51045] New: [C++0x] erroneous zero as null pointer constant warning at delete

[Bug c++/51045] New: [C++0x] erroneous zero as null pointer constant warning at delete

[jarrydb at cse dot unsw.edu.au]

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

             Bug #: 51045
           Summary: [C++0x] erroneous zero as null pointer constant
                    warning at delete
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: jarrydb@cse.unsw.edu.au
              Host: x86_64-unknown-linux-gnu
            Target: x86_64-unknown-linux-gnu
             Build: x86_64-unknown-linux-gnu


The attached code erroneously gives me a warning about using zero as a null
pointer constant where there is no zero at all. The line it points to is a
delete statement.

g++ -std=gnu++11 zeronull.cpp -c -Wzero-as-null-pointer-constant
zeronull.cpp: In instantiation of ‘A<T>::~A() [with T = Op]’:
zeronull.cpp:12:7:   required from here
zeronull.cpp:7:5: warning: zero as null pointer constant
[-Wzero-as-null-pointer-constant]


gcc -v

Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/home/jarrydb/current/soft/install-latest/libexec/gcc/x86_64-unknown-linux-gnu/4.7.0/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: /home/jarrydb/current/soft/src/gcc-git/configure
--prefix=/home/jarrydb/current/soft/install-latest --disable-multilib
--enable-languages=c,c++ : (reconfigured)
/home/jarrydb/current/soft/src/gcc-git/configure
--prefix=/home/jarrydb/current/soft/install-latest --disable-multilib
--enable-languages=c,c++
Thread model: posix
gcc version 4.7.0 20111108 (experimental) (GCC)

[Bug c++/51045] [C++0x] erroneous zero as null pointer constant warning at delete

[jarrydb at cse dot unsw.edu.au]

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

--- Comment #1 from Jarryd Beck <jarrydb at cse dot unsw.edu.au> 2011-11-09 02:15:34 UTC ---
Created attachment 25762
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25762
erroneous zero as null pointer warning

[Bug c++/51045] [C++0x] erroneous zero as null pointer constant warning at delete

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2011-11-09
         AssignedTo|unassigned at gcc dot       |paolo.carlini at oracle dot
                   |gnu.org                     |com
     Ever Confirmed|0                           |1

--- Comment #2 from Paolo Carlini <paolo.carlini at oracle dot com> 2011-11-09 10:41:58 UTC ---
Seems trivial, an integer_zero_node in build_dtor_call.

[Bug c++/51045] [C++0x] erroneous zero as null pointer constant warning at delete

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.0

[Bug c++/51045] erroneous zero as null pointer constant warning at delete

[paolo at gcc dot gnu.org]

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

--- Comment #3 from paolo at gcc dot gnu.org <paolo at gcc dot gnu.org> 2011-11-09 17:21:23 UTC ---
Author: paolo
Date: Wed Nov  9 17:21:19 2011
New Revision: 181214

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181214
Log:
/cp
2011-11-09  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/51045
    * init.c (build_new_1, build_vec_delete_1, build_delete):
    Use nullptr_node.

/testsuite
2011-11-09  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/51045
    * g++.dg/warn/Wzero-as-null-pointer-constant-2.C: New.


2011-11-09  Paolo Carlini  <paolo.carlini@oracle.com>

    * doc/invoke.texi ([Option Summary, C++ Language Options]):
    Add -Wzero-as-null-pointer-constant.

Added:
    trunk/gcc/testsuite/g++.dg/warn/Wzero-as-null-pointer-constant-2.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/init.c
    trunk/gcc/doc/invoke.texi

[Bug c++/51045] erroneous zero as null pointer constant warning at delete

[paolo at gcc dot gnu.org]

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

--- Comment #4 from paolo at gcc dot gnu.org <paolo at gcc dot gnu.org> 2011-11-09 17:22:25 UTC ---
Author: paolo
Date: Wed Nov  9 17:22:22 2011
New Revision: 181215

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181215
Log:
/cp
2011-11-09  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/51045
    * init.c (build_new_1, build_vec_delete_1, build_delete):
    Use nullptr_node.

/testsuite
2011-11-09  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/51045
    * g++.dg/warn/Wzero-as-null-pointer-constant-2.C: New.


2011-11-09  Paolo Carlini  <paolo.carlini@oracle.com>

    * doc/invoke.texi ([Option Summary, C++ Language Options]):
    Add -Wzero-as-null-pointer-constant.

Modified:
    trunk/gcc/testsuite/ChangeLog

[Bug c++/51045] erroneous zero as null pointer constant warning at delete

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

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

--- Comment #5 from Paolo Carlini <paolo.carlini at oracle dot com> 2011-11-09 17:23:02 UTC ---
Fixed.