14

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

我试过按照本教程设置CMAKE_CXX_CLANG_TIDY。我还尝试通过设置并指向其目录CMAKE_EXPORT_COMPILE_COMMANDS来生成编译数据库。ONrun-clang-tidy.py

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

  1. CMake 文件中启用的一些警告标志在 Clang 中不受支持，但在 GCC 中受支持（如-Wlogical-op）。由于编译器是 GCC，文件构建正确，并且标志被写入编译数据库，但clang-tidy抱怨它。
  2. clang-tidy 抱怨一些定义和函数不可用，即使代码编译得很好。例如，建议使用android-cloexec-open 检查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 的交叉编译工具链。

  1. 有没有办法禁止 CMake 将某些标志传递给 clang-tidy？我使用 clang-tidy 错了吗？
  2. 我怀疑这个问题与禁用 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 接收命令行并对其进行编辑以修复各种错误。以下是我尝试对命令行的修改：

  1. 删除无 clang 编译标志
    • 这有助于解决第一个警告之类的问题，因为现在我不会传递 clang 不知道的标志。似乎我无法配置 CMake 将不同的标志集传递给 GCC 和 clang-tidy，所以如果有人熟悉这个问题的一些解决方案，我会很高兴听到！
  2. 我更改了传递给 clang-tidy 的包含目录
    • 如帖子中所述，我使用自定义工具链（交叉编译）。我使用这篇文章和 Python 来提取标准包含目录的列表，并将它们作为-isystem <dir>. 我还添加了-nostdinc以便 clang-tidy 不会尝试查看他自己的标题而不是我的
      • 这有助于解决上述问题，因为现在O_CLOEXEC在工具链的标头中定义了各种定义，但是由于我的工具链基于 GCC，clang 无法解析<type_traits>包含对许多编译器内在函数的调用的标头
    • 我不确定在这种情况下最好的方法是什么

@shycha：感谢您的提示，我将尝试禁用此特定检查，然后再次编辑此帖子

4

2 回答 2

7

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

一般来说，我这样编译

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.cmakevia引用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 ...)andset(CMAKE_LINKER_ARM_COMPAT_STATIC ...)是因为 CMake 在检查编译器期间（即在运行make.

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

一般建议

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

工具链

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

自定义 CMake

一些变量，如（已经提到的）、、、CONFIG_SCRIPT_PRELOADED或不是通用的CMake 选项，即我只在 my 中使用它们，所以你可以放心地忽略它们。EXPORT_PACKAGE_TO_GLOBAL_REGISTRYDO_NOT_BUILD_TESTSDO_NOT_BUILD_BENCHMARKSCMakeLists.txt

*.cmake在每个文件末尾未设置的变量，例如 , build_test，extra_clang_tidy_unchecks_for_tests_only不需要出现在项目的 mainCMakeLists.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)
于 2020-01-23T16:43:42.577 回答
0

也许不完全是您正在寻找的，但我在 CMakeLists.txt 中使用它：

set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
add_custom_target(lint
    COMMAND sh -c "run-clang-tidy -header-filter=.* -checks=`tr '\\n' , <${CMAKE_SOURCE_DIR}/checks.txt` >lint.out 2>lint.err"
    COMMAND sh -c "grep warning: lint.out || true"
    COMMAND ls -lh ${CMAKE_BINARY_DIR}/lint.out
    VERBATIM
    WORKING_DIRECTORY ${CMAKE_BINARY_DIR}
)

make lint这会为 clang-tidy 检查创建一个单独的构建目标 ( )。clang-tidy 我的项目需要很长时间，所以我不想在每次构建时都运行它；make lint如果需要，可以手动运行，并且在每次推送到 repo 后也会在 CI 作业中执行（以使 CI 管道失败、阻止合并的方式，如果有任何发现）。

的输出make lint是包含尽可能少的上下文的整洁发现列表。完整的输出（包括结果的上下文）位于 中lint.out，错误消息位于 中lint.err，我将它们保存为 CI 人工制品。

checks.txt是项目根目录中的一个文本文件，它定义了要激活的 clang-tidy 检查，如下所示：

*
-altera-id-dependent-backward-branch
-altera-struct-pack-align
-altera-unroll-loops
-android-*

第一行启用所有可用的检查，其他行禁用我不想要的检查。

当然只能在类 Unix 系统中工作。

于 2021-12-07T15:18:54.750 回答