summaryrefslogtreecommitdiff
path: root/celt/fixed_debug.h
Commit message (Collapse)AuthorAgeFilesLines
* Relaxing checks for MULT16_32_QX()Jean-Marc Valin2022-07-221-2/+2
| | | | | | | | MULT16_32_QX() is now implemented using a signed-unsigned multiply, so the second argument can now have one extra bit compared to the old signed-signed implementation. Reviewed by Mark Harris
* Avoid left shifts of negative values in debug macrosJean-Marc Valin2022-07-051-3/+3
| | | | Reviewed by Mark Harris
* Silence some warnings for fixed-point debug buildsJean-Marc Valin2022-06-281-12/+12
| | | | Reviewed by Timothy B. Terriberry.
* celt_lpc: avoid overflows when computing lpcs in fixed pointFelicia Lim2021-03-021-0/+45
| | | | | | | The LPCs are computed in 32-bit, so increase the allowed range from +/-8 to +/-64 to avoid overflows caught during fuzzing. Before downshifting back down to the +/-8 range in the final 16-bit output, perform bandwidth extension to avoid any additional overflow issues.
* Fix compiler warningsMark Harris2017-02-261-2/+4
| | | | | | | | | | | | | | | | | | | - celt/modes.c:430:14: warning: cast from 'const unsigned char *' to 'opus_int16 *' increases required alignment from 1 to 2 [-Wcast-align] - 'C[0][1]' may be used uninitialized [-Wmaybe-uninitialized] - Unused variable/parameter - Value stored is never read - MSVC warnings about "possible loss of data" due to type conversions - MSVC warning C4146: unary minus operator applied to unsigned type - silk/NLSF_del_dec_quant.c:137:20: warning: array subscript is above array bounds [-Warray-bounds] (gcc -O3 false positive) - src/mlp_train.h:39:20: warning: function declaration isn't a prototype [-Wstrict-prototypes] - Remove SMALL_FOOTPRINT code from SSE 4.1 FIR implementation, matching the C implementation. The clang -Wcast-align warnings with SSE intrinsics are a known clang issue: https://llvm.org/bugs/show_bug.cgi?id=20670
* fixed_debug.h: delete some dead codeJames Zern2017-01-121-3/+0
| | | | | | this has the side-effect of removing some C++ style comments Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
* fixes overflows in celt_iir() by doing proper saturationJean-Marc Valin2016-07-221-0/+2
|
* silencing overflows in MDCT and FFTJean-Marc Valin2016-07-221-0/+6
|
* Adds SIG2WORD16() to fixed_debug.hJean-Marc Valin2014-06-181-0/+10
|
* Speed up the comb filter on ARM by using MAC16_32_Q16()Jean-Marc Valin2014-01-211-0/+1
|
* Adds SATURATE16() to the fixed-point debug build tooJean-Marc Valin2013-11-171-0/+10
|
* Replace "inline" with OPUS_INLINE.Gregory Maxwell2013-10-281-29/+29
| | | | | | | | | Newer versions of MSVC are unhappy with the strategy of the build environment redefining "inline" (even though they don't support the actual keyword). Instead we define OPUS_INLINE to the right thing in opus_defines.h. This is the same approach we use for restrict.
* Replace long long in celt/ with opus_int64.Gregory Maxwell2012-08-211-40/+40
|
* Use 64-bits in fixed point debug for _P16, fixes spurious failures.Gregory Maxwell2012-08-191-4/+4
|
* Add OPUS_{GET|SET}_GAIN CTLs for adjusting output gain.Gregory Maxwell2012-07-111-0/+35
| | | | This CTL was requested by Nicolas George for FFmpeg.
* When FIXED_DEBUG_ASSERT CELT will assert on fixed-debug error.Gregory Maxwell2012-05-151-1/+212
|
* Merge commit '390c89225d'Jean-Marc Valin2012-04-241-2/+2
|\
| * s/FOUNDATION/COPYRIGHT OWNER/ in CELT code and "glue code"Jean-Marc Valin2012-04-201-2/+2
| | | | | | | | Also added 3rd clause to "master" COPYING file
* | Fixes several overflows in the CELT fixed-pointJean-Marc Valin2012-04-121-3/+4
| | | | | | | | | | These were all mostly benign and would at worst result in (rare) suboptimal encoder decisions rather than signal corruption.
* | Check for underflow in the debug USUB32.Ralph Giles2012-03-051-0/+4
| |
* | Print the actual arguments to the debug UADD/SUB32 routines.Ralph Giles2012-03-051-4/+4
| | | | | | | | | | | | | | | | | | | | | | The versions of UADD32 and USUB32 in fixed_debug.h take unsigned long long (ideally 64 bit) inputs, and range check the arguments. However, the printed warnings cast them to (unsigned), which is confusing. Instead, print the full value passed in. NB this will make negative values print has huge, rather than negative, values.
* | Uncomment the reporting for UADD32 and USUB32 in fixed_debug.h.Ralph Giles2012-03-051-2/+2
| |
* | Used unsigned intermediates with unsigned arguments in fixed_debug.h.Ralph Giles2012-03-051-2/+2
| | | | | | | | Corrects a signed-compare warning.
* | Fix --enable-fixed-point-debug.Ralph Giles2012-03-051-1/+2
|/ | | | | | Use the celt_mips defined in fixed_debug.h from the unit tests by defining CELT_C earlier. We must export celt_mips so it can be used by clients calling the library, like opus_custom_demo.
* Rename '_FOO' to avoid potentional collisions with reserved identifiers.Ralph Giles2011-12-021-30/+30
| | | | | | | | | | | | | | C reserves identifiers of the from _[A-Z]+ and we have a number of those in the code. This patch renames the various function arguments, MACROS and preprocessor symbols to avoid the reserved form. It also removes the CHANNELS() macro altogether. This was a minor optimization for TI DSP to force a mono-only build, as were the associated local 'const' versions. Since stereo support is manditory, it wasn't worth keeping. Thanks to John Ridges for raising the issue, and Jean-Marc Valin and Greg Maxwell for reviewing the changes.
* renames the libcelt/ directory to celt/Jean-Marc Valin2011-09-131-0/+511
|
* Moved all SILK source code to the silk/ directoryJean-Marc Valin2011-04-281-6/+0
|
* Loss rate adaptation for the CELT layerJean-Marc Valin2011-04-251-5/+5
|
* draft updateJean-Marc Valin2011-03-311-5/+5
|
* Build fixesJean-Marc Valin2011-03-231-5/+5
|
* Making mode switching use the same window as CELT (squared)Jean-Marc Valin2011-03-211-5/+5
|
* CELT updateJean-Marc Valin2011-03-181-5/+5
| | | | With minor fixes
* More project files updateJean-Marc Valin2011-03-181-5/+5
|
* Fixes a few PLC/DTX bugs due to the recent decode API changeJean-Marc Valin2011-03-161-5/+5
|
* Adding constrained VBR modeJean-Marc Valin2011-03-141-5/+5
|
* Draft update (allocationGregory Maxwell2011-03-141-5/+5
|
* Disable newly introduced CELT signallingJean-Marc Valin2011-03-111-5/+5
|
* Nothing to see hereJean-Marc Valin2011-03-101-5/+5
|
* CELT updateJean-Marc Valin2011-03-091-18/+5
|
* Support for glitchles mode switchingJean-Marc Valin2011-03-071-5/+18
| | | | | | Uses a 5ms redundant CELT frame embedded into the SILK or hybrid packet to handle the switching. It's still possible to use the PLC-based method when no redundant packet is included.
* SILK/CELT updateJean-Marc Valin2011-02-281-5/+5
|
* preparing the next versionJean-Marc Valin2011-02-141-5/+5
|
* missing filesJean-Marc Valin2011-02-141-10/+5
|
* SILK and CELT updatesJean-Marc Valin2011-02-141-5/+10
|
* Update for in-band FECKoen Vos2011-02-141-5/+5
|
* draft dateJean-Marc Valin2011-02-041-0/+6
|
* CELT updateJean-Marc Valin2011-02-041-6/+0
|
* Tuning the hybrid bit-rate splitKoen Vos2011-02-041-24/+5
|
* Oops, fixed the CELT versionJean-Marc Valin2011-02-031-0/+25
|
* Update Opus range coder due to CELT refactoring.Timothy B. Terriberry2011-02-031-6/+0
| | | | | The byte buffer is now part of the range coder struct itself, and rangeenc.c and rangedec.c have gone away.