Commit e39dfd9d authored by Lucian Grijincu's avatar Lucian Grijincu Committed by Sara Golemon

folly: improve ProducerConsumerQueueBenchmark (misplaced doNotOptimizeAway)

Summary:
as per discussion from https://phabricator.fb.com/D1274352

After changing the queue length to be a small value (100) I didn't see
improvement from cache alignment of atomics (too much noise). Dropping
that and just fixing the benchmark.

Test Plan: n/a

Reviewed By: ngbronson@fb.com

FB internal diff: D1276591
parent 1321eef0
......@@ -72,8 +72,8 @@ struct ThroughputTest {
for (int i = 0; i < iters_; ++i) {
ThroughputType item = 0;
while (!queue_.read(item)) {
doNotOptimizeAway(item);
}
doNotOptimizeAway(item);
}
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment