From f65d8d2c92c7259fcfb5b94f3900cc92ca92e732 Mon Sep 17 00:00:00 2001 From: Igor Sugak Date: Fri, 6 Jan 2017 09:53:00 -0800 Subject: [PATCH] fix gcc-5 build Summary: ```lang=bash folly/test/SmallLocksBenchmark.cpp:212:18: error: use of undeclared identifier 'sqrt' double stdev = sqrt(accum / (results.size() - 1)); ^ folly/test/SmallLocksBenchmark.cpp:221:20: error: use of undeclared identifier 'sqrt' double stddev2 = sqrt(variance); ^ 2 errors generated. ``` Add the missing header. Reviewed By: yfeldblum Differential Revision: D4386343 fbshipit-source-id: 79c3dcae32c62f9b6d162862deabccf5fea7eaef --- folly/test/SmallLocksBenchmark.cpp | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/folly/test/SmallLocksBenchmark.cpp b/folly/test/SmallLocksBenchmark.cpp index db027e53..b3568d06 100644 --- a/folly/test/SmallLocksBenchmark.cpp +++ b/folly/test/SmallLocksBenchmark.cpp @@ -14,14 +14,16 @@ * limitations under the License. */ -#include -#include #include +#include #include #include #include #include +#include +#include + /* "Work cycle" is just an additional nop loop iteration. * A smaller number of work cyles will result in more contention, * which is what we're trying to measure. The relative ratio of @@ -209,7 +211,7 @@ static void runFairness() { std::for_each(results.begin(), results.end(), [&](const double d) { accum += (d - m) * (d - m); }); - double stdev = sqrt(accum / (results.size() - 1)); + double stdev = std::sqrt(accum / (results.size() - 1)); std::chrono::microseconds mx = *std::max_element(maxes.begin(), maxes.end()); std::chrono::microseconds agAqTime = std::accumulate( aqTime.begin(), aqTime.end(), std::chrono::microseconds(0)); @@ -218,7 +220,7 @@ static void runFairness() { std::chrono::microseconds mean = agAqTime / sum; double variance = (sum * agAqTimeSq - (agAqTime.count() * agAqTime.count())) / sum / (sum - 1); - double stddev2 = sqrt(variance); + double stddev2 = std::sqrt(variance); printf("Sum: %li Mean: %.0f stddev: %.0f\n", sum, m, stdev); printf( -- 2.34.1