mpusz/mp-units
1
0
Fork 1
mirror of https://github.com/mpusz/mp-units.git synced 2025-08-02 03:44:27 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: setting MP_UNITS_EXPLICIT_THIS_PARAMETER_SUPPORTED fixed to account for C++ version

Browse Source 
Resolves #685
This commit is contained in:
Mateusz Pusz
2025-03-09 16:39:17 +01:00
parent 8454949adb
commit 6b80ef6302
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/CMakeLists.txt
View File

@@ -66,7 +66,9 @@ endif()
# clang++-18 supports explicit `this` parameter # clang++-18 supports explicit `this` parameter
# https://github.com/llvm/llvm-project/issues/82780 # https://github.com/llvm/llvm-project/issues/82780
if(NOT ${projectPrefix}EXPLICIT_THIS_PARAMETER_SUPPORTED AND CMAKE_CXX_COMPILER_ID STREQUAL "Clang" if(NOT ${projectPrefix}EXPLICIT_THIS_PARAMETER_SUPPORTED
AND CMAKE_CXX_STANDARD GREATER_EQUAL 23
AND CMAKE_CXX_COMPILER_ID STREQUAL "Clang"
AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL "18" AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL "18"
) )
message(STATUS "Clang 18+ detected, overriding `no CRTP` support") message(STATUS "Clang 18+ detected, overriding `no CRTP` support")