[vsipl++] [patch] SIMD 'rscvmul' evaluators

Mark Mitchell mark at codesourcery.com
Tue Sep 5 20:08:43 UTC 2006


Don McCoy wrote:
> This patch corrects a problem where two of the SIMD evaluators were not 
> handling re-dimensioned (2-D --> 1-D) views correctly.  This only 
> affects the case where an element-wise multiplication is being performed 
> with a real scalar and a complex view (and the view was re-dim'd).
> 
> It is worth mentioning also that this defect was uncovered using the 
> profiling features recently added.   In general, the evaluators were 
> doing the right thing, but when they did not, it fell back to using loop 
> fusion, thereby still getting the correct answer (just not taking 
> advantage of SIMD instructions).

This is very nice on several levels: nice that the profilers found the 
problem, nice that the bug was performance degradation rather than 
failure, and nice that you were able to easily fix it!

-- 
Mark Mitchell
CodeSourcery
mark at codesourcery.com
(650) 331-3385 x713



More information about the vsipl++ mailing list