[llvm-commits] [llvm] r115473 - in /llvm/trunk: include/llvm/IntrinsicsX86.td lib/Target/X86/X86InstrMMX.td test/MC/X86/x86-32.s

[Bill Wendling]

On Oct 3, 2010, at 1:06 PM, Chris Lattner wrote:

> On Oct 3, 2010, at 12:54 PM, Eli Friedman wrote:
>>> +++ llvm/trunk/include/llvm/IntrinsicsX86.td Sun Oct  3 14:09:13 2010
>>> @@ -631,7 +631,7 @@
>>>              Intrinsic<[llvm_v16i8_ty], [llvm_v16i8_ty,
>>>                         llvm_v16i8_ty], [IntrNoMem]>;
>>>  def int_x86_ssse3_pshuf_w         : GCCBuiltin<"__builtin_ia32_pshufw">,
>>> -              Intrinsic<[llvm_x86mmx_ty], [llvm_x86mmx_ty, llvm_i32_ty],
>>> +              Intrinsic<[llvm_x86mmx_ty], [llvm_x86mmx_ty, llvm_i8_ty],
>>>                         [IntrNoMem]>;
>>> }
>> 
>> Not really about this patch, but why is the intrinsic named
>> "int_x86_ssse3_pshuf_w"?  pshufw is MMX2, unrelated to SSSE3.
> 
> Yeah, I have no idea about that either.  Bill, can you fix this?  Unless this builtin existed in 2.8 or earlier, we don't need to worry about autoupgrading it.
> 
Sure, I guess. (It does exist in 2.8 and earlier.)

-bw, rearranging the deck chairs on the MMX Titanic. :)