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

[Eli Friedman]

On Sun, Oct 3, 2010 at 12:09 PM, Chris Lattner <sabre at nondot.org> wrote:
> Author: lattner
> Date: Sun Oct  3 14:09:13 2010
> New Revision: 115473
>
> URL: http://llvm.org/viewvc/llvm-project?rev=115473&view=rev
> Log:
> the immediate field of pshufw is actually an 8-bit field, not a 8-bit field that is sign extended.  This fixes PR8288
>
> Modified:
>    llvm/trunk/include/llvm/IntrinsicsX86.td
>    llvm/trunk/lib/Target/X86/X86InstrMMX.td
>    llvm/trunk/test/MC/X86/x86-32.s
>
> Modified: llvm/trunk/include/llvm/IntrinsicsX86.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IntrinsicsX86.td?rev=115473&r1=115472&r2=115473&view=diff
> ==============================================================================
> --- llvm/trunk/include/llvm/IntrinsicsX86.td (original)
> +++ 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.

-Eli