projects / folly.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 1321eef) | patch
folly: improve ProducerConsumerQueueBenchmark (misplaced doNotOptimizeAway)
authorLucian Grijincu <lucian@fb.com>
Tue, 15 Apr 2014 03:15:10 +0000 (20:15 -0700)
committerSara Golemon <sgolemon@fb.com>
Fri, 18 Apr 2014 19:04:14 +0000 (12:04 -0700)
commite39dfd9da543edf915060d4f981ae145d8b90c2b
tree93dde41fc828ec5cc0b437dbe1a44261901c21b9tree | snapshot
parent1321eef01e58721142b3a034e1638747e251b7f7commit | diff
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
folly/test/ProducerConsumerQueueBenchmark.cpp diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.