[LLVMdev] Patch - Allow calls that return i8 or i16. On SPU.

Chris Lattner clattner at apple.com
Mon Apr 19 22:38:42 PDT 2010

On Apr 19, 2010, at 2:31 AM, Kalle Raiskila wrote:

> hi,
> either function call in this code makes llc (targeting SPU) assert:
> declare i8 @return_i8()
> declare i16 @return_i16()
> define void @testfunc() {
>  %rv1 = call i8 @return_i8()
>  %rv2 = call i16 @return_i16()
>  ret void
> }
> Attached is a patch to fix it, and remove some related redundant code.

Thanks applied here:

> Does this rather trivial fix warrant a test case?

Absolutely, and I included it in the patch above.  Please include new testcases in the .patch file, it makes them easier to apply.  Thanks for the fixes!


> kalle
> <spu_call_i8_i16.patch>_______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev

More information about the llvm-dev mailing list