c++ - 如何将 clang-tidy 集成到 CMake 和 GCC？

Question

我想将 clang-tidy 集成到使用自定义 GCC 工具链编译的基于 CMake 的 C 和 C++ 项目中。

我试过关注 this tutorial , 设置 CMAKE_CXX_CLANG_TIDY .我还尝试通过设置 CMAKE_EXPORT_COMPILE_COMMANDS 来生成编译数据库至 ON并指向 run-clang-tidy.py到它的目录。

在这两种情况下，我都遇到了(相同的)一些可能与 Clang 和 GCC 之间的差异有关的错误:

CMake 文件中启用的某些警告标志在 Clang 中不受支持，但在 GCC 中受支持(如 -Wlogical-op )。由于编译器是 GCC，文件正确构建，并且标志被写入编译数据库，但 clang-tidy提示它。

clang-tidy 提示一些定义和函数不可用，即使代码编译得很好。例如，android-cloexec-open check建议使用 O_CLOEXEC以提高安全性并强制关闭文件，但尝试使用此定义会导致未定义标识符错误(即使我们的 GCC 编译代码)。
作为一个没有找到的函数的例子，有 clock_gettime .

我们的代码使用 C11 标准和 C++14 标准编译，没有 GNU 扩展:

set(CMAKE_C_STANDARD 11)
set(CMAKE_CXX_STANDARD 14)

set(CMAKE_C_EXTENSIONS OFF)
set(CMAKE_CXX_EXTENSIONS OFF)

自定义工具链是一个在 Linux 上运行并编译到 FreeBSD 的交叉编译工具链。

有没有办法禁止 CMake 将某些标志传递给 clang-tidy？我使用 clang-tidy 错了吗？

我怀疑这个问题与禁用 GNU 扩展、使用交叉编译工具链以及一些在 Clang 中默认未定义但由 GCC 定义的功能测试宏有关(例如 _GNU_SOURCE/_POSIX_SOURCE )。如果是这种情况，我该如何检查？如果没有，我应该以不同的方式使用 clang-tidy 吗？

编辑

正如@pablo285 所问，这里有 2 个警告，我对单个文件发出警告，然后添加 --warnings-as-errors=* ，构建停止:

error: unknown warning option '-Wlogical-op' ; did you mean '-Wlong-long'? [clang-diagnostic-error]

<file path>: error: use of undeclared identifier 'O_CLOEXEC' [clang-diagnostic-error]
O_WRONLY | O_CLOEXEC
           ^

我决定编写一个 python 脚本来替换 clang-tidy，从 CMake 接收命令行并编辑它以修复各种错误。以下是我尝试过的对命令行的修改:

删除 none clang 编译标志

这有助于处理第一个警告之类的事情，因为现在我不会传递 clang 不知道的标志。似乎我无法配置 CMake 将不同的标志集传递给 GCC 和 clang-tidy，所以如果有人熟悉这个问题的一些解决方案，我会很高兴听到!

我更改了传递给 clang-tidy 的包含目录

正如帖子中提到的，我使用自定义工具链(交叉编译)。我用过 this post和Python提取标准包含目录列表，并将它们作为-isystem <dir>的列表添加到标志列表中。 .我还加了 -nostdinc这样 clang-tidy 就不会试图查看他自己的标题而不是我的

这有助于解决上述问题，因为现在有各种定义，例如 O_CLOEXEC在工具链的头文件中定义，但由于我的工具链基于 GCC，clang 无法解析 <type_traits>头文件包含对许多编译器内部函数的调用

我不确定在这种情况下最好的方法是什么

@shycha:谢谢你的提示，我会尝试禁用这个特定的检查，我会再次编辑这篇文章

Answer 1

好的，我想我有一个解决方案。几个晚上后，我能够让它工作。

一般来说，我是这样编译的

rm -rf build
mkdir build
cd build
cmake -C ../cmake-scripts/clang-tidy-all.cmake .. && make

哪里 cmake-scripts目录包含:

clang-tidy-all.cmake
toolchain_arm_clang.cmake

