Corrected comments for x86_sse_avx_find_peaks() intrinsics built function
authorPaul Davis <paul@linuxaudiosystems.com>
Wed, 13 May 2015 01:15:48 +0000 (21:15 -0400)
committerPaul Davis <paul@linuxaudiosystems.com>
Mon, 29 Jun 2015 18:18:13 +0000 (14:18 -0400)
libs/ardour/sse_functions_avx.cc

index 8c076aacb5fd5a61707a4083d74f7267c8f018e9..0b314948c52d862b47e2e328d048ea6db9d1497d 100644 (file)
@@ -28,7 +28,7 @@ x86_sse_avx_find_peaks(const ARDOUR::Sample* buf, ARDOUR::pframes_t nframes, flo
 {
        __m256 current_max, current_min, work;
 
-       // Load max and min values into all four slots of the XMM registers
+       // Load max and min values into all eight slots of the YMM registers
        current_min = _mm256_set1_ps(*min);
        current_max = _mm256_set1_ps(*max);
 
@@ -49,8 +49,8 @@ x86_sse_avx_find_peaks(const ARDOUR::Sample* buf, ARDOUR::pframes_t nframes, flo
                // load each 64 bytes into cash before processing
         while (nframes >= 16) {
 #if defined(COMPILER_MSVC) || defined(COMPILER_MINGW)                                   
-                               _mm_prefetch(((char*)buf+64), _mm_hint(0) );  // A total guess! Assumed to be eqivalent to
-#else                                              // the line below but waiting to be tested !!
+                               _mm_prefetch(((char*)buf+64), _mm_hint(0) );
+#else    
                 __builtin_prefetch(buf+64,0,0);
 #endif
                 work = _mm256_load_ps(buf);
@@ -112,9 +112,8 @@ x86_sse_avx_find_peaks(const ARDOUR::Sample* buf, ARDOUR::pframes_t nframes, flo
 
        *max = current_max[0];
 
-       // zero upper 128 bit of 256 bit ymm register to avoid penalties using non AVX instructions
+       // zero upper 128 bit of 256 bit ymm register to avoid penalties using non-AVX instructions
        _mm256_zeroupper ();
 }
 
 
-