On 13/07/14 10:36, Diego Biurrun wrote:
> On Sun, Jul 13, 2014 at 08:18:27AM +0000, Anton Khirnov wrote:
>> --- /dev/null
>> +++ b/libavcodec/x86/hevcdsp_init.c
>> @@ -0,0 +1,72 @@
>> +
>> +#include "config.h"
>> +
>> +#include "libavutil/cpu.h"
>> +#include "libavutil/x86/asm.h"
>> +#include "libavutil/x86/cpu.h"
>
> asm.h is unused.
>
>> +void ff_hevc_dsp_init_x86(HEVCDSPContext *c, const int bit_depth)
>> +{
>> + int mm_flags = av_get_cpu_flags();
>
> cpu_flags
>
>> + if (bit_depth == 8) {
>> + if (EXTERNAL_SSE2(mm_flags)) {
>> + c->hevc_v_loop_filter_chroma =
>> ff_hevc_v_loop_filter_chroma_8_sse2;
>> + c->hevc_h_loop_filter_chroma =
>> ff_hevc_h_loop_filter_chroma_8_sse2;
>> + }
>> + if (EXTERNAL_SSSE3(mm_flags) && ARCH_X86_64) {
>> + c->hevc_v_loop_filter_luma = ff_hevc_v_loop_filter_luma_8_ssse3;
>> + c->hevc_h_loop_filter_luma = ff_hevc_h_loop_filter_luma_8_ssse3;
>
> Why only on x86_64?
Because it uses too many registers for x86 I think.
lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