下面列出了两个重要文件。
但更重要的是，您需要如何编译它。

一、 toolchain_arm_clang.cmake直接引用自 clang-tidy-all.cmake通过 set(CMAKE_TOOLCHAIN_FILE ...) .但是，必须从构建目录的角度引用它，因此如果您使用多个级别的构建目录，例如: build/x86 , build/arm , build/darwin等，那么您必须相应地修改该路径。

二、目的 set(CONFIG_SCRIPT_PRELOADED ...)是确保配置脚本已预先加载，即 cmake -C ../cmake-scripts/clang-tidy-all.cmake ..通常，您会希望在 CMakeLists.txt 中的某处有这样的东西。文件:

message(STATUS "CONFIG_SCRIPT_PRELOADED: ${CONFIG_SCRIPT_PRELOADED}")
if(NOT CONFIG_SCRIPT_PRELOADED)
    message(FATAL_ERROR "Run cmake -C /path/to/cmake.script to preload a config script!")
endif()

三、还有 /lib/ld-musl-armhf.so.1硬编码在 set(CMAKE_LINKER_ARM_COMPAT_STATIC ...) ;在我使用的开发盒上，它指向 /lib/libc.so ，所以可以使用 /lib/libc.sh反而。我从来没有试过。

四、使用 set(CMAKE_C_LINK_EXECUTABLE ...)和 set(CMAKE_LINKER_ARM_COMPAT_STATIC ...)是因为 CMake 在检查编译器期间，即在运行 make 之前提示一些链接问题。 .

五、我只是编译 C++代码，所以如果你需要编译一些 C ，那么可能还需要正确配置 set(CMAKE_C_CREATE_SHARED_LIBRARY ...) ，但我不知道是否有这样的配置选项。

一般建议

不要立即集成它。首先使用一个库(最好是 C++ 一个)测试一些简单的 CMake 项目并使其工作，然后添加第二个库，但在 C 中。 ，再调整一下。并且只有在此之后将其合并到代码库中。

工具链

我使用自定义工具链 GCC 8.3.0和 musl C库，因此对于其他工具链，某些文件的位置可能不同。

自定义 CMake

一些变量，比如(已经提到) CONFIG_SCRIPT_PRELOADED , EXPORT_PACKAGE_TO_GLOBAL_REGISTRY , DO_NOT_BUILD_TESTS , 或 DO_NOT_BUILD_BENCHMARKS是 不是 通用 CMake 选项，即我只在 CMakeLists.txt 中使用它们，因此您可以放心地忽略它们。

在每个 *.cmake 末尾未设置的变量文件，例如， build_test , extra_clang_tidy_unchecks_for_tests_only , 不需要出现在项目的主目录 CMakeLists.txt .

铛

