r188609 - Fix the name and the type of the argument for intrinisc

Eli Friedman eli.friedman at gmail.com
Sun Aug 18 18:10:48 PDT 2013


Okay, thanks; then I won't worry about the LLVM side of this.

+    llvm::Type *VecTy = llvm::VectorType::get(Int64Ty, 2);
+    llvm::Type *PtrTy = Int8PtrTy;
+    Value *One = llvm::ConstantInt::get(
Int32Ty, 1);
+    Value *Tmp = Builder.CreateAlloca(VecTy, One);

You can simplify this part using CodeGenFunction::CreateMemTemp.

-Eli


On Sun, Aug 18, 2013 at 12:35 PM, Craig Topper <craig.topper at gmail.com>wrote:

> And I totally failed to reply to all when I did so.
>
>
> On Sun, Aug 18, 2013 at 11:58 AM, Juergen Ributzka <juergen at apple.com>wrote:
>
>> Sorry, forgot to mention that Craig reviewed it.
>>
>> -Juergen
>>
>> On Aug 18, 2013, at 11:50 AM, Eli Friedman <eli.friedman at gmail.com>
>> wrote:
>>
>> On Sat, Aug 17, 2013 at 9:40 AM, Juergen Ributzka <juergen at apple.com>wrote:
>>
>>> Author: ributzka
>>> Date: Sat Aug 17 11:40:09 2013
>>> New Revision: 188609
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=188609&view=rev
>>> Log:
>>> Fix the name and the type of the argument for intrinisc
>>> _mm256_broadcastsi128_si256 to align with the Intel documentation.
>>>
>>> This fixes bug PR 16581 and rdar:14747994.
>>>
>>>
>>> Who reviewed this?
>>
>> -Eli
>>
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>>
>
>
> --
> ~Craig
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130818/534e3483/attachment.html>


More information about the cfe-commits mailing list