Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(179)

Side by Side Diff: webrtc/common_audio/signal_processing/downsample_fast.c

Issue 2293893002: Add functions to interact with ASan and MSan, and some sample uses (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #include "webrtc/common_audio/signal_processing/include/signal_processing_librar y.h" 11 #include "webrtc/common_audio/signal_processing/include/signal_processing_librar y.h"
12 12
13 #include "webrtc/base/checks.h"
14 #include "webrtc/base/sanitizer.h"
15
13 // TODO(Bjornv): Change the function parameter order to WebRTC code style. 16 // TODO(Bjornv): Change the function parameter order to WebRTC code style.
14 // C version of WebRtcSpl_DownsampleFast() for generic platforms. 17 // C version of WebRtcSpl_DownsampleFast() for generic platforms.
15 int WebRtcSpl_DownsampleFastC(const int16_t* data_in, 18 int WebRtcSpl_DownsampleFastC(const int16_t* data_in,
16 size_t data_in_length, 19 size_t data_in_length,
17 int16_t* data_out, 20 int16_t* data_out,
18 size_t data_out_length, 21 size_t data_out_length,
19 const int16_t* __restrict coefficients, 22 const int16_t* __restrict coefficients,
20 size_t coefficients_length, 23 size_t coefficients_length,
21 int factor, 24 int factor,
22 size_t delay) { 25 size_t delay) {
26 int16_t* const original_data_out = data_out;
23 size_t i = 0; 27 size_t i = 0;
24 size_t j = 0; 28 size_t j = 0;
25 int32_t out_s32 = 0; 29 int32_t out_s32 = 0;
26 size_t endpos = delay + factor * (data_out_length - 1) + 1; 30 size_t endpos = delay + factor * (data_out_length - 1) + 1;
27 31
28 // Return error if any of the running conditions doesn't meet. 32 // Return error if any of the running conditions doesn't meet.
29 if (data_out_length == 0 || coefficients_length == 0 33 if (data_out_length == 0 || coefficients_length == 0
30 || data_in_length < endpos) { 34 || data_in_length < endpos) {
31 return -1; 35 return -1;
32 } 36 }
33 37
38 rtc_MsanCheckInitialized(coefficients, sizeof(coefficients[0]),
39 coefficients_length);
40
34 for (i = delay; i < endpos; i += factor) { 41 for (i = delay; i < endpos; i += factor) {
35 out_s32 = 2048; // Round value, 0.5 in Q12. 42 out_s32 = 2048; // Round value, 0.5 in Q12.
36 43
37 for (j = 0; j < coefficients_length; j++) { 44 for (j = 0; j < coefficients_length; j++) {
45 rtc_MsanCheckInitialized(&data_in[i - j], sizeof(data_in[0]), 1);
38 out_s32 += coefficients[j] * data_in[i - j]; // Q12. 46 out_s32 += coefficients[j] * data_in[i - j]; // Q12.
39 } 47 }
40 48
41 out_s32 >>= 12; // Q0. 49 out_s32 >>= 12; // Q0.
42 50
43 // Saturate and store the output. 51 // Saturate and store the output.
44 *data_out++ = WebRtcSpl_SatW32ToW16(out_s32); 52 *data_out++ = WebRtcSpl_SatW32ToW16(out_s32);
45 } 53 }
46 54
55 RTC_DCHECK_EQ(original_data_out + data_out_length, data_out);
56 rtc_MsanCheckInitialized(original_data_out, sizeof(original_data_out[0]),
57 data_out_length);
58
47 return 0; 59 return 0;
48 } 60 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698