[cmake-developers] Add CXX_STANDARD support for Intel C++ compilers

Brad King brad.king at kitware.com
Mon Nov 30 13:33:39 EST 2015


On 11/30/2015 01:18 PM, Levi Morrison wrote:
> My latest patch is attached. All tests seem to be working for me on
> the compiler versions I have access to. Can you check it for any
> glaring errors?

Good.  This hunk:

> +  if (NOT CMAKE_CXX_COMPILER_FORCED)
> +    if (NOT CMAKE_CXX_STANDARD_COMPUTED_DEFAULT)
> +      set(CMAKE_CXX_STANDARD_DEFAULT 98)
> +    else()
> +      set(CMAKE_CXX_STANDARD_DEFAULT ${CMAKE_CXX_STANDARD_COMPUTED_DEFAULT})
> +    endif()
> +  endif()

still needs to be revised to account for recent changes:

 Project: Guess default standard dialect if compiler was forced (#15852)
 https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=441dba80

If CMAKE_CXX_COMPILER_FORCED is set then we now fall back to a guess.
Otherwise it is an error for CMAKE_CXX_STANDARD_COMPUTED_DEFAULT to
not be set.

Thanks,
-Brad



More information about the cmake-developers mailing list