[FFmpeg-devel] [PATCH 03/12] Add vector_clipf_interleave() function to dsputil

Måns Rullgård mans
Tue Sep 29 10:01:48 CEST 2009


Michael Niedermayer <michaelni at gmx.at> writes:

> On Sun, Sep 27, 2009 at 11:49:19AM +0100, Mans Rullgard wrote:
> [...]
>> diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h
>> index 8af06ff..86ea1d0 100644
>> --- a/libavcodec/dsputil.h
>> +++ b/libavcodec/dsputil.h
>> @@ -399,6 +399,9 @@ typedef struct DSPContext {
>>                                      int step, int len);
>>      void (*int32_to_float_fmul_scalar)(float *dst, const int *src, float mul, int len);
>>      void (*vector_clipf)(float *dst /* align 16 */, const float *src /* align 16 */, float min, float max, int len /* align 16 */);
>> +    void (*vector_clipf_interleave)(float *dst, const float **src,
>> +                                    float min, float max, int len,
>> +                                    int channels);
>
> i think all pointers should require 16byte alignment

I said I'll add proper doxygen comments if the functions as such are
agreed upon.  That will include 16-byte alignment here.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list