Fix clang-15 build and WAVM

.github/workflows/test.yml

@@ -86,8 +86,8 @@ jobs:
         options:
           - name: "Self-hosted: Linux: gcc-12 ASAN"
             run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/gcc-12_cxx20.cmake -DASAN=ON
-          - name: "Self-hosted: Linux: clang-15 TSAN"
-            run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/clang-15_cxx20.cmake -DTSAN=ON
+          - name: "Self-hosted: Linux: clang-15 TSAN WAVM"
+            run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/clang-15_cxx20.cmake -DTSAN=ON -DWASM_COMPILER=WAVM
           - name: "Self-hosted: Linux: clang-15 UBSAN"
             run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/clang-15_cxx20.cmake -DUBSAN=ON
           - name: "Self-hosted: Linux: clang-15 External Project"
@@ -124,8 +124,8 @@ jobs:
         options:
           - name: "Self-hosted: Linux: gcc-12 ASAN"
             run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/gcc-12_cxx20.cmake -DASAN=ON
-          - name: "Self-hosted: Linux: clang-15 TSAN"
-            run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/clang-15_cxx20.cmake -DTSAN=ON
+          - name: "Self-hosted: Linux: clang-15 TSAN WAVM"
+            run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/clang-15_cxx20.cmake -DTSAN=ON -DWASM_COMPILER=WAVM
           - name: "Self-hosted: Linux: clang-15 UBSAN"
             run: ./housekeeping/make_build.sh -DCLEAR_OBJS=ON -DCMAKE_TOOLCHAIN_FILE=cmake/toolchain/clang-15_cxx20.cmake -DUBSAN=ON
           - name: "Self-hosted: Linux: clang-15 External Project"

core/parachain/validator/fragment_tree.hpp

@@ -523,8 +523,14 @@ namespace kagome::parachain::fragment {
     }
 
     std::vector<CandidateHash> getCandidates() const {
-      auto kv = std::views::keys(candidates);
-      return {kv.begin(), kv.end()};
+      // doesn't compile in clang-15 due to a bug
+      // std::views::keys(candidates)
+      std::vector<CandidateHash> res;
+      res.reserve(candidates.size());
+      for (auto &pair : candidates) {
+        res.push_back(pair.first);
+      }
+      return res;
     }
 
     template <typename Func>
