search for: test_unit_rotation

--- celt/tests/test_unit_dft.c | 3 +-- celt/tests/test_unit_mathops.c | 3 +-- celt/tests/test_unit_mdct.c | 3 +-- celt/tests/test_unit_rotation.c | 3 +-- 4 files changed, 4 insertions(+), 8 deletions(-) diff --git a/celt/tests/test_unit_dft.c b/celt/tests/test_unit_dft.c index bcee440..484dc57 100644 --- a/celt/tests/test_unit_dft.c +++ b/celt/tests/test_unit_dft.c @@ -47,8 +47,7 @@ #if defined(OPUS_X86_MAY_HAVE_SSE2) || defined(OPUS_...

--- celt/tests/test_unit_dft.c | 3 +-- celt/tests/test_unit_mathops.c | 3 +-- celt/tests/test_unit_mdct.c | 3 +-- celt/tests/test_unit_rotation.c | 3 +-- 4 files changed, 4 insertions(+), 8 deletions(-) diff --git a/celt/tests/test_unit_dft.c b/celt/tests/test_unit_dft.c index bcee440..484dc57 100644 --- a/celt/tests/test_unit_dft.c +++ b/celt/tests/test_unit_dft.c @@ -47,8 +47,7 @@ #if defined(OPUS_X86_MAY_HAVE_SSE2) || defined(OPUS_...

Fixes build failure on platforms with MAY_HAVE_SSE4_1 (but not PRESUME_SSE4_1) with --enable-intrinsics. --- silk/fixed/main_FIX.h | 11 +++++++++++ silk/x86/x86_silk_map.c | 2 ++ 2 files changed, 13 insertions(+) diff --git a/silk/fixed/main_FIX.h b/silk/fixed/main_FIX.h index ffeb4f3..375b5eb 100644 --- a/silk/fixed/main_FIX.h +++ b/silk/fixed/main_FIX.h @@ -97,6 +97,17 @@ void

Fixes build failure on platforms with MAY_HAVE_SSE4_1 (but not PRESUME_SSE4_1) with --enable-intrinsics. --- silk/fixed/main_FIX.h | 11 +++++++++++ silk/x86/x86_silk_map.c | 2 ++ 2 files changed, 13 insertions(+) diff --git a/silk/fixed/main_FIX.h b/silk/fixed/main_FIX.h index ffeb4f3..375b5eb 100644 --- a/silk/fixed/main_FIX.h +++ b/silk/fixed/main_FIX.h @@ -97,6 +97,17 @@ void

...on_intr.c - removed warnings due to not having constant pointers - Put simpler loop to take care of corner cases. Unrolling using intrinsics was not really mapping well to what was done in celt_pitch_xcorr_arm.s - Makefile.am Removed explicit -O3 optimization - test_unit_mathops.c, test_unit_rotation.c followed recommendation to use #if #elif to guarantee that only one of "arm/arm_celt_map.c" or "x86/x86_celt_map.c" is included Viswanath Puttagunta (1): armv7: celt_pitch_xcorr: Introduce ARM neon intrinsics Makefile.am | 12 ++ celt/ar...

...m/arm_celt_map.c | 15 ++- celt/arm/celt_neon_intr.c | 256 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 13 +- celt/cpu_support.h | 3 +- celt/pitch.h | 6 +- celt/tests/test_unit_mathops.c | 9 +- celt/tests/test_unit_rotation.c | 9 +- celt_sources.mk | 3 + configure.ac | 83 +++++++++++-- 10 files changed, 390 insertions(+), 19 deletions(-) create mode 100644 celt/arm/celt_neon_intr.c -- 1.7.9.5

.../arm_celt_map.c | 15 ++- celt/arm/celt_neon_intr.c | 242 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 13 ++- celt/cpu_support.h | 3 +- celt/pitch.h | 6 +- celt/tests/test_unit_mathops.c | 10 +- celt/tests/test_unit_rotation.c | 9 +- celt_sources.mk | 3 + configure.ac | 77 +++++++++++-- 10 files changed, 370 insertions(+), 19 deletions(-) create mode 100644 celt/arm/celt_neon_intr.c -- 1.7.9.5

