Adding -fcolor-diagnostics is causing problem to users that parse the build
authorRafael Espindola <rafael.espindola@gmail.com>
Tue, 8 May 2012 18:11:06 +0000 (18:11 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Tue, 8 May 2012 18:11:06 +0000 (18:11 +0000)
output. Peter Collingbourne also reports that it is showing up in
$(llvm-config --cflags).

Revert this for now since I don't know enough cmake to fix it properly.

This reverts commit 18efed7adc79c1970f307bb5b015d199012ba872.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@156392 91177308-0d34-0410-b5e6-96231b3b80d8

cmake/modules/HandleLLVMOptions.cmake

index da31cc4c8dd069f30d3a76ee0b9124cddd31a1bc..d10e59a09729f3b88d368f43fa19e5b0f1c1a68e 100644 (file)
@@ -190,10 +190,6 @@ elseif( LLVM_COMPILER_IS_GCC_COMPATIBLE )
     if( SUPPORTS_COVERED_SWITCH_DEFAULT_FLAG )
       add_llvm_definitions( -Wcovered-switch-default )
     endif()
-    check_cxx_compiler_flag("-Werror -fcolor-diagnostics" SUPPORTS_FCOLOR_DIAGNOSTICS_FLAG)
-    if( SUPPORTS_FCOLOR_DIAGNOSTICS_FLAG )
-      add_llvm_definitions( -fcolor-diagnostics )
-    endif()
   endif (LLVM_ENABLE_WARNINGS)
   if (LLVM_ENABLE_WERROR)
     add_llvm_definitions( -Werror )