This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: m68k-openbsd -fpic fix
- To: Richard Henderson <rth at cygnus dot com>
- Subject: Re: m68k-openbsd -fpic fix
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Mon, 28 Jun 1999 05:26:36 -0600
- cc: Marc Espie <espie at quatramaran dot ens dot fr>, egcs at egcs dot cygnus dot com, egcs-patches at egcs dot cygnus dot com
- Reply-To: law at cygnus dot com
In message <19990626153434.A28430@cygnus.com>you write:
> On Thu, Jun 24, 1999 at 06:22:47PM -0600, Jeffrey A Law wrote:
> > > See http://egcs.cygnus.com/ml/egcs-bugs/1999-05/msg00764.html
> > > for an example of how critical it is.
> >
> > I've already stated that bug is unlikely to get fixed.
>
> I had a look at this this morning. Unless I'm dreadfully mistaken,
> this is not a difficult bug, but in fact quite simple.
>
> m68k can load arbitrary floating point constants directly. There's
> no need to spill them to memory, pic or non-pic. It would probably
> make for smaller code if constants that appeared more than once
> were spilled to memory earlier, but no effort has been made to do
> this. In any case, there's no need to do it during reload.
>
> So a simple tweek to PREFERRED_RELOAD_CLASS seems to be all that's
> necessary.
>
>
> r~
>
>
> * m68k.h (PREFERRED_RELOAD_CLASS): Don't force any FP const_doubles
> to memory.
I went ahead and installed this (mainline and branch).
jeff