Make bm_min_iters a 32-bit flag
authorChristopher Dykes <cdykes@fb.com>
Tue, 6 Dec 2016 18:51:35 +0000 (10:51 -0800)
committerFacebook Github Bot <facebook-github-bot-bot@fb.com>
Tue, 6 Dec 2016 19:53:26 +0000 (11:53 -0800)
Summary:
The benchmark API only supports `unsigned int` as the iteration number, so this being 64-bit means it has to be truncated for no reason.
This change gets us closer to being able to enable MSVC's implicit truncation warnings.

Reviewed By: yfeldblum

Differential Revision: D4281264

fbshipit-source-id: 27397e3f23bace20f3cc457665ea6f21bf994da5

folly/Benchmark.cpp

index b09faf2eec011697634d3fb4e2d6a34a12d77635..060c4bf4de0b3dbd489fa37cb716ca49c14371ba 100644 (file)
@@ -45,7 +45,7 @@ DEFINE_int64(
     100,
     "Minimum # of microseconds we'll accept for each benchmark.");
 
-DEFINE_int64(
+DEFINE_int32(
     bm_min_iters,
     1,
     "Minimum # of iterations we'll try for each benchmark.");