[vsipl++] [patch] double support for SAL LU solver

Don McCoy don at codesourcery.com
Sun May 7 19:42:19 UTC 2006


Jules Bergmann wrote:
> 
> This looks good.  Can you:
> 
>  - Move the reciprocal call from sal_matfbs to sal_matlud.  That way
>    if multiple sal_matfbs calls are made (either because B/X have
>    multiple columns, or because the LU object is used multiple times),
>    vrecip will only get called once.
> 
>  - Create a typedef for the block_type of recip_.  That way the Ext_data
>    for recip_ is guaranteed to have the correct block type if recip_
>    ever changes.
> 
>  - a few more comments sprinkled below.
> 
> If these comments make sense, once you address them this looks good to 
> check in.

Committed with suggested changes.  Thanks for catching those things.

> 
> How do we test this?  By manually disabling the the mat_trans and 
> mat_herm cases?
> 

Exactly.


-- 
Don McCoy
don (at) CodeSourcery
(888) 776-0262 / (650) 331-3385, x712
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: lu2.diff
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20060507/fc2f1032/attachment.ksh>


More information about the vsipl++ mailing list