This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Missing return_label using g++ CVS under vax ultrix.
- To: Richard Henderson <rth at redhat dot com>
- Subject: Re: PATCH: Missing return_label using g++ CVS under vax ultrix.
- From: Jason Merrill <jason at redhat dot com>
- Date: 14 Dec 2000 17:05:57 +0000
- Cc: John David Anglin <dave at hiauly1 dot hia dot nrc dot ca>, gcc-patches at gcc dot gnu dot org, gcc-bugs at gcc dot gnu dot org
- References: <200011291813.NAA25179@hiauly1.hia.nrc.ca><u9d7exnb9i.fsf@casey.cygnus.co.uk> <20001213101741.B10569@redhat.com>
>>>>> "Richard" == Richard Henderson <rth@redhat.com> writes:
> On Wed, Dec 13, 2000 at 11:11:34AM +0000, Jason Merrill wrote:
>> Richard, was there a rationale for this hunk, or were you just being tidy?
> There was a named return value test that Did The Wrong Thing.
Hmm? I'm referring to this change. I don't see a named return value test
here.
*** semantics.c Tue Dec 12 10:58:03 2000
--- semantics.c.~1~ Tue Dec 5 10:53:25 2000
*************** genrtl_finish_function (fn)
*** 2603,2610 ****
/* Finish building code that will trigger warnings if users forget
to make their functions return values. */
! if (no_return_label || cleanup_label)
! emit_jump (return_label);
if (no_return_label)
{
/* We don't need to call `expand_*_return' here because we don't
--- 2603,2609 ----
/* Finish building code that will trigger warnings if users forget
to make their functions return values. */
! emit_jump (return_label);
if (no_return_label)
{
/* We don't need to call `expand_*_return' here because we don't