Skip to content

Commit 84936a5

Browse files
charles-typfacebook-github-bot
authored andcommitted
Update WDL to use latest folly
Summary: $title Differential Revision: D82332329
1 parent b71e24f commit 84936a5

2 files changed

Lines changed: 35 additions & 69 deletions

File tree

Lines changed: 34 additions & 68 deletions
Original file line numberDiff line numberDiff line change
@@ -1,54 +1,53 @@
11
diff --git a/CMakeLists.txt b/CMakeLists.txt
2-
index 44ba37edd..6ba5a69f1 100644
2+
index ecc11a796..0ffe5abe7 100644
33
--- a/CMakeLists.txt
44
+++ b/CMakeLists.txt
5-
@@ -566,8 +566,8 @@ install(
5+
@@ -568,6 +568,8 @@ install(
66
COMPONENT dev
77
)
88

9-
-option(BUILD_TESTS "If enabled, compile the tests." OFF)
10-
-option(BUILD_BENCHMARKS "If enabled, compile the benchmarks." OFF)
119
+option(BUILD_TESTS "If enabled, compile the tests." ON)
1210
+option(BUILD_BENCHMARKS "If enabled, compile the benchmarks." ON)
11+
option(BUILD_TESTS "If enabled, compile the tests." OFF)
12+
option(BUILD_BENCHMARKS "If enabled, compile the benchmarks." OFF)
1313
option(BUILD_BROKEN_TESTS "If enabled, compile tests that are known to be broken." OFF)
14-
option(BUILD_HANGING_TESTS "If enabled, compile tests that are known to hang." OFF)
15-
option(BUILD_SLOW_TESTS "If enabled, compile tests that take a while to run in debug mode." OFF)
16-
@@ -692,6 +692,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
17-
DIRECTORY container/test/
18-
TEST container_access_test SOURCES AccessTest.cpp
19-
TEST container_array_test SOURCES ArrayTest.cpp
14+
@@ -709,7 +711,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
15+
TEST container_fbvector_test SOURCES FBVectorTest.cpp
16+
BENCHMARK container_foreach_benchmark SOURCES ForeachBenchmark.cpp
17+
TEST container_foreach_test SOURCES ForeachTest.cpp
18+
- BENCHMARK container_hash_maps_bench SOURCES HashMapsBench.cpp
2019
+ BENCHMARK hash_maps_bench SOURCES HashMapsBench.cpp
21-
BENCHMARK container_bit_iterator_bench SOURCES BitIteratorBench.cpp
22-
TEST container_bit_iterator_test SOURCES BitIteratorTest.cpp
23-
TEST container_enumerate_test SOURCES EnumerateTest.cpp
24-
@@ -913,6 +914,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
20+
TEST container_heap_vector_types_test SOURCES heap_vector_types_test.cpp
21+
TEST container_map_util_test WINDOWS_DISABLED SOURCES MapUtilTest.cpp
22+
TEST container_merge_test SOURCES MergeTest.cpp
23+
@@ -916,7 +918,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
24+
2525
DIRECTORY io/test/
2626
TEST io_fs_util_test SOURCES FsUtilTest.cpp
27-
TEST io_iobuf_test WINDOWS_DISABLED SOURCES IOBufTest.cpp
27+
- BENCHMARK io_iobuf_benchmark WINDOWS_DISABLED SOURCES IOBufBenchmark.cpp
2828
+ BENCHMARK iobuf_benchmark WINDOWS_DISABLED SOURCES IOBufBenchmark.cpp
29+
TEST io_iobuf_test WINDOWS_DISABLED SOURCES IOBufTest.cpp
2930
TEST io_iobuf_cursor_test SOURCES IOBufCursorTest.cpp
3031
TEST io_iobuf_queue_test SOURCES IOBufQueueTest.cpp
31-
TEST io_record_io_test WINDOWS_DISABLED SOURCES RecordIOTest.cpp
32-
@@ -945,6 +947,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
33-
# TODO: Refactor EventHandlerTest to not use eventfd so it can work on Mac OS X.
34-
#TEST io_async_event_handler_test WINDOWS_DISABLED SOURCES EventHandlerTest.cpp
35-
TEST io_async_async_timeout_test SOURCES AsyncTimeoutTest.cpp
32+
@@ -957,7 +959,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
33+
SOURCES DelayedDestructionBaseTest.cpp
34+
TEST io_async_destructor_check_test SOURCES DestructorCheckTest.cpp
35+
# Fails with gtest macro error
36+
- BENCHMARK io_async_event_base_benchmark SOURCES EventBaseBenchmark.cpp
3637
+ BENCHMARK event_base_benchmark SOURCES EventBaseBenchmark.cpp
37-
TEST io_async_async_udp_socket_test APPLE_DISABLED WINDOWS_DISABLED
38-
SOURCES AsyncUDPSocketTest.cpp
39-
TEST io_async_delayed_destruction_test SOURCES DelayedDestructionTest.cpp
40-
@@ -1057,6 +1060,10 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
38+
TEST io_async_event_base_test BROKEN SOURCES EventBaseTest.cpp
39+
TEST io_async_event_base_local_test WINDOWS_DISABLED
40+
SOURCES EventBaseLocalTest.cpp
41+
@@ -1063,7 +1065,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
42+
TEST synchronization_rw_spin_lock_test SOURCES RWSpinLockTest.cpp
4143
TEST synchronization_semaphore_test WINDOWS_DISABLED
4244
SOURCES SemaphoreTest.cpp
45+
- BENCHMARK synchronization_small_locks_benchmark
46+
+ BENCHMARK small_locks_benchmark
47+
SOURCES SmallLocksBenchmark.cpp
4348
TEST synchronization_small_locks_test SOURCES SmallLocksTest.cpp
44-
+ BENCHMARK small_locks_benchmark SOURCES SmallLocksBenchmark.cpp
45-
+
46-
+ DIRECTORY test/function_benchmark/
47-
+ BENCHMARK function_benchmark SOURCES main.cpp benchmark_impl.cpp test_functions.cpp
4849

49-
DIRECTORY synchronization/detail/test/
50-
TEST synchronization_detail_hardware_test SOURCES HardwareTest.cpp
51-
@@ -1139,6 +1146,8 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
50+
@@ -1148,6 +1150,8 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
5251
TEST locks_test SOURCES SpinLockTest.cpp
5352
TEST math_test SOURCES MathTest.cpp
5453
TEST memcpy_test SOURCES MemcpyTest.cpp
@@ -57,45 +56,12 @@ index 44ba37edd..6ba5a69f1 100644
5756
TEST memory_idler_test SOURCES MemoryIdlerTest.cpp
5857
TEST memory_test WINDOWS_DISABLED
5958
SOURCES MemoryTest.cpp
60-
@@ -1165,6 +1174,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
61-
SOURCES ProducerConsumerQueueTest.cpp
62-
BENCHMARK range_find_benchmark SOURCES RangeFindBenchmark.cpp
63-
TEST random_test SOURCES RandomTest.cpp
64-
+ BENCHMARK random_benchmark SOURCES RandomBenchmark.cpp
65-
TEST range_test SOURCES RangeTest.cpp
66-
TEST replaceable_test WINDOWS_DISABLED SOURCES ReplaceableTest.cpp
67-
TEST scope_guard_test WINDOWS_DISABLED SOURCES ScopeGuardTest.cpp
68-
@@ -1227,6 +1237,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
59+
@@ -1240,7 +1244,7 @@ if (BUILD_TESTS OR BUILD_BENCHMARKS)
60+
folly_define_tests(
6961
DIRECTORY crypto/test/
7062
TEST crypto_blake2xb_test SOURCES Blake2xbTest.cpp
71-
TEST crypto_lt_hash_test SOURCES LtHashTest.cpp
63+
- BENCHMARK crypto_lt_hash_benchmark SOURCES LtHashBenchmark.cpp
7264
+ BENCHMARK lt_hash_benchmark SOURCES LtHashBenchmark.cpp
65+
TEST crypto_lt_hash_test SOURCES LtHashTest.cpp
7366
)
7467
endif()
75-
76-
diff --git a/folly/concurrency/test/ConcurrentHashMapBench.cpp b/folly/concurrency/test/ConcurrentHashMapBench.cpp
77-
index 770c47ed0..e1e5bf097 100644
78-
--- a/folly/concurrency/test/ConcurrentHashMapBench.cpp
79-
+++ b/folly/concurrency/test/ConcurrentHashMapBench.cpp
80-
@@ -214,7 +214,7 @@ void benches() {
81-
<< std::endl;
82-
std::cout << "Test name Max time Avg time Min time"
83-
<< std::endl;
84-
- for (int nthr : {1, 10}) {
85-
+ for (int nthr : {1, 8, 32, 64}) {
86-
std::cout << "========================= " << std::setw(2) << nthr
87-
<< " threads" << " =========================" << std::endl;
88-
bench_ctor_dtor(nthr, 0, "CHM ctor/dtor -- empty ");
89-
diff --git a/folly/lang/Hint-inl.h b/folly/lang/Hint-inl.h
90-
index 2c207a551..bd577ed98 100644
91-
--- a/folly/lang/Hint-inl.h
92-
+++ b/folly/lang/Hint-inl.h
93-
@@ -115,7 +115,7 @@ FOLLY_ALWAYS_INLINE void compiler_must_not_elide(T const& t, std::true_type) {
94-
95-
template <typename T>
96-
FOLLY_ALWAYS_INLINE void compiler_must_not_predict(T& t, std::false_type) {
97-
- asm volatile("" : "+r"(t));
98-
+ //asm volatile("" : "+r"(t));
99-
}
100-
101-
template <typename T>

packages/wdl_bench/install_wdl_bench.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ set -Eeuo pipefail
88
##################### BENCHMARK CONFIG #########################
99

1010
declare -A REPOS=(
11-
['folly']='https://github.com/facebook/folly.git'
11+
['folly']='7bce6fd7626b94e206d9ab66ac1ce34d82a1992c'
1212
['fbthrift']='https://github.com/facebook/fbthrift.git'
1313
['lzbench']='https://github.com/inikep/lzbench.git'
1414
['openssl']='https://github.com/openssl/openssl.git'

0 commit comments

Comments
 (0)