From 4ebf543b933d282f51b6f00d02db7f7b52d56575 Mon Sep 17 00:00:00 2001 From: "Arnaud A. de Grandmaison" Date: Fri, 14 Jun 2013 19:26:57 +0000 Subject: [PATCH 1/1] Cmake: add compiler option in a more idiomatic way git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@183995 91177308-0d34-0410-b5e6-96231b3b80d8 --- cmake/modules/HandleLLVMOptions.cmake | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/cmake/modules/HandleLLVMOptions.cmake b/cmake/modules/HandleLLVMOptions.cmake index 2939fa2afa2..b33632466c6 100644 --- a/cmake/modules/HandleLLVMOptions.cmake +++ b/cmake/modules/HandleLLVMOptions.cmake @@ -268,8 +268,6 @@ add_llvm_definitions( -D__STDC_FORMAT_MACROS ) add_llvm_definitions( -D__STDC_LIMIT_MACROS ) # clang doesn't print colored diagnostics when invoked from Ninja -if (UNIX AND - CMAKE_CXX_COMPILER_ID STREQUAL "Clang" AND - CMAKE_GENERATOR STREQUAL "Ninja") - append("-fcolor-diagnostics" CMAKE_C_FLAGS CMAKE_CXX_FLAGS) +if (UNIX AND CMAKE_GENERATOR STREQUAL "Ninja") + add_flag_if_supported("-fcolor-diagnostics") endif() -- 2.34.1