build: Replace robin-hood-hashing with unordered_dense

robin-hood-hashing is no longer developed. unordered_dense is its
replacement, by the same author.

Author: jeremyg-lunarg

.github/workflows/vvl.yml

@@ -76,18 +76,18 @@ jobs:
   linux:
     needs: check_vvl
     runs-on: ubuntu-22.04
-    name: "linux (${{matrix.sanitize}} sanitizer, ${{matrix.config}}, robinhood ${{matrix.robin_hood}} )"
+    name: "linux (${{matrix.sanitize}} sanitizer, ${{matrix.config}}, unordered_dense ${{matrix.unordered_dense}} )"
     strategy:
       fail-fast: false
       matrix:
         sanitize: [ address, thread ]
         config: [debug, release]
-        robin_hood: [ "ON" ]
+        unordered_dense: [ "ON" ]
         include:
-          # Test with Robin Hood disabled
+          # Test with unordered_dense disabled
           # Chromium build, and some package managers don't use it.
           - config: release
-            robin_hood: "OFF"
+            unordered_dense: "OFF"
             sanitize: address
         exclude:
           # Have found over time this finds nothing extra, while taking the longest and using the most CI minutes.
@@ -99,12 +99,12 @@ jobs:
       - uses: lukka/get-cmake@latest
       - uses: hendrikmuhs/[email protected]
         with:
-          key: ${{ matrix.config }}-${{ matrix.sanitize }}-${{matrix.robin_hood}}
+          key: ${{ matrix.config }}-${{ matrix.sanitize }}-${{matrix.unordered_dense}}
       - run: sudo apt-get -qq update && sudo apt-get install -y libwayland-dev xorg-dev
       # This is to combat a bug when using 6.6 linux kernels with thread/address sanitizer
       # https://github.com/google/sanitizers/issues/1716
       - run: sudo sysctl vm.mmap_rnd_bits=28
-      - run: python scripts/tests.py --build --config ${{ matrix.config }} --cmake='-DUSE_ROBIN_HOOD_HASHING=${{matrix.robin_hood}}'
+      - run: python scripts/tests.py --build --config ${{ matrix.config }} --cmake='-DUSE_CUSTOM_HASH_MAP=${{matrix.unordered_dense}}'
         env:
           CFLAGS: -fsanitize=${{ matrix.sanitize }}
           CXXFLAGS: -fsanitize=${{ matrix.sanitize }}
@@ -208,7 +208,7 @@ jobs:
             build-ci/external/glslang/build/install
             build-ci/external/googltest/build/install
             build-ci/external/mimalloc/build/install
-            build-ci/external/robin-hood-hashing/build/install
+            build-ci/external/unordered_dense/build/install
             build-ci/external/SPIRV-Headers/build/install
             build-ci/external/SPIRV-Tools/build/install
             build-ci/external/Vulkan-Headers/build/install

layers/CMakeLists.txt

