Add ARM validation to CI (#109)

This PR adds an Ubuntu ARM validation in github workflow. Some minor
changes in the codebase are also made for compatibility with ARM.

---------

Co-authored-by: Dmitry Razdoburdin <>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: Nikolay Petrov <[email protected]>

Author: 3 people

.github/workflows/build-linux-arm.yml

@@ -0,0 +1,72 @@
+# Copyright 2025 Intel Corporation
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+name: Linux Build and Test on ARM
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+permissions:
+  contents: read
+
+# This allows a subsequently queued workflow run to interrupt previous runs
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
+jobs:
+  build:
+    name: ${{ matrix.cxx }}, ${{ matrix.build_type }}
+    runs-on: ubuntu-22.04-arm
+    strategy:
+      matrix:
+        build_type: [RelWithDebugInfo]
+        cxx: [g++-11, g++-12, clang++-15]
+        include:
+          - cxx: g++-11
+            cc: gcc-11
+          - cxx: g++-12
+            cc: gcc-12
+          - cxx: clang++-15
+            cc: clang-15
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Configure build
+      working-directory: ${{ runner.temp }}
+      env:
+        CXX: ${{ matrix.cxx }}
+        CC: ${{ matrix.cc }}
+        TEMP_WORKSPACE: ${{ runner.temp }}
+      run: |
+        cmake -B${TEMP_WORKSPACE}/build -S${GITHUB_WORKSPACE} \
+              -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} \
+              -DSVS_BUILD_BINARIES=YES \
+              -DSVS_BUILD_TESTS=YES \
+              -DSVS_BUILD_EXAMPLES=YES \
+              -DSVS_EXPERIMENTAL_LEANVEC=YES
+
+    - name: Build Tests and Utilities
+      working-directory: ${{ runner.temp }}/build
+      run: make -j$(nproc)
+
+    - name: Run tests
+      env:
+          CTEST_OUTPUT_ON_FAILURE: 1
+      working-directory: ${{ runner.temp }}/build/tests
+      run: ctest -C ${{ matrix.build_type }}

include/svs/core/distance/cosine.h

@@ -29,7 +29,6 @@
 #include <iostream>
 #include <memory>
 #include <type_traits>
-#include <x86intrin.h>
 
 namespace svs::distance {
 // Forward declare implementation to allow entry point to be near the top.

include/svs/core/distance/euclidean.h

@@ -31,7 +31,6 @@
 #include <memory>
 #include <span>
 #include <type_traits>
-#include <x86intrin.h>
 
 // Implementation Notes regarding Intel(R) AVX Extentions
 // Top most entry in the bulleted list underneath each type pair <T,U> is the preferred

include/svs/core/distance/inner_product.h

@@ -29,7 +29,6 @@
 #include <iostream>
 #include <memory>
 #include <type_traits>
-#include <x86intrin.h>
 
 namespace svs::distance {
 // Forward declare implementation to allow entry point to be near the top.

include/svs/core/distance/simd_utils.h

@@ -16,6 +16,8 @@
 
 #pragma once
 
+#if defined(__i386__) || defined(__x86_64__)
+
 #include <array>
 #include <limits>
 #include <type_traits>
@@ -369,3 +371,5 @@ template <> struct ConvertForVNNI<int16_t, 32> {
 
 } // namespace simd
 } // namespace svs
+
+#endif // __i386__ || __x86_64__

include/svs/lib/prefetch.h

@@ -33,7 +33,7 @@ template <typename T> void prefetch_l0(const T* ptr) {
 }
 #else
 // Do nothing if prefetch is not-available.
-template <typename T> void prefetch_l0(const T* ptr) {}
+template <typename T> void prefetch_l0([[maybe_unused]] const T* ptr) {}
 #endif
 
 const size_t CACHELINE_BYTES = 64;

include/svs/lib/spinlock.h

@@ -21,7 +21,13 @@
 namespace svs {
 
 namespace detail {
-inline void pause() { __builtin_ia32_pause(); }
+inline void pause() {
+  #if defined(__i386__) || defined(__x86_64__)
+    __builtin_ia32_pause();
+  #else //  __aarch64__
+    asm volatile ("yield" ::: "memory");
+  #endif
+}
 } // namespace detail
 
 ///

tests/svs/core/distances/simd_utils.cpp

@@ -14,6 +14,8 @@
  * limitations under the License.
  */
 
+#if defined(__i386__) || defined(__x86_64__)
+
 #include <cstdint>
 #include <type_traits>
 
@@ -61,3 +63,5 @@ CATCH_TEST_CASE("Masks", "[distance]") {
     CATCH_REQUIRE(svs::create_mask<16>(svs::lib::MaybeStatic<100>()) == 0xF);
     CATCH_REQUIRE(svs::create_mask<16>(svs::lib::MaybeStatic<16>()) == 0xFFFF);
 }
+
+#endif  // defined(__i386__) || defined(__x86_64__)