Merge pull request #31 from simdjson/patch_for_reflect_cpp

adds a patch to make reflect-cpp build

Author: FranciscoThiesen

CMakeLists.txt

@@ -24,6 +24,7 @@ CPMAddPackage("gh:nlohmann/json@3.10.5")
 CPMAddPackage(
   NAME reflect-cpp
   GITHUB_REPOSITORY getml/reflect-cpp
+  PATCHES reflect-cpp.patch
   GIT_TAG v0.13.0
 )
 

benchmarks/src/CMakeLists.txt

@@ -64,7 +64,7 @@ target_link_libraries(SerializationBenchmark PRIVATE simdjson-serial nlohmann_js
 
 # cpp-reflect may fail to build on LLVM 19 due to this issue
 # https://github.com/hanickadot/compile-time-regular-expressions/issues/307
-option(SIMDJSON_BENCH_CPP_REFLECT "include cpp reflect in the benchmarks" OFF)
+option(SIMDJSON_BENCH_CPP_REFLECT "include cpp reflect in the benchmarks" ON)
 if(SIMDJSON_BENCH_CPP_REFLECT)
   message(STATUS "Including reflect-cpp in the benchmarks.")
   add_library(benchmark_reflect_serialization_twitter STATIC benchmark_reflect_serialization_twitter.cpp)
@@ -74,7 +74,7 @@ if(SIMDJSON_BENCH_CPP_REFLECT)
   target_compile_definitions(SerializationTwitterBenchmark PRIVATE SIMDJSON_BENCH_CPP_REFLECT=1)
 
 
-  add_library(benchmark_reflect_serialization STATIC benchmark_reflect_serialization_twitter.cpp)
+  add_library(benchmark_reflect_serialization STATIC benchmark_reflect_serialization.cpp)
   target_link_libraries(benchmark_reflect_serialization PRIVATE reflectcpp)
 
   target_link_libraries(SerializationBenchmark PRIVATE benchmark_reflect_serialization)

benchmarks/src/benchmark_reflect_serialization.hpp

@@ -2,5 +2,5 @@
 #define BENCHMARK_REFLECT_SERIALIZATION_TWITTER_HPP
 
 #include "user_profile.hpp"
-void bench_reflect_cpp(std::vector<User>& data)
+void bench_reflect_cpp(std::vector<User>& data);
 #endif

cmake/CPM.cmake

@@ -2,8 +2,8 @@
 #
 # SPDX-FileCopyrightText: Copyright (c) 2019-2023 Lars Melchior and contributors
 
-set(CPM_DOWNLOAD_VERSION 0.39.0)
-set(CPM_HASH_SUM "66639bcac9dd2907b2918de466783554c1334446b9874e90d38e3778d404c2ef")
+set(CPM_DOWNLOAD_VERSION 0.40.0)
+set(CPM_HASH_SUM "7b354f3a5976c4626c876850c93944e52c83ec59a159ae5de5be7983f0e17a2a")
 
 if(CPM_SOURCE_CACHE)
   set(CPM_DOWNLOAD_LOCATION "${CPM_SOURCE_CACHE}/cpm/CPM_${CPM_DOWNLOAD_VERSION}.cmake")
@@ -21,4 +21,4 @@ file(DOWNLOAD
      ${CPM_DOWNLOAD_LOCATION} EXPECTED_HASH SHA256=${CPM_HASH_SUM}
 )
 
-include(${CPM_DOWNLOAD_LOCATION})
+include(${CPM_DOWNLOAD_LOCATION})