From: Igor Sugak Date: Fri, 6 Jan 2017 17:53:00 +0000 (-0800) Subject: fix gcc-5 build X-Git-Tag: v2017.03.06.00~129 X-Git-Url: http://plrg.eecs.uci.edu/git/?p=folly.git;a=commitdiff_plain;h=f65d8d2c92c7259fcfb5b94f3900cc92ca92e732 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 --- 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(