Commit 3e3ac86d authored by Sandesh Venkatesh's avatar Sandesh Venkatesh
Browse files

Merge branch '875-possibe-saturation-in-fb-bwe' into 'main' 

proposed fix to 875

Closes #875

See merge request !584
parents d4167508 17d51d7d

lib_com/options.h

+1 −0
Original line number Diff line number Diff line
@@ -180,6 +180,7 @@ 
#define FIX_869_WRONG_UVGAIN_STEP               /* VA: Fix wrong decoding of uv gain quantizer for td-stereo*/



#define FIX_871_REMOVE_UNNECESSARY_CONDITION    /* VA: remove a condition that is not needed and prevented correct frame classification of the secondary channel of the TD stereo */

#define FIX_875_SATURATION_DURING_ROUNDING      /* VA: fix a possible saturation when rounding */

/* ################## End DEVELOPMENT switches ######################### */



/* clang-format on */

lib_com/swb_tbe_com_fx.c

+4 −0
Original line number Diff line number Diff line
@@ -3671,7 +3671,11 @@ void GenShapedSHBExcitation_ivas_fx( 
        {

            FOR( j = 0; j < 32; ++j )

            {

#ifndef FIX_875_SATURATION_DURING_ROUNDING

                White_exc16k_FB_temp[i * 32 + j] = mult_r( White_exc16k_FB_temp[i * 32 + j], cos_fb_exc_fx[j] );

#else

                White_exc16k_FB_temp[i * 32 + j] = mult_r_sat( White_exc16k_FB_temp[i * 32 + j], cos_fb_exc_fx[j] );

#endif

                move16();

            }

        }