[PATCH] Move duplicate Windows-specific compiler flags to a	common CMake variable
    Reid Kleckner 
    rnk at google.com
       
    Thu May 29 10:03:48 PDT 2014
    
    
  
drive by nits that were already there
================
Comment at: cmake/Modules/AddCompilerRT.cmake:129
@@ -126,1 +128,3 @@
 if(MSVC)
+  # MSVC system headers and gtest use a lot of deprecated stuff.
+  list(APPEND COMPILER_RT_TEST_CFLAGS
----------------
Can we use -isystem to suppress these?  Does -D_CRT_SECURE_NO_WARNINGS solve this problem?
================
Comment at: cmake/Modules/AddCompilerRT.cmake:137
@@ +136,3 @@
+
+  # We should teach clang to understand more pragmas.
+  list(APPEND COMPILER_RT_TEST_CFLAGS
----------------
ditto, this can be handled with -isystem.
http://reviews.llvm.org/D3952
    
    
More information about the llvm-commits
mailing list