[FFmpeg-devel] [PATCH] avcodec/magicyuv: add SIMD for median of 10bits

James Almer jamrial at gmail.com
Sat Dec 24 01:18:24 EET 2016


On 12/23/2016 8:00 PM, Ronald S. Bultje wrote:
> Hi,
> 
> On Fri, Dec 23, 2016 at 12:32 PM, Paul B Mahol <onemda at gmail.com> wrote:
> 
>> diff --git a/libavcodec/lossless_videodsp.h b/libavcodec/lossless_
>> videodsp.h
>>
> [..]
> 
>> @@ -32,6 +32,7 @@ typedef struct LLVidDSPContext {
>>
> [..]
> 
>> +    void (*add_magy_median_pred_int16)(uint16_t *dst, const uint16_t
>> *top, const uint16_t *diff, unsigned mask, int w, int *left, int *left_top);
>>
> 
> That seems wrong. Why would you add a magicuv-specific function to
> losslessdsp-context which is intended for functions shared between many
> (not just one) lossless codecs? You probably want a new dsp for magicyuv
> specifically.
> 
> I know this is tedious, but we're very specifically trying to prevent
> dsputil from ever happening again.
> 
> Ronald

Some functions in this dsp are used only by huffyuv. Only one is used by
both huffyuv and magicyuv.
To properly apply what you mention, it would need to be split in two,
huffyuvdsp and lldsp, then this new function added to a new dsp called
magicyuvdsp.



More information about the ffmpeg-devel mailing list