Commit 1880e851 authored by Christopher Dykes's avatar Christopher Dykes Committed by Facebook Github Bot

Don't use named variadic macro parameters

Summary: MSVC doesn't support them, so use `__VA_ARGS__` instead.

Reviewed By: yfeldblum

Differential Revision: D3507071

fbshipit-source-id: 186834c87e74257c709fcbc08e3bda7b1b55d02b
parent b9d8fba1
......@@ -37,11 +37,12 @@ namespace test {
// This is ugly, but better perf for DeterministicAtomic translates
// directly to more states explored and tested
#define FOLLY_TEST_DSCHED_VLOG(msg...) \
do { \
if (false) { \
VLOG(2) << std::hex << std::this_thread::get_id() << ": " << msg; \
} \
#define FOLLY_TEST_DSCHED_VLOG(...) \
do { \
if (false) { \
VLOG(2) << std::hex << std::this_thread::get_id() << ": " \
<< __VA_ARGS__; \
} \
} while (false)
/**
......
......@@ -1371,8 +1371,8 @@ BENCHMARK(single_thread_lock_unlock, iters) {
}
}
#define BENCH_BASE(args...) BENCHMARK_NAMED_PARAM(args)
#define BENCH_REL(args...) BENCHMARK_RELATIVE_NAMED_PARAM(args)
#define BENCH_BASE(...) BENCHMARK_NAMED_PARAM(__VA_ARGS__)
#define BENCH_REL(...) BENCHMARK_RELATIVE_NAMED_PARAM(__VA_ARGS__)
// 100% reads. Best-case scenario for deferred locks. Lock is colocated
// with read data, so inline lock takes cache miss every time but deferred
......
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