Moved to C++17

Author: JohanMabille

.github/workflows/cxx-versions.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        cxx-version: [14, 17, 20]
+        cxx-version: [17, 20]
     steps:
     - uses: actions/checkout@v6
     - name: Setup
@@ -28,7 +28,7 @@ jobs:
     runs-on: windows-2022
     strategy:
       matrix:
-        cxx-version: [14, 17, 20]
+        cxx-version: [17, 20]
     steps:
     - name: Setup compiler
       uses: ilammy/msvc-dev-cmd@v1

CMakeLists.txt

@@ -40,7 +40,7 @@ target_include_directories(xsimd INTERFACE
     $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
     $<INSTALL_INTERFACE:include>)
 
-target_compile_features(xsimd INTERFACE cxx_std_14)
+target_compile_features(xsimd INTERFACE cxx_std_17)
 
 # Only add xtl build option to the build tree, that is, if xsimd being locally
 # developed or is vendored.

README.md

@@ -43,7 +43,7 @@ The mathematical functions are a lightweight implementation of the algorithms or
 
 ## Requirements
 
-`xsimd` requires a C++14 compliant compiler. The following C++ compilers are supported:
+`xsimd` requires a C++17 compliant compiler. The following C++ compilers are supported:
 
 Compiler                | Version
 ------------------------|-------------------------------

benchmark/CMakeLists.txt

@@ -32,23 +32,6 @@ if (CMAKE_CXX_COMPILER_ID MATCHES "Clang" OR CMAKE_CXX_COMPILER_ID MATCHES "GNU"
     if(NOT CMAKE_CXX_FLAGS MATCHES "-march" AND NOT CMAKE_CXX_FLAGS MATCHES "-arch" AND NOT CMAKE_OSX_ARCHITECTURES)
         set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -march=native")
     endif()
-
-    if(NOT MSVC)
-        if (ENABLE_XTL_COMPLEX)
-            CHECK_CXX_COMPILER_FLAG("-std=c++17" HAS_CPP17_FLAG)
-            if (NOT HAS_CPP17_FLAG)
-                message(FATAL_ERROR "Unsupported compiler -- xsimd requires C++17 support when xtl complex support is enabled")
-            endif()
-            set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++17")
-        else()
-            CHECK_CXX_COMPILER_FLAG("-std=c++14" HAS_CPP14_FLAG)
-            if (NOT HAS_CPP14_FLAG)
-                message(FATAL_ERROR "Unsupported compiler -- xsimd requires C++14 support!")
-            else()
-                set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++14")
-            endif()
-        endif()
-    endif()
 endif()
 
 if(MSVC)

docs/source/index.rst

@@ -7,7 +7,7 @@
 .. image:: xsimd.svg
    :alt: xsimd
 
-C++14 wrappers for SIMD intrinsics.
+C++17 wrappers for SIMD intrinsics.
 
 Introduction
 ------------
@@ -69,7 +69,7 @@ The following SIMD instruction set extensions are supported:
 +--------------+---------------------------------------------------------+
 
 
-`xsimd` requires a C++14 compliant compiler. The following C++ compilers are supported:
+`xsimd` requires a C++17 compliant compiler. The following C++ compilers are supported:
 
 +-------------------------+-------------------------------+
 | Compiler                | Version                       |

examples/CMakeLists.txt

@@ -35,7 +35,7 @@ endif()
 
 add_executable(mandelbrot mandelbrot.cpp)
 target_link_libraries(mandelbrot PRIVATE xsimd)
-set_property(TARGET mandelbrot PROPERTY CXX_STANDARD 14)
+set_property(TARGET mandelbrot PROPERTY CXX_STANDARD 17)
 if(ENABLE_XTL_COMPLEX)
     target_link_libraries(mandelbrot PRIVATE xtl)
 endif()

test/CMakeLists.txt

@@ -59,23 +59,6 @@ if (CMAKE_CXX_COMPILER_ID MATCHES "Clang" OR CMAKE_CXX_COMPILER_ID MATCHES "GNU"
         set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wunused-parameter -Wextra -Wreorder")
         if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
           set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wshorten-64-to-32")
-      endif()
-        # Users may override the c++ standard:
-        if(NOT DEFINED CMAKE_CXX_STANDARD OR "${CMAKE_CXX_STANDARD}" STREQUAL "")
-            if (ENABLE_XTL_COMPLEX)
-                CHECK_CXX_COMPILER_FLAG("-std=c++17" HAS_CPP17_FLAG)
-                if (NOT HAS_CPP17_FLAG)
-                    message(FATAL_ERROR "Unsupported compiler -- xsimd requires C++17 support when xtl complex support is enabled")
-                endif()
-                set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++17")
-            else()
-                CHECK_CXX_COMPILER_FLAG("-std=c++14" HAS_CPP14_FLAG)
-                if (NOT HAS_CPP14_FLAG)
-                    message(FATAL_ERROR "Unsupported compiler -- xsimd requires C++14 support!")
-                else()
-                    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++14")
-                endif()
-            endif()
         endif()
 
         if (NOT CROSS_COMPILE_ARM)

test/architectures/CMakeLists.txt

@@ -18,7 +18,7 @@ foreach(INTEL_PROCESSOR ${INTEL_PROCESSORS})
         target_link_libraries(test_${INTEL_PROCESSOR} PRIVATE xsimd)
         add_dependencies(xtest test_${INTEL_PROCESSOR})
         if(ENABLE_XTL_COMPLEX)
-            target_compile_features(test_${INTEL_PROCESSOR} PRIVATE cxx_std_14)
+            target_compile_features(test_${INTEL_PROCESSOR} PRIVATE cxx_std_17)
             target_compile_definitions(test_${INTEL_PROCESSOR} PRIVATE XSIMD_ENABLE_XTL_COMPLEX=1)
             target_link_libraries(test_${INTEL_PROCESSOR} PRIVATE xtl)
         endif()

test/check_inline_specifier.sh

@@ -20,7 +20,7 @@ sed -r -n '/^####/,$ p' < $0 > $query_file
 
 log_file=`mktemp -t`
 
-clang-query --extra-arg "-std=c++14" --extra-arg="-I$top_srcdir/include" -f $query_file $top_srcdir/include/xsimd/xsimd.hpp -- | tee $log_file
+clang-query --extra-arg "-std=c++17" --extra-arg="-I$top_srcdir/include" -f $query_file $top_srcdir/include/xsimd/xsimd.hpp -- | tee $log_file
 
 { grep -E '^0 matches.' $log_file && failed=0 ; } || failed=1
 

test/test_wasm/test_wasm.sh

@@ -32,7 +32,7 @@ cd build
 emcmake cmake \
     -DBUILD_TESTS=ON \
     -DCMAKE_BUILD_TYPE=Release \
-    -DCMAKE_CXX_STANDARD=14 \
+    -DCMAKE_CXX_STANDARD=17 \
     -DDOWNLOAD_DOCTEST=ON \
     $SRC_DIR