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

Unified Diff: third_party/WebKit/Source/platform/audio/DirectConvolver.cpp

Issue 2384073002: reflow comments in platform/audio (Closed)
Patch Set: comments (heh!) Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/audio/DirectConvolver.cpp
diff --git a/third_party/WebKit/Source/platform/audio/DirectConvolver.cpp b/third_party/WebKit/Source/platform/audio/DirectConvolver.cpp
index c5728b351f3bf7a08a49fce7b8777c501cb75e78..a77ec195c71348b42e8420c1baab4f759170f861 100644
--- a/third_party/WebKit/Source/platform/audio/DirectConvolver.cpp
+++ b/third_party/WebKit/Source/platform/audio/DirectConvolver.cpp
@@ -84,8 +84,9 @@ void DirectConvolver::process(AudioFloatArray* convolutionKernel,
#else
size_t i = 0;
#if CPU(X86) || CPU(X86_64)
- // Convolution using SSE2. Currently only do this if both |kernelSize| and |framesToProcess|
- // are multiples of 4. If not, use the straightforward loop below.
+ // Convolution using SSE2. Currently only do this if both |kernelSize| and
+ // |framesToProcess| are multiples of 4. If not, use the straightforward loop
+ // below.
if ((kernelSize % 4 == 0) && (framesToProcess % 4 == 0)) {
// AudioFloatArray's are always aligned on at least a 16-byte boundary.
@@ -105,7 +106,8 @@ void DirectConvolver::process(AudioFloatArray* convolutionKernel,
convolutionSum = _mm_setzero_ps();
- // |kernelSize| is a multiple of 4 so we can unroll the loop by 4, manually.
+ // |kernelSize| is a multiple of 4 so we can unroll the loop by 4,
+ // manually.
for (size_t k = 0; k < kernelSize; k += 4) {
size_t dataOffset = i + k;
@@ -123,7 +125,9 @@ void DirectConvolver::process(AudioFloatArray* convolutionKernel,
} else {
#endif
-// FIXME: The macro can be further optimized to avoid pipeline stalls. One possibility is to maintain 4 separate sums and change the macro to CONVOLVE_FOUR_SAMPLES.
+// FIXME: The macro can be further optimized to avoid pipeline stalls. One
+// possibility is to maintain 4 separate sums and change the macro to
+// CONVOLVE_FOUR_SAMPLES.
#define CONVOLVE_ONE_SAMPLE \
do { \
sum += inputP[i - j] * kernelP[j]; \
« no previous file with comments | « third_party/WebKit/Source/platform/audio/DenormalDisabler.h ('k') | third_party/WebKit/Source/platform/audio/Distance.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698