[vsipl++] [patch] Profile_event class

Don McCoy don at codesourcery.com
Thu Jul 20 00:37:05 UTC 2006


Thank you Jules for the suggestions and the detailed explanations as
well.  All have been addressed with this revision.  The important points
being:

   - impl_performance() works independently of the profiler being enabled
   - It does share a timer resource internally for now
   - FFT/FFTM size is now part of the descriptive tag in the profile output.
   - Tags now use S,D,C,Z for float, double, complex<float> and
complex<double> respectively.
   - The tags are now generated at one point - from the base class
(rather than in each derived Fft type).

Also, thanks to Stefan for helping me find a defect in my handling of
the timer resource.  This was related to making an unintentional copy of
the object owning the timer.  I mention this because it helped me also
to realize that when an Fft object is copied a new timer is created.
This effectively erases the history that impl_performance() is able to
report.  I don't think this is unreasonable behavior, but thought I'd
point it out.

Regards,

-- 
Don McCoy
don (at) CodeSourcery
(888) 776-0262 / (650) 331-3385, x712

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pf6.changes
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20060719/70f7dd82/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pf6.diff
URL: <http://sourcerytools.com/pipermail/vsipl++/attachments/20060719/70f7dd82/attachment-0001.ksh>


More information about the vsipl++ mailing list