This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [C++ Patch] PR 67216 ("false is still a null pointer constant")

From: Paolo Carlini <paolo dot carlini at oracle dot com>
To: Jason Merrill <jason at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Mon, 17 Aug 2015 20:52:35 +0200
Subject: Re: [C++ Patch] PR 67216 ("false is still a null pointer constant")
Authentication-results: sourceware.org; auth=none
References: <55D22A8F dot 8060701 at oracle dot com> <55D22CFD dot 9040607 at redhat dot com>

Hi,

On 08/17/2015 08:50 PM, Jason Merrill wrote:

On 08/17/2015 02:40 PM, Paolo Carlini wrote:

the bug is very clear: in C++11 we reject the testcase because
null_ptr_cst_p returns 'true' for 'false', whereas [conv.ptr] is
carefully worded in terms of integer literals not boolean literals. The
obvious fix, using == INTEGER_TYPE instead of CP_INTEGRAL_TYPE_P,
appears to work well.

OK. I wonder if we can also drop the STRIP_NOPs on the C++11 path.

You are not alone ;) No, not trivially, without we ICE on pr51313.C.

Thanks,
Paolo.

Follow-Ups:
- Re: [C++ Patch] PR 67216 ("false is still a null pointer constant")
  - From: Jason Merrill

References:
- [C++ Patch] PR 67216 ("false is still a null pointer constant")
  - From: Paolo Carlini
- Re: [C++ Patch] PR 67216 ("false is still a null pointer constant")
  - From: Jason Merrill

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]