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..fb37e15b623736d3bbbdf5b586d0b4eb5e157e8f 100644
--- a/third_party/opus/opus.gyp
+++ b/third_party/opus/opus.gyp
@@ -5,10 +5,18 @@
 {
   'variables': {
     'conditions': [
-      ['OS=="android"', {
+      ['OS=="android" or chromeos==1', {
         'use_opus_fixed_point%': 1,

[Sergey Ulanov, 2014/03/19 19:21:51]

This will enable fixed-point math on Intel Chromebooks. It doesn't look correct
to me. If fixed-point is faster on Intel CPUs then we should always use it, not
only on ChromeOS. I _think_ we need just one condition that sets both
use_opus_fixed_point and use_opus_arm_optimization on ARM builds. Or is it that
you need fixed-point on Intel Android builds.

[tlegrand1, 2014/03/20 10:11:01]

I wanted to preserve the previous setting for Android, where we always used
fixed point. But I think you are right, we don't need fixed point in Intel CPUs.
Now is the best time to make the change.

On 2014/03/19 19:21:51, Sergey Ulanov wrote:

+        'conditions': [
+          ['target_arch=="arm"', {
+            'use_opus_arm_optimization%': 1,
+          }, {
+            'use_opus_arm_optimization%': 0,
+          }],
+        ],
       }, {
         'use_opus_fixed_point%': 0,
+        'use_opus_arm_optimization%': 0,
       }],
     ],
   },
@@ -65,9 +73,25 @@
           'sources/': [
             ['exclude', '/float/[^/]*_FLP.(h|c)$'],
           ],
+          'conditions': [
+            ['use_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',
+              ],
+              'includes': [
+                'opus_arm_srcs.gypi',
+              ],
+            }],
+          ],
         }],
       ],
-    },  # target opus
+    },
     {
       'target_name': 'opus_demo',
       'type': 'executable',
@@ -95,6 +119,6 @@
         'src/celt',
         'src/silk',
       ],
-    },  # target opus_demo
+    },
   ]
 }