[FFmpeg-devel] [PATCH 3/3] avcodec/utvideodec: use cached bitstream reader

[Paul B Mahol]

On 8/23/18, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> 2018-08-23 16:00 GMT+02:00, Paul B Mahol <onemda at gmail.com>:
>> On 8/23/18, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>>> 2018-08-23 15:35 GMT+02:00, Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>>>> 2018-08-23 15:08 GMT+02:00, Paul B Mahol <onemda at gmail.com>:
>>>>> From 100x real-time decoding to 138x real-time decoding for 320x240
>>>>> video.
>>>>
>>>> On x86_64 I get an even better improvement, on x86_32
>>>> decoding gets slower by approximately 10%.
>>>> Without the patch, decoding is faster on x86_32 than
>>>> x86_64 here...
>>>
>>> Using vanilla gcc-6.4.
>>>
>>> clang 3.4:
>>> 5% slower with patch on x86_32
>>>>40% faster with patch on x86_64
>>> x86_32 again faster than x86_64 without patch.
>>
>> x86_32 is going to be less and less used.
>
> Is there a reason why we cannot use
> the change in the x86_64 case (and others that we
> show to be faster) but not for x86_32?

The define could be only enabled for x86_64.