Age | Commit message (Collapse) | Author |
|
|
|
|
|
Call celt_inner_prod_neon() and remove redundant code.
Change-Id: I980e94330ae75c10297b9035fac221515aee144f
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
The floating-point optimizations are not bit exact with C functions,
because of the different orders of floating-point operations.
But they are bit exact with the simulation C functions which simulate
the floating operations in the optimizations.
Change-Id: I149fda5b602fd5712b16fc8983df3c6c0c9e76ad
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
This optimization is bit exact with C functions.
Change-Id: Ia9ce6dd3c20d2f56dbd43ddc02d1a6fd6554608d
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Should call celt_inner_prod().
This requires the API change of celt_pitch_xcorr() by passing in
"arch".
We tested on x86 and arm, and got bit exact results as original.
Change-Id: I606915da6a196f327ce81f4a5ae32811f4c1fabb
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Should call celt_inner_prod().
This change is bit exact as original, except for x86 floating-point.
In x86 floating-point, it calls celt_inner_prod_sse() which may have
different output with the change of floating-point operations' orders.
Change-Id: Ia2381e2e198a84296ac28305183f15be842b3454
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
|
|
|
|
This was sometimes causing the music probability to be stuck at
zero at the beginning of files
|
|
It's unnecessary, and isn't defined correctly on floating-point.
This makes us correctly use Neon functions (in floating-point mode) on
platforms where Neon is detected by RTCD.
Signed-off-by: Timothy B. Terriberry <tterribe@xiph.org>
|
|
|
|
|
|
|
|
|
|
We boost bands that either cause leakage or are filled with leakage
|
|
Now scaling the energy to the same value is would be in floating-point
|
|
It seems like letting CBR use up to 2/3 of the bit is still a win
|
|
Thanks to petrufm for pointing that out:
https://github.com/xiph/opus/issues/35
|
|
This makes it possible to run the testvectors with asan enabled.
|
|
We now include the object files for those rather than attempt to
problems.
|
|
And run the build request only at the last configuration.
Also add a check if api_key is unset to not try to request a new build.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
The appveyor script needs an encrypted api key to trigger
downstream builds, and there doesn't seem to be an easy
way to make the builds we run for the main repo something
other contributors can control.
Signed-off-by: Ricardo Constantino <wiiaboo@gmail.com>
|
|
Using my API key for now, but needs rillian's to work.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
Artifact with preserved paths is more useful for CI.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
Less clutter on systems that hide dotfiles.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
|
The code would have run fine on 32-bit archs, but would have overflowed
on a 16-bit arch
|
|
Some informal tests seem to confirm that reducing the trim at 32-64 kbps
improves quality (better HF). It's not clear whether it's also the case
at 96 kb/s and above, so we're leaving it as is for those rates.
This corresponds to buildC in this thread:
https://hydrogenaud.io/index.php/topic,113985.0.html
Also see:
https://hydrogenaud.io/index.php/topic,111798.0.html
|
|
Change-Id: I607a8b75b0711a485384d6f854cf6e2ec18b38b7
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
The constant was copied from the analysis code which uses +/-1 as float
scaling, but surround_analysis() uses +/-32767.
|
|
The optimization is bit exact with C function.
Change-Id: Ifb8f04b19f2d576e79ce5dcfa7e0fc374d71d6c8
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Split to silk_biquad_alt_stride1() and silk_biquad_alt_stride2(),
so that it can be optimized more efficiently when stride is 2.
This change in C code is bit exact with the origin.
Change-Id: Idaefe670397016ace2a489e3435ac61b7dbe79d5
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Makes things a little easier to read.
Signed-off-by: Gian-Carlo Pascutto <gcp@sjeng.org>
|
|
Make it easier for users unfamiliar with C applications
to installed the necessary build dependencies.
Signed-off-by: Gian-Carlo Pascutto <gcp@sjeng.org>
|
|
This fixes an issue with gcc 4.9 on Debian 8, at least,
which defines __STDC__ but not __STDC_VERSION__, unlike
more recent gcc.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
The optimization is bit exact with C function.
This optimization speeds up fixed-point SILK encoder on NEON about
5% to 8%. (Tested on Acer Chromebook, ARMv7 Processor rev 3 (v7l).)
Change-Id: I582f6f3585b7946149e16a2ad3084ebc0ae79a4f
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
|
|
C4146: unary minus operator applied to unsigned type, result still unsigned
Fixed in a previous commit.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
This warning complains of files that don't define any new symbols
not being included in linking.
GCC doesn't seem to complain about those, so neither should VS.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
Closes https://github.com/xiph/opus/pull/34
|
|
opus_compare.c defines the bitstream so we avoid changing it by
ignoring the non-breaking warning instead of changing the file.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
Resolves https://github.com/xiph/opus/issues/21
|
|
- 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
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
Closes https://github.com/xiph/opus/pull/26
|
|
They're only needed in that project, no need to be in common.props.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Should make the projects a bit easier to digest and the differences
between the different configurations and platforms be more obvious.
A (happy?) side effect of this is the static libraries are now about
the same size as the ones produced by GCC and seem to build fine with
opus-tools.
Needs testing with other third party applications to make sure nothing
is missing that shouldn't.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|