Enabling ARM optimizations for Opus on Android and ChromeOS

third_party/opus/opus.gyp

Index: third_party/opus/opus.gyp
diff --git a/third_party/opus/opus.gyp b/third_party/opus/opus.gyp
index be0ae4e01efe77acc50a3ae8d6343e89849926c7..f153073b3fd6425e484511a64a573e7b03bf4895 100644
--- a/third_party/opus/opus.gyp
+++ b/third_party/opus/opus.gyp
@@ -7,6 +7,7 @@
     'conditions': [
       ['OS=="android"', {
         'use_opus_fixed_point%': 1,
+        'opus_arm_optimization%': 1,

[tlegrand1, 2014/02/11 15:45:07]

As you see we only enable for Android at the moment.

[Sergey Ulanov, 2014/02/13 20:34:28]

Android is now always ARM (e.g. there are Intel-based Android devices), and ARM
not always Android. So this doesn't look right to me. Should we look for
target_arch=="arm" instead?

       }, {
         'use_opus_fixed_point%': 0,
       }],
@@ -65,6 +66,41 @@
           'sources/': [
             ['exclude', '/float/[^/]*_FLP.(h|c)$'],
           ],
+          'conditions': [
+            ['opus_arm_optimization==1', {
+              'defines': [
+                'OPUS_ARM_ASM',
+                'OPUS_ARM_INLINE_ASM',
+                'OPUS_ARM_INLINE_EDSP',
+                'OPUS_ARM_MAY_HAVE_EDSP',
+                'OPUS_ARM_MAY_HAVE_MEDIA',
+                'OPUS_ARM_MAY_HAVE_NEON',
+                'OPUS_HAVE_RTCD',
+              ],
+              'include_dirs': [
+                'src/celt/arm',
+              ],
+              'includes': [
+                'opus_srcs_arm.gypi',
+              ],
+              'conditions': [
+                ['(target_arch=="arm" and arm_version==7) or target_arch=="armv7"', {

[minyue, 2014/02/18 13:52:56]

We may try to change this line to

['target_arch=="arm" and arm_fpu=="neon"'],

and remove

  88                   'cflags': [
  89                     '-mfpu=neon',
  90                   ],
  91                   'cflags/': [
  92                     ['exclude', '^-mfpu=.*'],
  93                     ['include', '-mfpu=neon'],
  94                   ],

+                  'cflags': [
+                    '-mfpu=neon',
+                  ],
+                  'cflags/': [
+                    ['exclude', '^-mfpu=.*'],
+                    ['include', '-mfpu=neon'],
+                  ],
+                }, {
+                  'sources!': [
+                    'src/celt/pitch_arm.h',
+                    'src/celt/arm/celt_pitch_xcorr_arm_gnu.S',
+                  ],
+                }],
+              ],
+            }],
+          ],
         }],
       ],
     },  # target opus