gcc-patches@gcc.gnu.org
[Top] [All Lists]

Patch (x86 sse2) (revised): wrong code for movhlps

Subject: Patch (x86 sse2) (revised): wrong code for movhlps
From: Dale Johannesen
Date: Mon, 5 Dec 2005 15:33:57 -0800
OK, let's try this again. It's actually just the testcase that is revised per
Andrew's suggestion.  (The problem is that the alternatives of movhlps
were not semantically equivalent - the last two have the halves reversed
relative to the first one.  So matching one instead of another leads to
wrong code.) I said Friday this patch wasn't right, but I was being misled
by the unrelated _MM_TRANSPOSE4_PS bug; I now believe it is right.
Tested by running i386.exp; I don't believe this occurs elsewhere.  OK?

2005-12-05  Dale Johannesen  <dalej@xxxxxxxxx>

        * config/i386/sse.md (sse_movhlps):  Reverse operands for
        2nd and 3rd alternatives.

2005-12-05  Dale Johannesen  <dalej@xxxxxxxxx>

        * gcc.target/i386/sse-17.c:  New.

Attachment: diffs1205a.txt
Description: Text document

Attachment: sse-17.c
Description: Text document


<Prev in Thread] Current Thread [Next in Thread>