@@ -79,11 +79,15 @@ target_link_libraries(VkLayer_utils PUBLIC
 target_include_directories(VkLayer_utils SYSTEM PRIVATE external)
 target_include_directories(VkLayer_utils PUBLIC . ${API_TYPE})
 
-find_package(robin_hood CONFIG)
-option(USE_ROBIN_HOOD_HASHING "robin_hood provides faster versions of std::unordered_map and std::unordered_set" ${robin_hood_FOUND})
 if (USE_ROBIN_HOOD_HASHING)
-    target_link_libraries(VkLayer_utils PUBLIC robin_hood::robin_hood)
-    target_compile_definitions(VkLayer_utils PUBLIC USE_ROBIN_HOOD_HASHING)
+  message(WARNING "Option USE_ROBIN_HOOD_HASHING has been deprecated. Use USE_CUSTOM_HASH_MAP instead.")
+endif ()
+
+find_package(unordered_dense CONFIG)
+option(USE_CUSTOM_HASH_MAP "Use a custom hash map (unordered_dense). It provides faster versions of std::unordered_map and std::unordered_set" ${unordered_dense_FOUND})
+if (USE_CUSTOM_HASH_MAP)
+    target_link_libraries(VkLayer_utils PUBLIC unordered_dense::unordered_dense)
+    target_compile_definitions(VkLayer_utils PUBLIC USE_CUSTOM_HASH_MAP)
 endif()
 
 # Using mimalloc on non-Windows OSes currently results in unit test instability with some

layers/containers/custom_containers.h

@@ -30,8 +30,8 @@
 #include <optional>
 #include <utility>
 
-#ifdef USE_ROBIN_HOOD_HASHING
-#include "robin_hood.h"
+#ifdef USE_CUSTOM_HASH_MAP
+#include "ankerl/unordered_dense.h"
 #else
 #include <map>
 #include <set>
@@ -43,56 +43,23 @@
 // namespace aliases to allow map and set implementations to easily be swapped out
 namespace vvl {
 
-#ifdef USE_ROBIN_HOOD_HASHING
+#ifdef USE_CUSTOM_HASH_MAP
 template <typename T>
-using hash = robin_hood::hash<T>;
+struct hash {
+    std::size_t operator()(const T &s) const noexcept { return static_cast<std::size_t>(internal_hash{}(s)); }
 
-template <typename Key, typename Hash = robin_hood::hash<Key>, typename KeyEqual = std::equal_to<Key>>
-using unordered_set = robin_hood::unordered_set<Key, Hash, KeyEqual>;
-
-template <typename Key, typename T, typename Hash = robin_hood::hash<Key>, typename KeyEqual = std::equal_to<Key>>
-using unordered_map = robin_hood::unordered_map<Key, T, Hash, KeyEqual>;
-
-template <typename Key, typename T>
-using map_entry = robin_hood::pair<Key, T>;
-
-// robin_hood-compatible insert_iterator (std:: uses the wrong insert method)
-// NOTE: std::iterator was deprecated in C++17, and newer versions of libstdc++ appear to mark this as such.
-template <typename T>
-struct insert_iterator {
-    using iterator_category = std::output_iterator_tag;
-    using value_type = typename T::value_type;
-    using iterator = typename T::iterator;
-    using difference_type = void;
-    using pointer = void;
-    using reference = T &;
-
-    insert_iterator(reference t, iterator i) : container(&t), iter(i) {}
-
-    insert_iterator &operator=(const value_type &value) {
-        auto result = container->insert(value);
-        iter = result.first;
-        ++iter;
-        return *this;
-    }
-
-    insert_iterator &operator=(value_type &&value) {
-        auto result = container->insert(std::move(value));
-        iter = result.first;
-        ++iter;
-        return *this;
-    }
-
-    insert_iterator &operator*() { return *this; }
+  private:
+    using internal_hash = ankerl::unordered_dense::hash<T>;
+};
 
-    insert_iterator &operator++() { return *this; }
+template <typename Key, typename Hash = ankerl::unordered_dense::hash<Key>, typename KeyEqual = std::equal_to<Key>>
+using unordered_set = ankerl::unordered_dense::set<Key, Hash, KeyEqual>;
 
-    insert_iterator &operator++(int) { return *this; }
+template <typename Key, typename T, typename Hash = ankerl::unordered_dense::hash<Key>, typename KeyEqual = std::equal_to<Key>>
+using unordered_map = ankerl::unordered_dense::segmented_map<Key, T, Hash, KeyEqual>;
 
-  private:
-    T *container;
-    iterator iter;
-};
+template <typename Key, typename T>
+using map_entry = std::pair<Key, T>;
 #else
 template <typename T>
 using hash = std::hash<T>;
@@ -105,9 +72,6 @@ using unordered_map = std::unordered_map<Key, T, Hash, KeyEqual>;
 
 template <typename Key, typename T>
 using map_entry = std::pair<Key, T>;
-
-template <typename T>
-using insert_iterator = std::insert_iterator<T>;
 #endif
 
 #if 1

layers/core_checks/cc_image_layout.cpp

@@ -161,11 +161,8 @@ void CoreChecks::TransitionFinalSubpassLayouts(vvl::CommandBuffer &cb_state) {
 
 static GlobalImageLayoutRangeMap *GetLayoutRangeMap(GlobalImageLayoutMap &map, const vvl::Image &image_state) {
     // This approach allows for a single hash lookup or/create new
-    auto &layout_map = map[&image_state];
-    if (!layout_map) {
-        layout_map.emplace(image_state.subresource_encoder.SubresourceCount());
-    }
-    return &(*layout_map);
+    auto result = map.emplace(&image_state, image_state.subresource_encoder.SubresourceCount());
+    return &(result.first->second.value());
 }
 
 // Helper to update the Global or Overlay layout map

scripts/CMakeLists.txt

@@ -126,9 +126,9 @@ if (UPDATE_DEPS)
         include("${UPDATE_DEPS_DIR}/helper.cmake")
     endif()
 endif()
-if (ROBIN_HOOD_HASHING_INSTALL_DIR)
-    list(APPEND CMAKE_PREFIX_PATH ${ROBIN_HOOD_HASHING_INSTALL_DIR})
-    set(CMAKE_REQUIRE_FIND_PACKAGE_robin_hood TRUE PARENT_SCOPE)
+if (UNORDERED_DENSE_INSTALL_DIR)
+    list(APPEND CMAKE_PREFIX_PATH ${UNORDERED_DENSE_INSTALL_DIR})
+    set(CMAKE_REQUIRE_FIND_PACKAGE_unordered_dense TRUE PARENT_SCOPE)
 endif()
 if (GLSLANG_INSTALL_DIR)
     list(APPEND CMAKE_PREFIX_PATH ${GLSLANG_INSTALL_DIR})

scripts/gn/secondary/build_overrides/vulkan_validation_layers.gni

@@ -18,7 +18,7 @@ vulkan_utility_libraries_dir = "//external/Vulkan-Utility-Libraries"
 vvl_spirv_tools_dir = "//external/SPIRV-Tools"
 vvl_spirv_headers_dir = "//external/SPIRV-Headers"
 vvl_glslang_dir = "//external/glslang/"
-robin_hood_headers_dir = "//external/robin-hood-hashing/src/include"
+unordered_dense_headers_dir = "//external/unordered_dense/include"
 
 # Subdirectories for generated files
 vulkan_data_subdir = ""

scripts/known_good.json

@@ -46,15 +46,12 @@
             "commit": "02433568af277324e6d7fe4582b663f14165c563"
         },
         {
-            "name": "robin-hood-hashing",
-            "url": "https://github.com/martinus/robin-hood-hashing.git",
-            "sub_dir": "robin-hood-hashing",
-            "build_dir": "robin-hood-hashing/build",
-            "install_dir": "robin-hood-hashing/build/install",
-            "cmake_options": [
-                "-DRH_STANDALONE_PROJECT=OFF"
-            ],
-            "commit": "3.11.5"
+            "name": "unordered_dense",
+            "url": "https://github.com/martinus/unordered_dense.git",
+            "sub_dir": "unordered_dense",
+            "build_dir": "unordered_dense/build",
+            "install_dir": "unordered_dense/build/install",
+            "commit": "v4.0.4"
         },
         {
             "name": "mimalloc",
@@ -149,7 +146,7 @@
         "Vulkan-Utility-Libraries": "VULKAN_UTILITY_LIBRARIES_INSTALL_DIR",
         "SPIRV-Headers": "SPIRV_HEADERS_INSTALL_DIR",
         "SPIRV-Tools": "SPIRV_TOOLS_INSTALL_DIR",
-        "robin-hood-hashing": "ROBIN_HOOD_HASHING_INSTALL_DIR",
+        "unordered_dense": "UNORDERED_DENSE_INSTALL_DIR",
         "googletest": "GOOGLETEST_INSTALL_DIR",
         "mimalloc": "MIMALLOC_INSTALL_DIR"
     }