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()
-Brad
_______________________________________________
cmake-developers mailing list
[email protected]
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