$ clang --version
clang version 10.0.0 (https://github.com/llvm/llvm-project.git 4650b2f36949407ef25686440e3d65ac47709deb)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/local/bin

文件
clang-tidy-all.cmake :

set(ALL_CXX_WARNING_FLAGS --all-warnings -Weverything -Wno-c++98-compat -Wno-c++98-c++11-compat -Wno-c++98-c++11-c++14-compat -Wno-padded -Wno-c++98-compat-pedantic)
set(CXX_COMPILE_OPTIONS "-std=c++17;-O3;${ALL_CXX_WARNING_FLAGS}" CACHE INTERNAL "description")


set(CMAKE_CROSSCOMPILING True)
set(CMAKE_TOOLCHAIN_FILE "../cmake-scripts/toolchain_arm_clang.cmake" CACHE FILEPATH "CMake toolchain file")

set(CONFIG_SCRIPT_PRELOADED true CACHE BOOL "Ensures that config script was preloaded")


set(build_test False)

if(build_test)
    message(STATUS "Using test mode clang-tidy checks!")
    set(extra_clang_tidy_unchecks_for_tests_only ",-google-readability-avoid-underscore-in-googletest-name,-cppcoreguidelines-avoid-magic-numbers,-cppcoreguidelines-special-member-functions")
endif()

set(CMAKE_CXX_CLANG_TIDY "clang-tidy;--enable-check-profile;--checks=-*,abseil-string-find-startswith,bugprone-*,cert-*,clang-analyzer-*,cppcoreguidelines-*,google-*,hicpp-*,llvm-*,misc-*,modernize-*,-modernize-use-trailing-return-type,performance-*,readability-*,-readability-static-definition-in-anonymous-namespace,-readability-simplify-boolean-expr,portability-*${extra_clang_tidy_unchecks_for_tests_only}" CACHE INTERNAL "clang-tidy")

message(STATUS "build_test: ${build_test}")
message(STATUS "extra_clang_tidy_unchecks_for_tests_only: ${extra_clang_tidy_unchecks_for_tests_only}")
message(STATUS "CMAKE_CXX_CLANG_TIDY: ${CMAKE_CXX_CLANG_TIDY}")

# We want to skip building tests when clang-tidy is run (it takes too much time and serves nothing)
if(DEFINED CMAKE_CXX_CLANG_TIDY AND NOT build_test)
    set(DO_NOT_BUILD_TESTS true CACHE BOOL "Turns OFF building tests")
    set(DO_NOT_BUILD_BENCHMARKS true CACHE BOOL "Turns OFF building benchmarks")
endif()



unset(build_test)
unset(extra_clang_tidy_unchecks_for_tests_only)
set(EXPORT_PACKAGE_TO_GLOBAL_REGISTRY "OFF" CACHE INTERNAL "We don't export clang-tidy-all version to global register")

toolchain_arm_clang.cmake :

set(CMAKE_SYSTEM_NAME Linux)
set(CMAKE_SYSTEM_VERSION 4.14.0)
set(CMAKE_SYSTEM_PROCESSOR arm)


set(gcc_version 8.3.0)
set(x_tools "/opt/zynq/xtl")

set(CMAKE_C_COMPILER "clang" CACHE INTERNAL STRING)
set(CMAKE_CXX_COMPILER "clang++" CACHE INTERNAL STRING)
set(CMAKE_RANLIB "llvm-ranlib" CACHE INTERNAL STRING)
set(CMAKE_AR "llvm-ar" CACHE INTERNAL STRING)
set(CMAKE_AS "llvm-as" CACHE INTERNAL STRING)
set(CMAKE_LINKER "ld.lld" CACHE INTERNAL STRING)

execute_process(
    COMMAND bash -c "dirname `whereis ${CMAKE_LINKER} | tr -s ' ' '\n' | grep ${CMAKE_LINKER}`"
    OUTPUT_VARIABLE cmake_linker_dir
)
string(REGEX REPLACE "\n$" "" cmake_linker_dir "${cmake_linker_dir}")
set(cmake_linker_with_dir "${cmake_linker_dir}/${CMAKE_LINKER}" CACHE INTERNAL STRING)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -iwithsysroot /include/c++/${gcc_version} -iwithsysroot /include/c++/${gcc_version}/arm-linux-musleabihf" CACHE INTERNAL STRING)


set(CMAKE_SYSROOT ${x_tools}/arm-linux-musleabihf)
set(CMAKE_FIND_ROOT_PATH ${x_tools}/arm-linux-musleabihf)
set(CMAKE_INSTALL_PREFIX ${x_tools}/arm-linux-musleabihf)

set(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER)
set(CMAKE_FIND_ROOT_PATH_MODE_LIBRARY BOTH)
set(CMAKE_FIND_ROOT_PATH_MODE_INCLUDE BOTH)
set(CMAKE_FIND_ROOT_PATH_MODE_PACKAGE NEVER)



set(triple arm-linux-musleabihf)
set(CMAKE_LIBRARY_ARCHITECTURE ${triple})
set(CMAKE_C_COMPILER_TARGET ${triple})
set(CMAKE_CXX_COMPILER_TARGET ${triple})


set(lib_path_arm ${x_tools}/arm-linux-musleabihf/lib)

## Bootstrap library stuff:
set(Scrt1_o ${lib_path_arm}/Scrt1.o)
set(crti_o ${lib_path_arm}/crti.o)
set(crtn_o ${lib_path_arm}/crtn.o)

set(lib_path_gcc ${x_tools}/lib/gcc/${triple}/${gcc_version})
set(crtbeginS_o ${lib_path_gcc}/crtbeginS.o)
set(crtendS_o ${lib_path_gcc}/crtendS.o)


# Clang as linker
# --no-pie disable position independent executable, which is required when building
# statically linked executables.
set(CMAKE_CXX_LINK_EXECUTABLE "clang++ --target=${triple} -Wl,--no-pie --sysroot=${CMAKE_SYSROOT} ${CMAKE_CXX_FLAGS} -fuse-ld=${cmake_linker_with_dir} <CMAKE_CXX_LINK_FLAGS> <LINK_FLAGS> <LINK_LIBRARIES> <OBJECTS> -o  <TARGET> ")
set(CMAKE_CXX_CREATE_SHARED_LIBRARY "clang++ -Wl, --target=${triple} --sysroot=${CMAKE_SYSROOT} ${CMAKE_CXX_FLAGS} -fuse-ld=${cmake_linker_with_dir} -shared <CMAKE_CXX_LINK_FLAGS> <LINK_FLAGS> <LINK_LIBRARIES> <OBJECTS> -o  <TARGET> ")
#
# Do not use CMAKE_CXX_CREATE_STATIC_LIBRARY -- it is created automatically
# by cmake using ar and ranlib
#
#set(CMAKE_CXX_CREATE_STATIC_LIBRARY "clang++ -Wl,--no-pie,--no-export-dynamic,-v -v --target=${triple} --sysroot=${CMAKE_SYSROOT} ${CMAKE_CXX_FLAGS} -fuse-ld=ld.lld <CMAKE_CXX_LINK_FLAGS> <LINK_FLAGS> <LINK_LIBRARIES> <OBJECTS> -o  <TARGET> ")


## Linker as linker
set(CMAKE_LINKER_ARM_COMPAT_STATIC "-pie -EL -z relro -X --hash-style=gnu --eh-frame-hdr -m armelf_linux_eabi -dynamic-linker /lib/ld-musl-armhf.so.1 ${Scrt1_o} ${crti_o} ${crtbeginS_o} -lstdc++ -lm -lgcc_s -lgcc -lc ${crtendS_o} ${crtn_o}")
set(CMAKE_C_LINK_EXECUTABLE "${CMAKE_LINKER} ${CMAKE_LINKER_ARM_COMPAT_STATIC} <CMAKE_C_LINK_FLAGS> <LINK_FLAGS> <LINK_LIBRARIES> <OBJECTS> -o  <TARGET>")


# Debian bug 708744(?)
#include_directories("${CMAKE_SYSROOT}/usr/include/")
#include_directories("${CMAKE_SYSROOT}/usr/include/c++/${gcc_version}")
#include_directories("${CMAKE_SYSROOT}/usr/include/c++/${gcc_version}/${triple}")

## Clang workarounds:
set(toolchain_lib_dir_0 "${CMAKE_SYSROOT}/lib")
set(toolchain_lib_dir_1 "${CMAKE_SYSROOT}/../lib")
set(toolchain_lib_dir_2 "${CMAKE_SYSROOT}/../lib/gcc/${triple}/${gcc_version}")
set(CMAKE_TOOLCHAIN_LINK_FLAGS "-L${toolchain_lib_dir_0} -L${toolchain_lib_dir_1} -L${toolchain_lib_dir_2}")

## CMake workarounds
set(CMAKE_EXE_LINKER_FLAGS ${CMAKE_TOOLCHAIN_LINK_FLAGS} CACHE INTERNAL "exe link flags")
set(CMAKE_MODULE_LINKER_FLAGS ${CMAKE_TOOLCHAIN_LINK_FLAGS} CACHE INTERNAL "module link flags")
set(CMAKE_SHARED_LINKER_FLAGS ${CMAKE_TOOLCHAIN_LINK_FLAGS} CACHE INTERNAL "shared link flags")



unset(cmake_linker_with_dir)
unset(cmake_linker_dir)