[FFmpeg-devel] [PATCH 18/21] avcodec/smacker: Use symbols table

[Paul B Mahol]

On 8/1/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> Up until now, the return value of get_vlc2() has been used as an index
> in an array that contained the value one is really interested in. Yet
> since b613bacca9c256f1483c46847f713e47a0e9a5f6 this is no longer
> necessary, as one can store the value that is right now stored in the
> array in the VLC internal table.
>
> This also means that all the information from the eight bit Huffman trees
> are now stored in the corresponding VLC table; this will enable us to
> remove several allocations lateron.
>
> This improved performance: For GCC 9 the time for one call of
> smka_decode_frame() for the sample from ticket #2425 decreased from
> 1811706 to 1794494 decicycles; for Clang 9 the number went from 1471663
> to 1449420 decicycles.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavcodec/smacker.c | 54 ++++++++++++++++++++++++--------------------
>  1 file changed, 30 insertions(+), 24 deletions(-)

Probably OK