diff options
| author | Rostislav Pehlivanov <atomnuker@gmail.com> | 2017-08-18 17:27:41 +0100 |
|---|---|---|
| committer | Rostislav Pehlivanov <atomnuker@gmail.com> | 2017-08-18 17:27:41 +0100 |
| commit | 8e53cd1fab86b8774f0b5b0f186d8e0cd9c1895b (patch) | |
| tree | 767ed1735644b59fa6c26e8cf4b730e7485cbb99 | |
| parent | 7205513f8f4b32c403c733d7d2ce2f440837397d (diff) | |
ops_pvq_search: remove dead macro
There's no point in toggling it, even for debugging. Its just worse.
Signed-off-by: Rostislav Pehlivanov <atomnuker@gmail.com>
| -rw-r--r-- | libavcodec/x86/opus_pvq_search.asm | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/libavcodec/x86/opus_pvq_search.asm b/libavcodec/x86/opus_pvq_search.asm index f0d9950e34..beb6cbcc9b 100644 --- a/libavcodec/x86/opus_pvq_search.asm +++ b/libavcodec/x86/opus_pvq_search.asm @@ -34,13 +34,6 @@ ALIGNMODE p6 ; Opus also does use rsqrt approximation in their intrinsics code. %define USE_APPROXIMATION 1 -; Presearch tries to quantize by using the property Sum( abs(x[i]*K)/Sx ) = K. -; If we use truncation for the division result then the integer Sum would be <= K. -; By using nearest rounding we get closer approximation, but -; we could also get more than K pulses and we have to remove the extra ones. -; This method is the main improvement of the ffmpeg C function over the Opus original. -%define PRESEARCH_ROUNDING 1 - SECTION_RODATA 64 const_float_abs_mask: times 8 dd 0x7fffffff @@ -286,11 +279,7 @@ align 16 %%loop_guess: movaps m1, [tmpX + r4] ; m1 = X[i] mulps m2, m0, m1 ; m2 = res*X[i] - %if PRESEARCH_ROUNDING == 0 - cvttps2dq m2, m2 ; yt = (int)truncf( res*X[i] ) - %else cvtps2dq m2, m2 ; yt = (int)lrintf( res*X[i] ) - %endif paddd m5, m2 ; Sy += yt cvtdq2ps m2, m2 ; yt = (float)yt mulps m1, m2 ; m1 = X[i]*yt |