still compile with avx2

Author: TheNumbat

README.md

@@ -60,10 +60,10 @@ For faster parallel builds, you can instead generate [ninja](https://ninja-build
 
 Only the following configurations are supported:
 
-| OS      | Compiler    | Arch   |
-|---------|-------------|--------|
-| Windows | MSVC 19.37+ | x86_64 |
-| Linux   | Clang 17+   | x86_64 |
+| OS      | Compiler    | Arch |
+|---------|-------------|------|
+| Windows | MSVC 19.37+ | AVX2 |
+| Linux   | Clang 17+   | AVX2 |
 
 Other configurations (macOS, aarch64, GCC, etc.) may be added in the future.
 

rpp/CMakeLists.txt

@@ -87,13 +87,13 @@ if(MSVC)
     string(REPLACE "/GR" "" CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS})
 
     target_compile_definitions(rpp PRIVATE WIN32_LEAN_AND_MEAN NOMINMAX _HAS_EXCEPTIONS=0)
-    target_compile_options(rpp PRIVATE /MP /W4 /GR- /GS- /EHa- /wd4201)
+    target_compile_options(rpp PRIVATE /arch:AVX2 /MP /W4 /GR- /GS- /EHa- /wd4201)
 elseif(CLANG)
     if(CLANG_VERSION_MAJOR LESS 17)
         message(FATAL_ERROR "Unsupported Clang version: only 17+ is supported.")
     endif()
 
-    target_compile_options(rpp PRIVATE -Wall -Wextra -fno-exceptions -fno-rtti)
+    target_compile_options(rpp PRIVATE -mavx2 -Wall -Wextra -fno-exceptions -fno-rtti)
 else()
     message(FATAL_ERROR "Unsupported compiler: only MSVC and Clang are supported.")
 endif()

rpp/impl/simd.cpp

@@ -3,6 +3,10 @@
 
 #include <immintrin.h>
 
+#ifndef __AVX2__
+#error "Unsupported architecture: AVX2 is required".
+#endif
+
 namespace rpp::SIMD {
 
 static_assert(sizeof(F32x4) == 16);

rpp/simd.h

@@ -3,11 +3,11 @@
 
 #include "base.h"
 
+// The implementation of these functions are compiled assuming AVX2 support.
+
 namespace rpp::SIMD {
 
 struct F32x4 {
-    // Assumes SSE4.2 is supported.
-
     alignas(16) f32 data[4];
 
     [[nodiscard]] static F32x4 set1(f32 v) noexcept;
@@ -29,8 +29,6 @@ struct F32x4 {
 };
 
 struct F32x8 {
-    // Assumes AVX2 is supported.
-
     alignas(32) f32 data[8];
 
     [[nodiscard]] static F32x8 set1(f32 v) noexcept;