Re: [C++ Patch] PR 65323

[Jason Merrill <jason@redhat.com>]

On 03/06/2015 03:36 AM, Paolo Carlini wrote:
> this is a regression about duplicate warnings with
> -Wzero-as-null-pointer-constant. The regression is rather old, affects
> 4_8-branch too, and started when check_default_argument got a
> perform_implicit_conversion_flags call which warns a first time, then
> maybe_warn_zero_as_null_pointer_constant as called by
> check_default_argument itself warns a second time. The latter call is
> even older, dates back to c++/52718, I think we can now safely remove it
> and keep on returning nullptr_node to avoid warning later still at the
> call sites (that was the point of c++/52718). Tested x86_64-linux.

Do we need this special handling at all?  When I remove that whole 'if' 
block I still only get one warning from the 52718 testcase.

Jason