From 7abf638436fa706bffb7b49e44110f4a79f5619b Mon Sep 17 00:00:00 2001 From: Erik Norvell Date: Fri, 23 Jan 2026 13:56:45 +0100 Subject: [PATCH] Small synch with BASOP to remove unnecessary abs and related comment --- lib_com/options.h | 1 + lib_rend/ivas_objectRenderer_hrFilt.c | 5 +++++ 2 files changed, 6 insertions(+) diff --git a/lib_com/options.h b/lib_com/options.h index 42a41806b2..bad3f056ec 100644 --- a/lib_com/options.h +++ b/lib_com/options.h @@ -165,6 +165,7 @@ #define FIX_2344_ALIGN_PREPROC /* VA: basop issue 2344: Align pre_proc_ivas() between FLP and BASOP */ #define FIX_FLOAT_1501_UNIT_VALUE_IN_OMASA /* Nokia: Fix float issue 1501, uninitialized value in ivas_masa_combine_directions for OMASA */ #define FIX_2331_CLANG18_MSAN_UNINIT_VARIABLE /* FhG: Fix issue 2331: Uninitialized variable */ +#define FIX_2235_TD_RENDERER_WORD16 /* Eri: For float: small synch with BASOP, removing unnecessary abs. BASOP: Use Word16 in TD renderer without converting to Word32 */ /* #################### End BE switches ################################## */ diff --git a/lib_rend/ivas_objectRenderer_hrFilt.c b/lib_rend/ivas_objectRenderer_hrFilt.c index 637318fd05..7cfbbc5ed5 100644 --- a/lib_rend/ivas_objectRenderer_hrFilt.c +++ b/lib_rend/ivas_objectRenderer_hrFilt.c @@ -461,7 +461,12 @@ static void getStandardBSplineSampVec( { d = 2 * ( BsLen[shape_idx] - 1 ) - d; } +#ifdef FIX_2235_TD_RENDERER_WORD16 + assert( d >= 0 && d < BsLen[shape_idx] ); + BfVec[i] = BsShape[BsStart[shape_idx] + d]; +#else BfVec[i] = BsShape[BsStart[shape_idx] + (int16_t) abs( d )]; /*TT, verify if abs is needed */ +#endif NzIdx[i] = nI + i; } -- GitLab