Re: x86 patch: SSE-based FP<=>int conversions, round 4

[Stuart Hastings <stuart at apple dot com>]

Bleah.  My recently-upgraded email client used an illegal MIME type in  
the original, and the list robot rejected it.  Here's a copy for the  
list archive.

stuart

Begin forwarded message:

> From: Stuart Hastings <stuart@apple.com>
> Date: February 13, 2007 10:53:56 AM PST
> To: Richard Guenther <richard.guenther@gmail.com>
> Cc: Richard Henderson <rth@redhat.com>, gcc-patches@gcc.gnu.org
> Subject: Re: x86 patch: SSE-based FP<=>int conversions, round 4
>
>
> On Feb 10, 2007, at 7:49 AM, Richard Guenther wrote:
>
> > On 2/9/07, Richard Henderson <rth@redhat.com> wrote:
> > > On Thu, Feb 08, 2007 at 04:56:08PM -0800, Stuart Hastings wrote:
> > > > I suppose the review would be easier if I attached a patch to
> > > > review.  Duh.
> > >
> > > Ok.
> >
> > It looks like this caused PR30757.
> >
> > double log(double x);
> > void f() { unsigned char a = log(0); }
> >
> > ./cc1 -quiet -march=athlon-xp -mfpmath=sse t.i
> > t.i: In function 'f':
> > t.i:2: error: unrecognizable insn:
> > (insn 11 10 12 3 (set (reg:SI 61)
> >       (fix:SI (reg:DF 58 [ D.1820 ]))) -1 (nil)
> >   (nil))
> > t.i:2: internal compiler error: in extract_insn, at recog.c:2108
> > Please submit a full bug report,
> > with preprocessed source if appropriate.
> > See <URL:http://gcc.gnu.org/bugs.html> for instructions.
>
> Here's a fix:

Attachment: gcc.fsf.cvt7.diffs.txt
Description: Text document

> 2007-02-13  Stuart Hastings  <stuart@apple.com>
>
> 	* config/i386/i386.md (fixuns_truncdfhi2): Require SSE2.
>
> Bootstrapped, DejaGnu on Linux/x86_32 with -march=athlon-xp (an SSE1  
> target) -mfpmath=sse; no regressions.
>
> O.K. to commit?
>
> stuart hastings
> Apple