[vsipl++] [patch] SAL dispatch for matrix and vector products

Don McCoy don at codesourcery.com
Tue Nov 15 21:08:00 UTC 2005


Jules Bergmann wrote:

>> I added SAL_USE_MAT_MUL for use in selecting the newer functions.
>
> This should start with VSIP_IMPL, i.e. VSIP_IMPL_SAL_USE_MAT_MUL
>
Fixed.  Committed.

>>  The default (0) selects the old ones.  It is tempting to make this a 
>> configuration option, but i really don't anticipate changing this 
>> switch back and forth.  I'm sure we'll just change it to a 1 when 
>> Mercury indicates it is a good time to do so.
>
>
> That's a good idea.  It's possible that people will want to use our 
> library with older versions of SAL.  After checking this in, can you 
> add a configure option?
>
Ok.  I'll make it override the above -- i.e. configure can optionally 
set it and i'll add #ifndef ... to the code so that it still defaults to 
0.  If there is a alternate method that is preferred, please let me 
know.  Specifically, did you want it to be a mandatory option (when 
using SAL) so that the user is forced to select one or the other?


-- 
Don McCoy
CodeSourcery, LLC




More information about the vsipl++ mailing list