[LLVMdev] [Patch] Fix SSE2 packing intrinsics return type

[Eli Friedman]

On Tue, Jun 9, 2009 at 10:15 PM, Chris Lattner<clattner at apple.com> wrote:
>
> On Jun 9, 2009, at 5:56 PM, Eli Friedman wrote:
>
>> On Tue, Jun 9, 2009 at 2:58 PM, Nicolas Capens<nicolas at capens.net>
>> wrote:
>>> Please consider committing the attached patch. I believe the SSE2
>>> packsswb,
>>> packssdw and packuswb intrinsics have an incorrect return type.
>>
>> If we really wanted to do this, an AutoUpgrade patch would be
>> necessary for backwards-compatibility.  I'm not sure it's worth
>> bothering.
>
> Did they work before with llvm-gcc?  If not, then don't worry about
> backward compatibility.

They work just fine as-is due to the rough equivalence of all SSE2
types; it's just not quite clean to use an i16 vector for the return
when the result actually contains i8s.

-Eli