Add option for running sanitizers.

Author: greg7mdp

cmake/helpers.cmake

@@ -8,6 +8,14 @@ function(gtl_set_target_options my_target)
     $<$<CXX_COMPILER_ID:GNU>:-pedantic -Wall -Wextra -Wcast-align -Wcast-qual -Wdisabled-optimization -Winit-self -Wmissing-include-dirs -Woverloaded-virtual -Wredundant-decls -Wshadow -Wswitch-default -Wunused -Wno-interference-size>
     $<$<CXX_COMPILER_ID:MSVC>:/W4 /Zc:__cplusplus /bigobj>
   )
+
+  if (GTL_SANITIZE)
+    set(clang_san_options -g -fsanitize=undefined -fno-sanitize-recover=all -fsanitize=float-divide-by-zero
+                             -fsanitize=float-cast-overflow -fno-sanitize=null -fno-sanitize=alignment -fPIE
+                             -fno-omit-frame-pointer )
+    target_compile_options(${my_target} PUBLIC ${clang_san_options} )
+    target_link_options(${my_target} PUBLIC ${clang_san_options} )
+  endif()
 endfunction()
 
 # -------------------------------------------------------------

tests/phmap/hash_generator_testing.hpp

@@ -72,7 +72,7 @@ std::mt19937_64* GetSharedRng() {
     return rng;
 }
 
-enum Enum {
+enum Enum : uint64_t {
     kEnumEmpty,
     kEnumDeleted,
 };
@@ -99,24 +99,15 @@ template<>
 struct Generator<Enum> {
     Enum operator()() const {
         std::uniform_int_distribution<typename std::underlying_type<Enum>::type> dist;
-
-        while (true) {
-            auto variate = dist(*GetSharedRng());
-            if (variate != kEnumEmpty && variate != kEnumDeleted)
-                return static_cast<Enum>(variate);
-        }
+        return static_cast<Enum>(dist(*GetSharedRng()));
     }
 };
 
 template<>
 struct Generator<EnumClass> {
     EnumClass operator()() const {
         std::uniform_int_distribution<typename std::underlying_type<EnumClass>::type> dist;
-        while (true) {
-            EnumClass variate = static_cast<EnumClass>(dist(*GetSharedRng()));
-            if (variate != EnumClass::kEmpty && variate != EnumClass::kDeleted)
-                return static_cast<EnumClass>(variate);
-        }
+        return static_cast<EnumClass>(dist(*GetSharedRng()));
     }
 };