@@ -659,7 +665,13 @@ namespace kagome::parachain::fragment {
               hasher_->blake2b_256(child_constraints.required_parent);
 
           storage.iterParaChildren(
-              required_head_hash, [&](const CandidateEntry &candidate) {
+              required_head_hash,
+              // clang-15 doesn't like capturing structured bindings into
+              // lambdas
+              [&,
+               earliest_rp = earliest_rp,
+               child_depth = child_depth,
+               modifications = modifications](const CandidateEntry &candidate) {
                 auto pending =
                     scope.getPendingAvailability(candidate.candidate_hash);
                 Option<RelayChainBlockInfo> relay_parent_opt;

core/runtime/wavm/intrinsics/intrinsic_functions.cpp

@@ -51,6 +51,16 @@ namespace kagome::runtime::wavm {
     return WAVM::IR::ValueType::i64;
   }
 
+  template <>
+  WAVM::IR::ValueType get_wavm_type<uint32_t>() {
+    return WAVM::IR::ValueType::i32;
+  }
+
+  template <>
+  WAVM::IR::ValueType get_wavm_type<uint64_t>() {
+    return WAVM::IR::ValueType::i64;
+  }
+
   template <auto Method, typename... Args>
   auto host_method_thunk(WAVM::Runtime::ContextRuntimeData *, Args... args) {
     return std::invoke(Method, peekHostApi(), args...);
@@ -77,10 +87,9 @@ namespace kagome::runtime::wavm {
     }
 #define REGISTER_HOST_METHOD(Ret, name, ...)                                \
   registerMethod<&host_api::HostApi::name, Ret __VA_OPT__(, ) __VA_ARGS__>( \
-      module, #name);                                                       \
+      module, #name);
 
     REGISTER_HOST_METHODS
-
   }
 
 }  // namespace kagome::runtime::wavm

test/core/parachain/prospective_parachains.cpp

@@ -355,7 +355,7 @@ class ProspectiveParachainsTest : public testing::Test {
     EXPECT_CALL(*block_tree_, getBlockHeader(hash))
         .WillRepeatedly(Return(header));
 
-    BlockNumber min_min = [&]() -> BlockNumber {
+    BlockNumber min_min = [&, number = number]() -> BlockNumber {
       std::optional<BlockNumber> min_min;
       for (const auto &[_, data] : leaf.para_data) {
         min_min = min_min ? std::min(*min_min, data.min_relay_parent)
@@ -438,7 +438,7 @@ class ProspectiveParachainsTest : public testing::Test {
       }
     }
 
-    prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+    (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
         .new_head = {update.new_head},
         .lost = update.lost,
     });
@@ -568,7 +568,7 @@ class ProspectiveParachainsTest : public testing::Test {
         .new_head = {},
         .lost = {hash},
     };
-    prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+    (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
         .new_head = {},
         .lost = update.lost,
     });
@@ -608,7 +608,7 @@ TEST_F(ProspectiveParachainsTest, shouldDoNoWorkIfAsyncBackingDisabledForLeaf) {
       .WillRepeatedly(
           Return(outcome::failure(ParachainProcessorImpl::Error::NO_STATE)));
 
-  prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+  (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
       .new_head = {update.new_head},
       .lost = update.lost,
   });
@@ -1348,7 +1348,7 @@ TEST_F(ProspectiveParachainsTest, FragmentTree_usesAncestryOnlyWithinSession) {
     }
   }
 
-  prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+  (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
       .new_head = {update.new_head},
       .lost = update.lost,
   });
@@ -1393,7 +1393,7 @@ TEST_F(ProspectiveParachainsTest, FragmentTree_correctlyUpdatesLeaves) {
   activate_leaf(leaf_b, test_state, async_backing_params);
   activate_leaf(leaf_b, test_state, async_backing_params);
 
-  prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+  (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
       .new_head = {},
       .lost = {},
   });
@@ -1429,7 +1429,7 @@ TEST_F(ProspectiveParachainsTest, FragmentTree_correctlyUpdatesLeaves) {
     };
     // handle_leaf_activation_2(update2, leaf_c, test_state,
     // async_backing_params);
-    prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+    (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
         .new_head = {},
         .lost = update2.lost,
     });
@@ -1465,7 +1465,7 @@ TEST_F(ProspectiveParachainsTest, FragmentTree_correctlyUpdatesLeaves) {
         .new_head = {},
         .lost = {leaf_a.hash, leaf_b.hash, leaf_c.hash},
     };
-    prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
+    (void)prospective_parachain_->onActiveLeavesUpdate(network::ExViewRef{
         .new_head = {},
         .lost = update2.lost,
     });

test/core/runtime/wavm/wavm_runtime_test.hpp

@@ -39,7 +39,7 @@ class WavmRuntimeTest : public RuntimeTestBase {
     std::shared_ptr<kagome::runtime::wavm::IntrinsicModuleInstance>
         intrinsic_module_instance = intrinsic_module->instantiate();
     resolver_ = std::make_shared<kagome::runtime::wavm::IntrinsicResolverImpl>(
-        intrinsic_module_instance);
+        compartment, intrinsic_module_instance);
 
     auto module_factory =
         std::make_shared<kagome::runtime::wavm::ModuleFactoryImpl>(
@@ -53,13 +53,9 @@ class WavmRuntimeTest : public RuntimeTestBase {
             std::nullopt,
             hasher_);
 
-
     auto instance_env_factory =
         std::make_shared<kagome::runtime::wavm::InstanceEnvironmentFactory>(
-            trie_storage_,
-            serializer_,
-            host_api_factory_,
-            module_factory);
+            trie_storage_, serializer_, host_api_factory_, module_factory);
 
     return module_factory;
   }