[llvm-commits] [llvm] r156392 - /llvm/trunk/cmake/modules/HandleLLVMOptions.cmake

[Rafael Espindola]

Author: rafael
Date: Tue May  8 13:11:06 2012
New Revision: 156392

URL: http://llvm.org/viewvc/llvm-project?rev=156392&view=rev
Log:
Adding -fcolor-diagnostics is causing problem to users that parse the build
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.

Modified:
    llvm/trunk/cmake/modules/HandleLLVMOptions.cmake

Modified: llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/HandleLLVMOptions.cmake?rev=156392&r1=156391&r2=156392&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/HandleLLVMOptions.cmake (original)
+++ llvm/trunk/cmake/modules/HandleLLVMOptions.cmake Tue May  8 13:11:06 2012
@@ -190,10 +190,6 @@
     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 )