...elt/arm/celt_ne10_mdct.c | 3 ++ celt/arm/pitch_arm.h | 2 +- celt/dump_modes/Makefile | 2 +- celt/pitch.h | 5 +-- celt/tests/test_unit_dft.c | 3 +- celt/tests/test_unit_mathops.c | 7 ++-- celt/tests/test_unit_mdct.c | 4 +- celt/tests/test_unit_rotation.c | 5 ++- configure.ac | 93 +++++++++++++++++++---------------------- 12 files changed, 67 insertions(+), 67 deletions(-) diff --git a/Makefile.am b/Makefile.am index 3a75740..8bd7447 100644 --- a/Makefile.am +++ b/Makefile.am @@ -47,7 +47,7 @@ if CPU_ARM CELT_SOURCES += $(C...

...| 4 +- celt/arm/arm_celt_map.c | 15 +++++++- celt/arm/celt_neon_intr.c | 81 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 15 +++++++- celt/pitch.h | 17 ++++++-- celt/tests/test_unit_mathops.c | 2 +- celt/tests/test_unit_rotation.c | 2 +- celt_sources.mk | 3 ++ configure.ac | 56 ++++++++++++++++++++++----- silk/SigProc_FIX.h | 4 +- silk/macros.h | 4 +- 13 files changed, 205 insertions(+), 25 deletions(-) create mode 100644 celt/arm/celt_ne...

...+++++++++++++++++++++++++++++++ celt/celt_lpc.h | 5 + celt/tests/test_unit_dft.c | 1 + celt/tests/test_unit_mathops.c | 1 + celt/tests/test_unit_mdct.c | 1 + celt/tests/test_unit_optimization_lpc.c | 96 ++++++++++++ celt/tests/test_unit_rotation.c | 1 + celt_headers.mk | 1 + celt_sources.mk | 1 + tests/test_unit_optimization.c | 62 ++++++++ 14 files changed, 541 insertions(+), 4 deletions(-) create mode 100644 celt/arm/celt_lpc_arm.h create mode 100644 celt/arm/c...

Hi all, This is Linfeng Zhang from Google. I'll work on ARM NEON optimization in the next few months. I'm submitting 2 patches in the following couple of emails, which have the new created celt_fir_neon(). I revised celt_fir_c() to not pass in argument "mem" in Patch 1. If there are concerns to this change, please let me know. Many thanks to your comments. Linfeng Zhang

...m/arm_celt_map.c | 15 ++- celt/arm/celt_neon_intr.c | 249 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 13 +- celt/cpu_support.h | 3 +- celt/pitch.h | 6 +- celt/tests/test_unit_mathops.c | 6 +- celt/tests/test_unit_rotation.c | 6 +- celt_sources.mk | 3 + configure.ac | 83 +++++++++++-- 10 files changed, 377 insertions(+), 19 deletions(-) create mode 100644 celt/arm/celt_neon_intr.c diff --git a/Makefile.am b/Makefile.am index e20f7b4..95323ca 100644 --- a/Makefile.am +++...

.../arm_celt_map.c | 15 ++- celt/arm/celt_neon_intr.c | 242 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 13 ++- celt/cpu_support.h | 3 +- celt/pitch.h | 6 +- celt/tests/test_unit_mathops.c | 10 +- celt/tests/test_unit_rotation.c | 9 +- celt_sources.mk | 3 + configure.ac | 77 +++++++++++-- 10 files changed, 370 insertions(+), 19 deletions(-) create mode 100644 celt/arm/celt_neon_intr.c diff --git a/Makefile.am b/Makefile.am index e20f7b4..0687814 100644 --- a/Makefile.am +++...

...m/arm_celt_map.c | 15 ++- celt/arm/celt_neon_intr.c | 256 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 13 +- celt/cpu_support.h | 3 +- celt/pitch.h | 6 +- celt/tests/test_unit_mathops.c | 9 +- celt/tests/test_unit_rotation.c | 9 +- celt_sources.mk | 3 + configure.ac | 83 +++++++++++-- 10 files changed, 390 insertions(+), 19 deletions(-) create mode 100644 celt/arm/celt_neon_intr.c diff --git a/Makefile.am b/Makefile.am index e20f7b4..be53754 100644 --- a/Makefile.am +++...

.../arm_celt_map.c | 15 ++- celt/arm/celt_neon_intr.c | 242 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 13 ++- celt/cpu_support.h | 3 +- celt/pitch.h | 6 +- celt/tests/test_unit_mathops.c | 10 +- celt/tests/test_unit_rotation.c | 9 +- celt_sources.mk | 3 + configure.ac | 77 +++++++++++-- 10 files changed, 370 insertions(+), 19 deletions(-) create mode 100644 celt/arm/celt_neon_intr.c -- 1.7.9.5

...5 ++- > celt/arm/celt_neon_intr.c | 249 +++++++++++++++++++++++++++++++++++++++ > celt/arm/pitch_arm.h | 13 +- > celt/cpu_support.h | 3 +- > celt/pitch.h | 6 +- > celt/tests/test_unit_mathops.c | 6 +- > celt/tests/test_unit_rotation.c | 6 +- > celt_sources.mk | 3 + > configure.ac | 83 +++++++++++-- > 10 files changed, 377 insertions(+), 19 deletions(-) > create mode 100644 celt/arm/celt_neon_intr.c > > diff --git a/Makefile.am b/Makefile.am > index e20f7b4..95...

...| 4 +- celt/arm/arm_celt_map.c | 15 +++++++- celt/arm/celt_neon_intr.c | 81 +++++++++++++++++++++++++++++++++++++++ celt/arm/pitch_arm.h | 15 +++++++- celt/pitch.h | 17 ++++++-- celt/tests/test_unit_mathops.c | 2 +- celt/tests/test_unit_rotation.c | 2 +- celt_sources.mk | 3 ++ configure.ac | 56 ++++++++++++++++++++++----- silk/SigProc_FIX.h | 4 +- silk/macros.h | 4 +- 13 files changed, 205 insertions(+), 25 deletions(-) create mode 100644 celt/arm/celt_ne...

...mium to validate licenses for third-party dependencies. Specifically the following files don't have copyright header: ./src/opus_compare.c ./src/mlp_data.c ./celt/tests/test_unit_dft.c ./celt/tests/test_unit_mathops.c ./celt/tests/test_unit_entropy.c ./celt/tests/test_unit_types.c ./celt/tests/test_unit_rotation.c ./celt/tests/test_unit_laplace.c ./celt/tests/test_unit_mdct.c ./celt/tests/test_unit_cwrs32.c ./autogen.sh ./win32/config.h ./tests/run_vectors.sh ./doc/build_draft.sh ./doc/build_oggdraft.sh Thanks! -- Sergey Ulanov

...rectory `/prj/avspw/karthikr/Development/BFamily/Broadcast/SDM845/Opus/opus-1.1.4 PASS: celt/tests/test_unit_types PASS: celt/tests/test_unit_mathops PASS: celt/tests/test_unit_entropy PASS: celt/tests/test_unit_laplace PASS: celt/tests/test_unit_dft PASS: celt/tests/test_unit_mdct PASS: celt/tests/test_unit_rotation PASS: celt/tests/test_unit_cwrs32 PASS: tests/test_opus_api PASS: tests/test_opus_decode ./test-driver: line 107: 36946 Segmentation fault (core dumped) "$@" > $log_file 2>&1 FAIL: tests/test_opus_encode PASS: tests/test_opus_padding ================================== op...

...#include "arm/arm_celt_map.c" > #endif You should keep the #elif (the intent was to _guarantee_ that only one of x86_celt_map.c or arm_celt_map.c would be included). Instead, just move the #if defined(OPUS_ARM_NEON_INTR) block inside that #elif (both in test_unit_mathops.c and test_unit_rotation.c).