This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH]: SH 2a - Part 4: Begin using RETURN_VALUE
- From: Corinna Vinschen <vinschen at redhat dot com>
- To: gdb-patches at sources dot redhat dot com
- Date: Wed, 6 Oct 2004 11:08:09 +0200
- Subject: Re: [PATCH]: SH 2a - Part 4: Begin using RETURN_VALUE
- References: <20040909124701.GA7927@cygbert.vinschen.de> <16738.43902.850632.187540@localhost.redhat.com>
- Reply-to: gdb-patches at sources dot redhat dot com
On Oct 5 10:11, Elena Zannoni wrote:
> Corinna Vinschen writes:
> > this is SH 2a patch 4. It does not introduce new functionality, but
> > instead it converts sh-tdep.c to use RETURN_VALUE instead of
> > STORE_RETURN_VALUE, EXTRACT_RETURN_VALUE, RETURN_VALUE_ON_STACK and
> > USE_STRUCT_CONVENTION.
>
> OK
Thanks, applied.
> > What this patch does *not* do is this: I would love to rename the
> > functions sh_default_store/extract_value and sh3e_sh4_store/extract_value
> > to sh_store/extract_value_nofpu and sh_store/extract_value_fpu.
> > This would decouple the sense of these functions from the cpu names,
> > which is rather irritating and wrong anyway. If that's ok, I'd
> > submit another patch which just performs the renaming.
>
> It wasn't wrong and irritating at the time it was introduced.
Yes, that's apparently right. As I wrote, "I would love to...", because
time has gone a bit over that naming scheme and I think, changing the
names to reflect their usage in a more general way would help when reading
the code in future.
Is it ok to rename them?
> > BEFORE AFTER
>
> Thanks for providing the test results. Are these with
> sources.redhat.com sources? Or some internal only tree? What version
> of gcc? So we have a good reference point.
I used sources sources (erm...) to evaluate these results, exactly
CVS HEAD from 2004-09-08. GCC was a 3.4 with SH2a patches applied.
Corinna
--
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat, Inc.