This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: RFA: MIPS paired single support
- From: "Fu, Chao-Ying" <fu at mips dot com>
- To: "Richard Sandiford" <rsandifo at redhat dot com>, "James E Wilson" <wilson at specifixinc dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>, "David Ung" <davidu at mips dot com>, "Nigel Stephens" <nigel at mips dot com>, "Uhler, Mike" <uhler at mercury dot mips dot com>, "Thekkath, Radhika" <radhika at mercury dot mips dot com>
- Date: Mon, 30 Aug 2004 15:49:30 -0700
- Subject: RE: RFA: MIPS paired single support
Hello Richard, Jim,
Thanks for all your hard work!
For the testcases, we can put them in testsuite/gcc.dg/ .
But, we may need to fix two things as follows.
1. Change the target from "mips*-*-*" to "mips*-*-elf", such that
other mips targets won't fail.
2. For these files: mips-3d-2.c, mips-3d-3.c, mips-3d-4.c, mips-3d-5.c,
mips-3d-6.c, mips-3d-7.c, mips-3d-8.c, mips-ps-3.c, mips-ps-4.c,
we need to add the following to avoid being tested under "-ffast-math".
#ifndef __FAST_MATH__
// QNaN Tests
#endif
Best Regards,
Chao-ying Fu
MIPS Technologies, Inc.
-----Original Message-----
From: gcc-patches-owner@gcc.gnu.org
[mailto:gcc-patches-owner@gcc.gnu.org]On Behalf Of Richard Sandiford
Sent: Sunday, August 29, 2004 2:28 AM
To: James E Wilson
Cc: gcc-patches@gcc.gnu.org; Fu, Chao-Ying; David Ung; Nigel Stephens;
Uhler, Mike; Thekkath, Radhika
Subject: Re: RFA: MIPS paired single support
Richard Sandiford <rsandifo@redhat.com> writes:
> James E Wilson <wilson@specifixinc.com> writes:
>> I do think that reducing the size of the problem would make things
>> easier for both of us. If there is nothing really wrong with the patch,
>> then why not accept it in its current form, and require me to fix
>> problems after check-in?
>
> OK, go ahead.
Jim seems to be away from email at the moment, so I went ahead and
applied it. Hope that's OK. Thanks to both Jim and MIPS for the work
(and for their patience).
I committed the patch exactly as posted except that I added a copyright
banner to the new mips-ps-3d.md file. I also tweaked the changelog
(very) slightly. I didn't apply the testcases because there seemed to
be some disagreement about where they should go. Bootstrapped &
regression tested on mips-sgi-irix6.5.
FWIW, the reason for committing it now (rather than waiting a bit longer)
is that I've been doing subsequent MIPS work relative to this patch.
Most of the work is only really suitable for stage 2 and that deadline
is fast approaching.
Richard
PS. For avoidance of doubt, the rest of my mail still stands...
> WRT your TODO list, I've already volunteered to do these two items:
>
>> - reduce number of patterns, maybe encode comparison code into UNSPEC
>> parallel instead of into UNSPEC number
>> - use mode macros for v2sf patterns as appropriate
>
> and I get the impression that the following items are the ones you're
> least happy with, so I'll volunteer to do them as well:
>
>> - break dependency between mips.c bdesc table and mips.h BUILTINS_* enum table
>> - in mips_expand_* functions, use loops to reduce code duplication
>> - rewrite mips_expand_* function to avoid huge switch tables
>> - try to reduce or eliminate the MIPS_BUILTIN_* enums
>> - in general, reduce cut and paste more
>
> I'll also deal with:
>
>> - split the 4-wide (8 length) instructions after reload
>
> since it will be easier to do after (or at the same time as) macroisation.
>
> I'll leave you or MIPS to deal with:
>
>> - improve docs for new options
>> - movdf/movsf/movv2sf patterns have bad type attribute of fpstore for m/G alt
>> - add __mips3d predefined macro
>> - add new attributes for recip/sqrt step 1/2 patterns
>
> Richard