search for: pr4882

Hello, This patch fixes bug 4882. Regards -Jakub -------------- next part -------------- A non-text attachment was scrubbed... Name: pr4882.patch Type: application/octet-stream Size: 6856 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090907/266f900c/attachment.obj>

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. > - const Type *Ty = Type::getInt64Ty(SI->getContext()); > - MemSetF = Intrinsic::getDeclaration(M, Intrinsic::memset, &Ty, 1); > + const Type *Tys[] =

...> > 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>