[Lldb-commits] [lldb] 755afc0 - [CMake] Re-enable -Wno-gnu-anonymous-struct & -Wno-nested-anon-types.

[Jonas Devlieghere via lldb-commits]

Author: Jonas Devlieghere
Date: 2019-11-18T16:58:40-08:00
New Revision: 755afc0af8f9a448befb0802a045746dc72ef759

URL: https://github.com/llvm/llvm-project/commit/755afc0af8f9a448befb0802a045746dc72ef759
DIFF: https://github.com/llvm/llvm-project/commit/755afc0af8f9a448befb0802a045746dc72ef759.diff

LOG: [CMake] Re-enable -Wno-gnu-anonymous-struct & -Wno-nested-anon-types.

We're checking for support but we're discarding the result. My best
guess is that these warnings were disabled in the past. However, I don't
see a reason to keep it that way.

Added: 
    

Modified: 
    lldb/cmake/modules/LLDBConfig.cmake

Removed: 
    


################################################################################
diff  --git a/lldb/cmake/modules/LLDBConfig.cmake b/lldb/cmake/modules/LLDBConfig.cmake
index 2a709f81981f..5e45eb559d2d 100644
--- a/lldb/cmake/modules/LLDBConfig.cmake
+++ b/lldb/cmake/modules/LLDBConfig.cmake
@@ -365,9 +365,15 @@ endif ()
 
 check_cxx_compiler_flag("-Wno-gnu-anonymous-struct"
                         CXX_SUPPORTS_NO_GNU_ANONYMOUS_STRUCT)
+if (CXX_SUPPORTS_NO_GNU_ANONYMOUS_STRUCT)
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-anonymous-struct")
+endif ()
 
 check_cxx_compiler_flag("-Wno-nested-anon-types"
                         CXX_SUPPORTS_NO_NESTED_ANON_TYPES)
+if (CXX_SUPPORTS_NO_NESTED_ANON_TYPES)
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-nested-anon-types")
+endif ()
 
 # Disable MSVC warnings
 if( MSVC )