Browse Source

Merge topic 'iwyu-better-ci-logs'

bf3425c3d2 ci: add a comment to run IWYU in verbose mode
25f0b4f397 CMake: add an option to run IWYU in verbose mode

Acked-by: Kitware Robot <[email protected]>
Merge-request: !8130
Brad King 2 years ago
parent
commit
09cb6c48e2
3 changed files with 8 additions and 1 deletions
  1. 2 0
      .gitlab/ci/configure_debian10_iwyu.cmake
  2. 5 0
      CMakeLists.txt
  3. 1 1
      Utilities/IWYU/mapping.imp

+ 2 - 0
.gitlab/ci/configure_debian10_iwyu.cmake

@@ -1,4 +1,6 @@
 set(CMake_RUN_IWYU ON CACHE BOOL "")
+# Uncomment to diagnose IWYU problems as needed.
+#set(CMake_IWYU_VERBOSE ON CACHE BOOL "")
 set(IWYU_COMMAND "/usr/bin/include-what-you-use-6.0" CACHE FILEPATH "")
 
 include("${CMAKE_CURRENT_LIST_DIR}/configure_common.cmake")

+ 5 - 0
CMakeLists.txt

@@ -313,6 +313,11 @@ if(CMake_RUN_IWYU)
   endif()
   set(CMAKE_CXX_INCLUDE_WHAT_YOU_USE
     "${IWYU_COMMAND};-Xiwyu;--mapping_file=${CMake_SOURCE_DIR}/Utilities/IWYU/mapping.imp;-w")
+  option(CMake_IWYU_VERBOSE "Run include-what-you-use in verbose mode" OFF)
+  if (CMake_IWYU_VERBOSE)
+    list(APPEND CMAKE_CXX_INCLUDE_WHAT_YOU_USE
+      -Xiwyu -v7)
+  endif ()
   list(APPEND CMAKE_CXX_INCLUDE_WHAT_YOU_USE ${CMake_IWYU_OPTIONS})
 endif()
 

+ 1 - 1
Utilities/IWYU/mapping.imp

@@ -76,7 +76,7 @@
   # __decay_and_strip is used internally in the C++11 standard library.
   # IWYU does not classify it as internal and suggests to add <type_traits>.
   # To ignore it, we simply map it to a file that is included anyway.
-  # Use '-Xiwyu -v7' to see the fully qualified names that need this.
+  # Use 'CMake_IWYU_VERBOSE' to see the fully qualified names that need this.
   # TODO: Can this be simplified with an @-expression?
   #{ symbol: [ "@std::__decay_and_strip<.*>::__type", private, "\"cmConfigure.h\"", public ] },
   { symbol: [ "std::__decay_and_strip<bool>::__type", private, "\"cmConfigure.h\"", public ] },