On 11/16/2015 12:21 PM, Levi Morrison wrote: > I found and fixed a mistake in my macro.
Great. Did you mean to attach the corrected version? > I have now tested this on Intel 12, 13, 14, and 15 for both C++11 and C++14. How did you test these? For full testing we need the CMake test suite to pass with the Intel compilers. In particular, tests with 'CompileFeatures' in their name cover this functionality. > What do I do now to get it reviewed and committed? Posting here is sufficient. This thread is the review ;) 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
