[vsipl++] [patch] Kernel updates

[Jules Bergmann]

Stefan,

> I have a single (minor) issue:
> 
>> +    * src/vsip/opt/cbe/spu/alf_pwarp_ub.cpp: Move ALF decls to
>> +      alf_decl.hpp.  Add missing argument to kernel.
>> +    * src/vsip/opt/ceb/spu/alf_decls.hpp: New file, decls for functions
>> +      expected by ALF.
> 
> I don't quite understand the purpose of this split. Do you propose we 
> adhere to the convention of naming these functions always 'input', 
> 'output', and 'kernel' ? The only reason ALF expects those names is 
> because you refer to them in the ALF_ACCEL_EXPORT_API macros, and it 
> would have been enough to put an 'extern "C"' linkage spec in front of 
> those function definitions, no matter how they are actually named.

The main intent was to prevent the 23 lines of declaration having to be 
replicated in every C++ kernel.  It does however require following a 
naming convention for the routines.  I didn't see that as a bad thing 
though.  Do you think it will create a problem?

It's sort of a moot point right now, as C++ kernels don't work with ALF 3.0.

> 
> The patch looks good otherwise.

Thanks!

				-- Jules

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