On Wednesday 23 February 2011, Brad King wrote:
> On 02/23/2011 03:36 PM, Alexander Neundorf wrote:
> > This is now on stage in the "ReworkedAsmSupport" branch.
> > I'd like to merge this into next, if there are no objections.
> > It implements what we discussed here, i.e. if there is already a C/CXX
> > compiler it tries to use that too for assembler, if not, it tries to find
> > such a compiler.
>
> Looks good! BTW, this block:
>
> +if(CMAKE_C_COMPILER_ID STREQUAL "GNU"
> + OR CMAKE_C_COMPILER_ID STREQUAL "HP"
> + OR CMAKE_C_COMPILER_ID STREQUAL "Intel"
> + OR CMAKE_C_COMPILER_ID STREQUAL "SunPro"
> + OR CMAKE_C_COMPILER_ID STREQUAL "XL")
> ...
> +endif(CMAKE_C_COMPILER_ID STREQUAL "GNU"
> + OR CMAKE_C_COMPILER_ID STREQUAL "HP"
> + OR CMAKE_C_COMPILER_ID STREQUAL "Intel"
> + OR CMAKE_C_COMPILER_ID STREQUAL "SunPro"
> + OR CMAKE_C_COMPILER_ID STREQUAL "XL")
>
> is more easily written
>
> if("${CMAKE_C_COMPILER_ID}" MATCHES "^(GNU|HP|Intel|SunPro|XL)$")
> ...
> endif()
Ok.
I merged this now into next on stage.
I also removed the "Assembler support is still experimental" message now.
Alex
_______________________________________________
cmake-developers mailing list
cmake-developers@cmake.org
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
