Implement SIMD conversion in BinaryRecording.cpp

Author: jsiegle

Source/Processors/RecordNode/BinaryFormat/BinaryRecording.cpp

@@ -22,6 +22,7 @@
 */
 
 #include "BinaryRecording.h"
+#include "SIMDConverter.h"
 
 #include "../../Settings/DataStream.h"
 #include "../../Settings/InfoObject.h"
@@ -651,10 +652,10 @@ void BinaryRecording::writeContinuousData (int writeChannel,
         m_bufferSize = size;
     }
 
-    /* Convert signal from float to int w/ bitVolts scaling */
-    double multFactor = 1 / (float (0x7fff) * getContinuousChannel (realChannel)->getBitVolts());
-    FloatVectorOperations::copyWithMultiply (m_scaledBuffer.getData(), dataBuffer, multFactor, size);
-    AudioDataConverters::convertFloatToInt16LE (m_scaledBuffer.getData(), m_intBuffer.getData(), size);
+    /* Convert signal from float to int16 w/ bitVolts scaling using SIMD-optimized conversion.
+       The scale factor converts microvolts to int16 units: output = input / bitVolts */
+    float scaleFactor = 1.0f / getContinuousChannel (realChannel)->getBitVolts();
+    SIMDConverter::convertFloatToInt16 (dataBuffer, m_intBuffer.getData(), scaleFactor, size);
 
     /* Get the file index that belongs to the current recording channel */
     int fileIndex = m_fileIndexes[writeChannel];

Tests/Processors/RecordNodeTests.cpp

@@ -206,8 +206,8 @@ class RecordNodeTests :  public testing::Test {
     }
 
     static int16_t minValPossible() {
-        // The min value is actually -32767 in the math in RecordNode, not -32768 like the "true" min for int16_t
-        return (std::numeric_limits<int16_t>::min)() + 1;
+        // The SIMDConverter correctly uses the full int16 range [-32768, 32767]
+        return (std::numeric_limits<int16_t>::min)();
     }
 
     static int16_t maxValPossible() {