[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