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: [PATCH] Fix PR c++/18969 (invalid return statement diagnosed too late)

From: Jason Merrill <jason at redhat dot com>
To: Patrick Palka <patrick at parcs dot ath dot cx>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Manuel LÃpez-IbÃÃez <lopezibanez at gmail dot com>
Date: Thu, 20 Aug 2015 18:42:37 -0400
Subject: Re: [PATCH] Fix PR c++/18969 (invalid return statement diagnosed too late)
Authentication-results: sourceware.org; auth=none
References: <1437878959-10982-1-git-send-email-patrick at parcs dot ath dot cx> <55B4581F dot 1080900 at redhat dot com> <CA+C-WL-dxy4jO+8w1ZLE5iSampTi+oMecUpkSuuubw22=x-vBQ at mail dot gmail dot com> <CA+C-WL__qEBQrH+ZFK1Nij1FiLvCCbnzTFevL3h_W4+8eTxWpw at mail dot gmail dot com>

On 07/26/2015 01:50 PM, Patrick Palka wrote:

Hmm, on second thought, I don't think this fix is right.  It may be
the case that the 'return b;' was there to make instantiation of that
template a compile-time error. By changing it to 'return;'
instantiation is allowed.  Is this property important here?  Should I
preserve the original property (that instantiation is a compile-time
error) by instead doing the following?


Sure, can't hurt.

Jason

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