Skip to content
This repository was archived by the owner on Apr 28, 2023. It is now read-only.

Commit eec8932

Browse files
Bump default tuner_threads to 8
The previous default of 1 makes it too easy to run the tuner sequentially and claim that TC is slow. Bumping to 8 CPU threads by default. This is arguably unreasonable but if one cannot afford 8 threads for autotuning TCs, one should not be in the business of running the tuner in the first place.
1 parent c0fb4a1 commit eec8932

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tc/core/flags.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ DEFINE_uint32(
8080
tuner_gen_number_elites,
8181
10,
8282
"The number of best candidates that are preserved intact between generations");
83-
DEFINE_uint32(tuner_threads, 1, "Number of CPU threads to use when autotuning");
83+
DEFINE_uint32(tuner_threads, 8, "Number of CPU threads to use when autotuning");
8484
DEFINE_string(
8585
tuner_devices,
8686
"0",

0 commit comments

Comments
 (0)