[vsipl++] [patch] Share ALF w/CML; Fix vmul_c kernel

Jules Bergmann jules at codesourcery.com
Wed May 14 13:25:11 UTC 2008


> 
> Shouldn't we pass the 'num_accelerators' argument down to cml_init() ? 
> Otherwise there is no way to mandate how many SPUs to use from SV++. Or 
> am I missing something ?

Yes definitely!  I was thinking of a FIXME to that effect.



>>  -include src/vsip/opt/cbe/alf/src/spu/GNUmakefile.inc
>>  
> 
> I just noticed this inclusion, which I missed to remove in my previous 
> patch. (The '-' at the beginning makes 'make' ignore the failure, so we 
> didn't see it.)
> Can you please remove this line when you commit the patch ? The alf/ 
> subdirectory doesn't exist any more.

Yes, will do.


-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705



More information about the vsipl++ mailing list