Move to new simd approach (#152)

which allows us to do arch64ec as neon
surge-synthesizer · Nov 1, 2024 · 94cc27d · 94cc27d
1 parent 0f7c329
commit 94cc27d
Show file tree

Hide file tree

Showing 16 changed files with 1,428 additions and 172 deletions.
diff --git a/.github/workflows/build-tests.yml b/.github/workflows/build-tests.yml
@@ -7,22 +7,48 @@ on:
 
 jobs:
   build_feature:
-    name: Build Tests
+    name: Test ${{ matrix.name }}
     runs-on: ${{ matrix.os }}
     strategy:
+      fail-fast: false
       matrix:
-        os: [ ubuntu-latest, macos-latest, windows-latest ]
         include:
           - os: ubuntu-latest
             name: linux
+            runTest: true
             testExe: build/sst-effects-test
+
           - os: macos-latest
-            name: mac
+            name: mac-x86
+            runTest: true
             testExe: build/sst-effects-test
+            cmakeArgs: -DCMAKE_OSX_ARCHITECTURES=x86_64
+
+          - os: macos-latest
+            name: mac-arm
+            cmakeArgs: -DCMAKE_OSX_ARCHITECTURES=arm64
+
+          - os: macos-latest
+            name: mac-arm-nonative
+            cmakeArgs: -DCMAKE_OSX_ARCHITECTURES=arm64 -DSST_BASIC_BLOCKS_SIMD_OMIT_NATIVE_ALIASES=TRUE
+
           - os: windows-latest
-            name: win
+            name: win-x86
+            runTest: true
             testExe: build/Release/sst-effects-test.exe
 
+          - os: windows-latest
+            name: win-arm64
+            cmakeArgs: -G"Visual Studio 17 2022" -A arm64 -DCMAKE_SYSTEM_VERSION=10
+
+          - os: windows-latest
+            name: win-arm64ec
+            cmakeArgs: -G"Visual Studio 17 2022" -A arm64ec -DCMAKE_SYSTEM_VERSION=10
+
+          - os: windows-latest
+            name: win-arm64-non-native
+            cmakeArgs: -G"Visual Studio 17 2022" -A arm64 -DCMAKE_SYSTEM_VERSION=10 -DSST_BASIC_BLOCKS_SIMD_OMIT_NATIVE_ALIASES=TRUE
+
     steps:
 
       - name: Checkout code
@@ -32,10 +58,11 @@ jobs:
 
       - name: Build Smoke test
         run: |
-          cmake -S . -B ./build -DCMAKE_BUILD_TYPE=Release -DSST_EFFECTS_BUILD_TESTS=TRUE -DCMAKE_OSX_ARCHITECTURES="arm64;x86_64"
+          cmake -S . -B ./build -DCMAKE_BUILD_TYPE=Release -DSST_EFFECTS_BUILD_TESTS=TRUE ${{ matrix.cmakeArgs }}
           cmake --build ./build --config Release
 
       - name: Run Smoke Test
+        if: ${{ matrix.runTest }}
         run: |
           ls ${{ matrix.testExe }}
           ${{ matrix.testExe }}
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -18,6 +18,15 @@ if (${SST_EFFECTS_BUILD_TESTS})
     message(STATUS "Building tests")
     include(cmake/CPM.cmake)
 
+    if (NOT TARGET simde)
+        CPMAddPackage(NAME simde
+                GITHUB_REPOSITORY simd-everywhere/simde
+                VERSION 0.7.2
+        )
+        add_library(simde INTERFACE)
+        target_include_directories(simde INTERFACE ${simde_SOURCE_DIR})
+    endif ()
+
     if (NOT TARGET sst-basic-blocks)
         CPMAddPackage(NAME sst-basic-blocks
                 GITHUB_REPOSITORY surge-synthesizer/sst-basic-blocks
@@ -53,14 +62,6 @@ if (${SST_EFFECTS_BUILD_TESTS})
         )
     endif ()
 
-    if (NOT TARGET simde)
-        CPMAddPackage(NAME simde
-                GITHUB_REPOSITORY simd-everywhere/simde
-                VERSION 0.7.2
-        )
-        add_library(simde INTERFACE)
-        target_include_directories(simde INTERFACE ${simde_SOURCE_DIR})
-    endif ()
 
     add_executable(${PROJECT_NAME}-test
             tests/sst-effects-test.cpp