On 10/26/2015 12:38 PM, Andersson, Joakim wrote:
> Attached is a patch that adds support for the ARMCC toolchain.

Thanks.  It looks good except that I'd like to discuss of compiler id.
The name "ARMCC" appears to refer to the "armcc" command-line tool.
Our convention for the compiler id is to refer to the vendor or maintaining
organization behind the compiler.  Help/variable/CMAKE_LANG_COMPILER_ID.rst
lists the associated vendors and should be updated for this too.

Thanks,
-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to