search for: inline_optim

...tion that NEON will be supported... so as Jonathan Lennox suggested earlier, most of these flags can be hardcoded..... even run time checks can be disabled as ARMv8 cpu in aarch64 mode is guaranteed to have NEON/ASIMD. > >> AS_GCC_INLINE_ASSEMBLY( >> [inline_optimization="ARM"], >> [inline_optimization="disabled"] >> @@ -212,6 +211,35 @@ AS_IF([test x"${enable_asm}" = x"yes"],[ >> AS_ASM_ARM_MEDIA([OPUS_ARM_INLINE_MEDIA=1], >> [OPUS_ARM_INLINE_ME...

...e 1.1 release is out but unfortunately due to a relatively late change to add additional ARM NEON support the commit in question broke the autoconf script.. checking for C99 variable-size arrays... yes checking for cos in -lm... (cached) yes /home/ports/pobj/opus-1.1/opus-1.1/configure[12816]: ${inline_optimization:0:3}": bad substitution AM_CONDITIONAL([OPUS_ARM_INLINE_ASM], [test x"${inline_optimization:0:3}" = x"ARM"]) AM_CONDITIONAL([OPUS_ARM_EXTERNAL_ASM], [test x"${asm_optimization:0:3}" = x"ARM"]) -- This message has been scanned for viru...

--- configure.ac | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/configure.ac b/configure.ac index 019d36d..13ed33f 100644 --- a/configure.ac +++ b/configure.ac @@ -199,6 +199,25 @@ cpu_arm=no AS_IF([test x"${enable_asm}" = x"yes"],[ inline_optimization="No inline ASM for your platform, please send patches" case $host_cpu in + arm64*|aarch64*) + dnl Currently we only have asm for fixed-point + AS_IF([test "$enable_float" != "yes"],[ + cpu_arm64=yes + AC_DEFINE([OPUS_ARM...

--- configure.ac | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/configure.ac b/configure.ac index 6f61d2e..4f7ec75 100644 --- a/configure.ac +++ b/configure.ac @@ -199,6 +199,25 @@ cpu_arm=no AS_IF([test x"${enable_asm}" = x"yes"],[ inline_optimization="No inline ASM for your platform, please send patches" case $host_cpu in + arm64*|aarch64*) + dnl Currently we only have asm for fixed-point + AS_IF([test "$enable_float" != "yes"],[ + cpu_arm64=yes + AC_DEFINE([OPUS_ARM...

Hello, I received feedback from engineers working on NE10 [1] that it would be better to use NE10 [1] for FFT optimizations for opus use cases. However, these FFT patches are currently in review and haven't been integrated into NE10 yet. While the FFT functions in NE10 are getting baked, I wanted to optimize the celt_pitch_xcorr (floating point only) and use it to introduce ARM NEON

...NTR = \ +celt/arm/celt_neon_intr.c + CELT_AM_SOURCES_ARM_ASM = \ celt/arm/armopts.s.in diff --git a/configure.ac b/configure.ac index 9b2f51f..09657b6 100644 --- a/configure.ac +++ b/configure.ac @@ -198,12 +198,11 @@ cpu_arm=no AS_IF([test x"${enable_asm}" = x"yes"],[ inline_optimization="No ASM for your platform, please send patches" + OPUS_ARM_NEON_INTR_CPPFLAGS= case $host_cpu in arm*) - dnl Currently we only have asm for fixed-point - AS_IF([test "$enable_float" != "yes"],[ cpu_arm=yes - A...

On 25 November 2014 at 10:11, Viswanath Puttagunta <viswanath.puttagunta at linaro.org> wrote: > > On 25 November 2014 at 09:39, Jonathan Lennox <jonathan at vidyo.com> wrote: > > > > On Nov 25, 2014, at 10:07 AM, Viswanath Puttagunta <viswanath.puttagunta at linaro.org> wrote: > >> > >> > Also is there plans to make the NEON optimisations

...ax_pitch, arch)) > +#endif > + And this. > diff --git a/configure.ac b/configure.ac > index 9b2f51f..09657b6 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -198,12 +198,11 @@ cpu_arm=no > > AS_IF([test x"${enable_asm}" = x"yes"],[ > inline_optimization="No ASM for your platform, please send patches" > + OPUS_ARM_NEON_INTR_CPPFLAGS= > case $host_cpu in > arm*) > - dnl Currently we only have asm for fixed-point > - AS_IF([test "$enable_float" != "yes"],[ >...

...only have asm for fixed-point 205 AS_IF([test "$enable_float" != "yes"],[ 206 cpu_arm64=yes 207 AC_DEFINE([OPUS_ARM64_ASM], [], [Make use of ARM64 asm optimization]) 208 AS_GCC_INLINE_ASSEMBLY( 209 [inline_optimization="ARM64"], 210 [inline_optimization="disabled"] 211 ) 212 AS_IF([test x"$inline_optimization" = x"ARM64"],[ 213 AM_CONDITIONAL([OPUS_ARM64_INLINE_ASM],[true]) 214...

...> 205 AS_IF([test "$enable_float" != "yes"],[ > 206 cpu_arm64=yes > 207 AC_DEFINE([OPUS_ARM64_ASM], [], [Make use of ARM64 asm > optimization]) > 208 AS_GCC_INLINE_ASSEMBLY( > 209 [inline_optimization="ARM64"], > 210 [inline_optimization="disabled"] > 211 ) > 212 AS_IF([test x"$inline_optimization" = x"ARM64"],[ > 213 AM_CONDITIONAL([OPUS_ARM64_INLINE_ASM],[true]) &g...

...9;d continue to use NEON here with vld1_dup_f32()/vmla_f32()/etc. > +} > +#if ((defined(OPUS_ARM_ASM) && defined(FIXED_POINT)) \ > + || defined(OPUS_ARM_NEON_INTR)) > #include "arm/arm_celt_map.c" > #endif > > + Unrelated whitespace change. > - inline_optimization="No ASM for your platform, please send patches" > + inline_optimization="No in-line ASM for your platform, please send patches" "inline" is one word. > + AC_MSG_CHECKING(if compiler supports arm neon intrinsics) Capitalize ARM and NEON, please....

...b/configure.ac index 0ba4a80..443362f 100644 --- a/configure.ac +++ b/configure.ac @@ -333,9 +333,9 @@ AS_IF([test x"${enable_asm}" = x"yes"],[ AM_CONDITIONAL([CPU_ARM], [test "$cpu_arm" = "yes"]) AM_CONDITIONAL([OPUS_ARM_INLINE_ASM], - [test x"${inline_optimization:0:3}" = x"ARM"]) + [test x"${inline_optimization%% *}" = x"ARM"]) AM_CONDITIONAL([OPUS_ARM_EXTERNAL_ASM], - [test x"${asm_optimization:0:3}" = x"ARM"]) + [test x"${asm_optimization%% *}" = x"ARM"]) AS_IF(...

...onfigure.ac.orig Thu Dec 5 00:26:45 2013 +++ configure.ac Sun Dec 15 17:32:25 2013 @@ -333,9 +333,9 @@ AS_IF([test x"${enable_asm}" = x"yes"],[ AM_CONDITIONAL([CPU_ARM], [test "$cpu_arm" = "yes"]) AM_CONDITIONAL([OPUS_ARM_INLINE_ASM], - [test x"${inline_optimization:0:3}" = x"ARM"]) + [expr x"${inline_optimization}" : x"ARM" >/dev/null]) AM_CONDITIONAL([OPUS_ARM_EXTERNAL_ASM], - [test x"${asm_optimization:0:3}" = x"ARM"]) + [expr x"${asm_optimization}" : x"ARM" >/...

Hi, Optimizes celt_pitch_xcorr for floating point. Changes from RFCv1: - Rebased on top of commit aad281878: Fix celt_pitch_xcorr_c signature. which got rid of ugly code around CELT_PITCH_XCORR_IMPL passing of "arch" parameter. - Unified with --enable-intrinsics used by x86 - Modified algorithm to be more in-line with algorithm in celt_pitch_xcorr_arm.s Viswanath Puttagunta

Hi, Optimizes celt_pitch_xcorr for ARM NEON floating point. Changes from RFCv3: - celt_neon_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,

...nable intrinsics optimizations (only for fixed point x86)])],, + [AS_HELP_STRING([--enable-intrinsics], [Enable intrinsics optimizations for ARM(float) X86(fixed)])],, [enable_intrinsics=no]) rtcd_support=no cpu_arm=no AS_IF([test x"${enable_asm}" = x"yes"],[ - inline_optimization="No ASM for your platform, please send patches" + inline_optimization="No inline ASM for your platform, please send patches" case $host_cpu in arm*) dnl Currently we only have asm for fixed-point @@ -343,7 +343,6 @@ AS_IF([test x"${enable_asm}...

...nable intrinsics optimizations (only for fixed point x86)])],, + [AS_HELP_STRING([--enable-intrinsics], [Enable intrinsics optimizations for ARM(float) X86(fixed)])],, [enable_intrinsics=no]) rtcd_support=no cpu_arm=no AS_IF([test x"${enable_asm}" = x"yes"],[ - inline_optimization="No ASM for your platform, please send patches" + inline_optimization="No in-line ASM for your platform, please send patches" case $host_cpu in arm*) dnl Currently we only have asm for fixed-point @@ -343,7 +343,6 @@ AS_IF([test x"${enable_asm...

...nable intrinsics optimizations (only for fixed point x86)])],, + [AS_HELP_STRING([--enable-intrinsics], [Enable intrinsics optimizations for ARM(float) X86(fixed)])],, [enable_intrinsics=no]) rtcd_support=no cpu_arm=no AS_IF([test x"${enable_asm}" = x"yes"],[ - inline_optimization="No ASM for your platform, please send patches" + inline_optimization="No inline ASM for your platform, please send patches" case $host_cpu in arm*) dnl Currently we only have asm for fixed-point @@ -343,7 +343,6 @@ AS_IF([test x"${enable_asm}...

Hi, Optimizes celt_pitch_xcorr for floating point. Changes from RFCv2: - Changes recommended by Timothy for celt_neon_intr.c everything except, left the unrolled loop still unrolled - configure.ac - use AC_LINK_IFELSE instead of AC_COMPILE_IFELSE - Moved compile flags into Makefile.am - OPUS_ARM_NEON_INR --> typo --> OPUS_ARM_NEON_INTR Viswanath Puttagunta (1): armv7:

From: Viswanath Puttagunta <viswanath.puttagunta at linaro.org> Hi, Optimizes celt_pitch_xcorr for floating point. Changes from RFCv1: - Rebased on top of commit aad281878: Fix celt_pitch_xcorr_c signature. which got rid of ugly code around CELT_PITCH_XCORR_IMPL passing of "arch" parameter. - Unified with --enable-intrinsics used by x86 - Modified algorithm to be more