[patch] Minor CFAR changes

Jules Bergmann jules at codesourcery.com
Thu Jun 8 21:47:28 UTC 2006


This patch makes some minor changes to CFAR.  For the slice version, it 
changes expressions to be more amenable to math library dispatch and 
changes the dimension-ordering to make subviews dense.  For the vector 
version, it fixes a bug with sum (should be reset to 0 for each vector), 
reduces the temporary footprint used, and uses get() instead of ().

Attached graphs show original (cfar-orig) and new (cfar) performance, 
for GCC 3.4 and GCC 4.1 on Pastec.  The changes for the slice version 
have a larger impact.  Using 4.1 is a win!

I am cleaning up the C, C-simd and VSIPL++ SIMD versions.  I'm planning 
to put the C and C-simd versions in a separate source file (cfar_c.cpp) 
and the VSIPL++ SIMD version in cfar.cpp (as t_cfar_hybrid).

I "fixed" gnuplot to avoid using both red and green, although the color 
choices still don't seem ideal.

				-- Jules

-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: cfar-1.diff
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20060608/f58f6383/attachment.ksh>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cfar-all-23.png
Type: image/png
Size: 4404 bytes
Desc: not available
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20060608/f58f6383/attachment.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cfar-all-3.png
Type: image/png
Size: 4515 bytes
Desc: not available
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20060608/f58f6383/attachment-0001.png>


More information about the vsipl++ mailing list