On Sep 7, 2009, at 5:02 PM, Duncan Sands wrote:
> Hi Jakub, looks good.
>
>> + LLVMContext *Context = &SI->getContext();
>
> I guess this could be
> LLVMContext &Context = SI->getContext();
> which means you can use Context rather than *Context below.
Right, C bad habit ;) Fixed.
>> - const Type *Ty = Type::getInt64Ty(SI->getContext());
>> - MemSetF = Intrinsic::getDeclaration(M, Intrinsic::memset,
>> &Ty, 1);
>> + const Type *Tys[] = {Type::getInt64Ty(*Context)};
>> + MemSetF = Intrinsic::getDeclaration(M, Intrinsic::memset,
>> + Tys, 1);
>
> Why did you introduce Tys and not stick with Ty?
Ah, I did "svn update" after my correction and I didn't notice
that.
Fixed.
I also added "-disable-output" to test-case.
-Jakub
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr4882-2.patch
Type: application/octet-stream
Size: 6696 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20090907/b766cf19/attachment.obj>