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

[Chris Lattner]

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.

-Chris