From 4c279b852c7f3c92654fa09f813259ff427d1830 Mon Sep 17 00:00:00 2001 From: aluebs Date: Tue, 8 Mar 2016 01:48:17 -0800 Subject: [PATCH] Drop 48kHz sample rate support in the APM for ARM architecture The 3-band splitting filter is highly complex on this architecture. Today this is not a problem, because on those platforms we mostly use AECM which forces us to downsample to 16kHz anyway, but this is a way of guarding against it. In the long term we want to optimize the 3-band splitting filter for ARM architectures, but for now we can just disable it. Review URL: https://codereview.webrtc.org/1766103002 Cr-Commit-Position: refs/heads/master@{#11900} --- webrtc/modules/audio_processing/audio_processing_impl.cc | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/webrtc/modules/audio_processing/audio_processing_impl.cc b/webrtc/modules/audio_processing/audio_processing_impl.cc index a7c120f395..3c67a9bf28 100644 --- a/webrtc/modules/audio_processing/audio_processing_impl.cc +++ b/webrtc/modules/audio_processing/audio_processing_impl.cc @@ -112,8 +112,12 @@ struct AudioProcessingImpl::ApmPrivateSubmodules { const int AudioProcessing::kNativeSampleRatesHz[] = { AudioProcessing::kSampleRate8kHz, AudioProcessing::kSampleRate16kHz, +#ifdef WEBRTC_ARCH_ARM_FAMILY + AudioProcessing::kSampleRate32kHz}; +#else AudioProcessing::kSampleRate32kHz, AudioProcessing::kSampleRate48kHz}; +#endif // WEBRTC_ARCH_ARM_FAMILY const size_t AudioProcessing::kNumNativeSampleRates = arraysize(AudioProcessing::kNativeSampleRatesHz); const int AudioProcessing::kMaxNativeSampleRateHz = AudioProcessing::