From 7b6914f91dcf8f2ec0b28f4e70385053c4083389 Mon Sep 17 00:00:00 2001
From: Paul Ferrand <paul@ferrand.cc>
Date: Sat, 30 May 2020 22:53:41 +0200
Subject: [PATCH] Corrected the benchmarks to properly set the SIMD operations

---
 benchmarks/BM_divide.cpp | 4 ++++
 benchmarks/BM_gain.cpp   | 6 ++++++
 2 files changed, 10 insertions(+)

diff --git a/benchmarks/BM_divide.cpp b/benchmarks/BM_divide.cpp
index 770fac78..48f5064e 100644
--- a/benchmarks/BM_divide.cpp
+++ b/benchmarks/BM_divide.cpp
@@ -46,6 +46,7 @@ BENCHMARK_DEFINE_F(Divide, Straight)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(Divide, Scalar)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::divide, false);
         sfz::divide<float>(input, divisor, absl::MakeSpan(output));
     }
 }
@@ -53,6 +54,7 @@ BENCHMARK_DEFINE_F(Divide, Scalar)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(Divide, SIMD)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::divide, true);
         sfz::divide<float>(input, divisor, absl::MakeSpan(output));
     }
 }
@@ -60,6 +62,7 @@ BENCHMARK_DEFINE_F(Divide, SIMD)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(Divide, Scalar_Unaligned)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::divide, false);
         sfz::divide<float>(absl::MakeSpan(input).subspan(1), absl::MakeSpan(divisor).subspan(1), absl::MakeSpan(output).subspan(1));
     }
 }
@@ -67,6 +70,7 @@ BENCHMARK_DEFINE_F(Divide, Scalar_Unaligned)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(Divide, SIMD_Unaligned)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::divide, true);
         sfz::divide<float>(absl::MakeSpan(input).subspan(1), absl::MakeSpan(divisor).subspan(1), absl::MakeSpan(output).subspan(1));
     }
 }
diff --git a/benchmarks/BM_gain.cpp b/benchmarks/BM_gain.cpp
index 3a753389..cd2096d7 100644
--- a/benchmarks/BM_gain.cpp
+++ b/benchmarks/BM_gain.cpp
@@ -66,6 +66,7 @@ BENCHMARK_DEFINE_F(GainSingle, Straight)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(GainSingle, Scalar)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::gain, false);
         sfz::applyGain<float>(gain, input, absl::MakeSpan(output));
     }
 }
@@ -73,6 +74,7 @@ BENCHMARK_DEFINE_F(GainSingle, Scalar)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(GainSingle, SIMD)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::gain, true);
         sfz::applyGain<float>(gain, input, absl::MakeSpan(output));
     }
 }
@@ -88,6 +90,7 @@ BENCHMARK_DEFINE_F(GainArray, Straight)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(GainArray, Scalar)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::gain, false);
         sfz::applyGain<float>(gain, input, absl::MakeSpan(output));
     }
 }
@@ -95,6 +98,7 @@ BENCHMARK_DEFINE_F(GainArray, Scalar)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(GainArray, SIMD)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::gain, true);
         sfz::applyGain<float>(gain, input, absl::MakeSpan(output));
     }
 }
@@ -102,6 +106,7 @@ BENCHMARK_DEFINE_F(GainArray, SIMD)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(GainArray, Scalar_Unaligned)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::gain, false);
         sfz::applyGain<float>(absl::MakeSpan(gain).subspan(1), absl::MakeSpan(input).subspan(1), absl::MakeSpan(output).subspan(1));
     }
 }
@@ -109,6 +114,7 @@ BENCHMARK_DEFINE_F(GainArray, Scalar_Unaligned)(benchmark::State& state) {
 BENCHMARK_DEFINE_F(GainArray, SIMD_Unaligned)(benchmark::State& state) {
     for (auto _ : state)
     {
+        sfz::setSIMDOpStatus(sfz::SIMDOps::gain, true);
         sfz::applyGain<float>(absl::MakeSpan(gain).subspan(1), absl::MakeSpan(input).subspan(1), absl::MakeSpan(output).subspan(1));
     }
 }