[FFmpeg-devel] [PATCH 2/3] avcodec/tak: remove unnecessary calls to emms_c()

[James Almer]

On 21/02/15 9:18 AM, Paul B Mahol wrote:
> On 2/21/15, James Almer <jamrial at gmail.com> wrote:
>> It's already called by scalarproduct_int16 if required.
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/takdec.c | 3 ---
>>  1 file changed, 3 deletions(-)
>>
>> diff --git a/libavcodec/takdec.c b/libavcodec/takdec.c
>> index 0f808e0..77170b5 100644
>> --- a/libavcodec/takdec.c
>> +++ b/libavcodec/takdec.c
>> @@ -480,8 +480,6 @@ static int decode_subframe(TAKDecContext *s, int32_t
>> *decoded,
>>              memcpy(s->residues, &s->residues[y], 2 * filter_order);
>>      }
>>
>> -    emms_c();
>> -
>>      return 0;
>>  }
>>
>> @@ -641,7 +639,6 @@ static int decorrelate(TAKDecContext *s, int c1, int c2,
>> int length)
>>              memcpy(s->residues, &s->residues[tmp], 2 * filter_order);
>>          }
>>
>> -        emms_c();
>>          break;
>>      }
>>      }
>> --
>> 2.3.0
> 
> Should be OK if no crashes happens.

The new version of the previous patch i applied uses AV_ZERO128, which may use mmx code, 
so better keep these in place.

Patch dropped.