AAC encoder: refactor to resynchronize MIPS port
authorClaudio Freire <klaussfreire@gmail.com>
Tue, 15 Sep 2015 06:59:45 +0000 (03:59 -0300)
committerClaudio Freire <klaussfreire@gmail.com>
Thu, 17 Sep 2015 02:14:26 +0000 (23:14 -0300)
commit8df9bf8e39b5f38c75d63c0ef17965382e634b1c
treed9350e8cc6d1c1543facb7c58e477e669cc411f9
parent344d519040f03515bb915bf418f3ab9e38ada33b
AAC encoder: refactor to resynchronize MIPS port

This patch refactors the AAC coders to reuse code
between the MIPS port and the regular, portable C code.
There were two main functions that had to use
hand-optimized versions of quantization code:
 - search_for_quantizers_twoloop
 - codebook_trellis_rate

Those two were split into their own template header
files so they can be inlined inside both the MIPS port
and the generic code. In each context, they'll link
to their specialized implementations, and thus be
optimized by the compiler.

This approach I believe is better than maintaining
several copies of each function. As past experience has
proven, having to keep those in sync was error prone.
In this way, they will remain in sync by default.

Also, an implementation of the dequantized output
argument for the optimized quantize_and_encode
functions is included in the patch. While the current
implementation of search_for_pred still isn't using
it, future iterations of main prediction probably will.
It should not imply any measurable performance hit while
not being used.
libavcodec/aaccoder.c
libavcodec/aaccoder_trellis.h [new file with mode: 0644]
libavcodec/aaccoder_twoloop.h [new file with mode: 0644]
libavcodec/aacenc_quantization.h
libavcodec/mips/aaccoder_mips.c
tests/fate/aac.mak