From a1d8ce76d552a915b6cdd320a439bc186672f5b6 Mon Sep 17 00:00:00 2001 From: Alaric de Ruiter Date: Sat, 4 Jan 2025 14:59:28 +0100 Subject: [PATCH 1/2] fixed key lengths --- bench/sse42.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/bench/sse42.c b/bench/sse42.c index f13919e..018b75c 100644 --- a/bench/sse42.c +++ b/bench/sse42.c @@ -42,7 +42,7 @@ struct put_small_keys_sse42 { UBENCH_F_SETUP(put_small_keys_sse42) { const unsigned max_keys = 1024 * 1024; - const unsigned key_len = 8; + const unsigned key_len = 8+1; char *const keys = malloc(max_keys * key_len); unsigned i; @@ -95,7 +95,7 @@ struct put_large_keys_sse42 { UBENCH_F_SETUP(put_large_keys_sse42) { const unsigned max_keys = 1024 * 1024; - const unsigned key_len = 1024; + const unsigned key_len = 1024+1; char *const keys = malloc(max_keys * key_len); unsigned i; @@ -133,7 +133,7 @@ struct get_small_keys_sse42 { UBENCH_F_SETUP(get_small_keys_sse42) { const unsigned max_keys = 1024 * 1024; - const unsigned key_len = 8; + const unsigned key_len = 8+1; char *const keys = malloc(max_keys * key_len); unsigned i; struct hashmap_s hashmap; @@ -184,7 +184,7 @@ struct get_large_keys_sse42 { UBENCH_F_SETUP(get_large_keys_sse42) { const unsigned max_keys = 1024 * 1024; - const unsigned key_len = 1024; + const unsigned key_len = 1024+1; char *const keys = malloc(max_keys * key_len); unsigned i; struct hashmap_s hashmap; From d4068f451978a2a63d52df8bea8baf39f7e8743f Mon Sep 17 00:00:00 2001 From: Alaric de Ruiter Date: Sat, 4 Jan 2025 15:00:30 +0100 Subject: [PATCH 2/2] linked math library for platforms that don't do it automatically --- bench/CMakeLists.txt | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/bench/CMakeLists.txt b/bench/CMakeLists.txt index f16ea8e..72fb24f 100644 --- a/bench/CMakeLists.txt +++ b/bench/CMakeLists.txt @@ -34,6 +34,11 @@ add_executable(hashmap_bench sse42.c ) +find_library(MATH_LIBRARY m) +if(MATH_LIBRARY) + target_link_libraries(hashmap_bench PUBLIC ${MATH_LIBRARY}) +endif() + if (NOT "${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "arm64") if("${CMAKE_C_COMPILER_ID}" STREQUAL "GNU") set_source_files_properties(sse42.c PROPERTIES