[llvm-commits] [llvm] r112881 - /llvm/trunk/include/llvm/IntrinsicsX86.td
Eric Christopher
echristo at apple.com
Thu Sep 2 15:46:56 PDT 2010
On Sep 2, 2010, at 3:45 PM, Dale Johannesen wrote:
>
> On Sep 2, 2010, at 3:31 PMPDT, Bill Wendling wrote:
>
>> On Sep 2, 2010, at 3:30 PM, Eric Christopher wrote:
>>
>>> On Sep 2, 2010, at 3:26 PM, Bill Wendling wrote:
>>>
>>>> -// def int_x86_mmx_palignr_b : GCCBuiltin<"__builtin_ia32_palignr">,
>>>> -// Intrinsic<[llvm_v1i64_ty], [llvm_v1i64_ty,
>>>> -// llvm_v1i64_ty, llvm_i8_ty], [IntrNoMem]>;
>>>> +// def int_x86_mmx_palignr_b : GCCBuiltin<"__builtin_ia32_palignr">,
>>>> +// Intrinsic<[llvm_v8i8_ty], [llvm_8i8_ty,
>>>> +// llvm_v8i8_ty, llvm_i8_ty], [IntrNoMem]>;
>>>
>>> This will, if it's ever re-enabled, likely not match the one coming from llvm-gcc.
>>>
>> The one from llvm-gcc was changed awhile ago from "long long" to "v1di". I think the original was correct, but clang doesn't seem to agree... Eh...I'll change it back.
>
> llvm-gcc isn't currently producing an intrinsic for palignr because there isn't one. I believe v8i8 corresponds best to what the instruction does, fwiw, but really in intrinsics "correct" is what we define it to be.
>
> No rush on palignr, that needs some work in the BE before I can turn it on.
>
It was at one point. I didn't keep up with what happened here over the last day or so, been too busy watching things break. What happened with palignr?
-eric
More information about the llvm-commits
mailing list