clang-tidy in Chromium is very experimental, and is somewhat painful to use. We are exploring making it easier and integrating with existing tools, but aren‘t there yet. If you don’t want to wait and enjoy tinkering, forge ahead. Otherwise, feel free to turn back now.
clang-tidy is a clang-based C++ “linter” tool. Its purpose is to provide an extensible framework for diagnosing and fixing typical programming errors, like style violations, interface misuse, or bugs that can be deduced via static analysis.
In addition to a full Chromium checkout, you need the clang-tidy binary. We recommend checking llvm‘s clang source and building the clang-tidy binary directly. Instructions for getting started with clang are available from llvm. You’ll need to get llvm, clang, and the extra clang tools (you won‘t need Compiler-RT or libcxx). If you don’t have it, you'll also need to install cmake as a part of this process.
Instead of building with
"Unix Makefiles", generate build files for Ninja with
cmake -GNinja ../llvm
Then, instead of using
make, use ninja to build the clang-tidy binary with
This binary will be at (build)/bin/clang-tidy.
If you intend to use the
fix feature of clang-tidy, you'll also need to build the
Running clang-tidy is (hopefully) simple.
ninja -C out/Release chrome
ninja -C out/Release -t compdb objcxx cxx > compile_commands.json
<PATH_TO_LLVM_SRC>/tools/clang/tools/extra/clang-tidy/tool/run-clang-tidy.py \ -p ../.. \# Set the root project directory, where compile_commands.json is. # Set the clang-tidy binary path, if it's not in your $PATH. -clang-tidy-binary <PATH_TO_LLVM_BUILD>/bin/clang-tidy \ # Set the clang-apply-replacements binary path, if it's not in your $PATH # and you are using the `fix` behavior of clang-tidy. -clang-apply-replacements-binary \ <PATH_TO_LLVM_BUILD>/bin/clang-apply-replacements \ # The checks to employ in the build. Use `-*` to omit default checks. -checks=<CHECKS> \ -header-filter=<FILTER> \# Optional, limit results to only certain files. -fix \# Optional, used if you want to have clang-tidy auto-fix errors. chrome/browser # The path to the files you want to check. Copy-Paste Friendly (though you'll still need to stub in the variables): <PATH_TO_LLVM_SRC>/tools/clang/tools/extra/clang-tidy/tool/run-clang-tidy.py \ -p ../.. \ -clang-tidy-binary <PATH_TO_LLVM_BUILD>/bin/clang-tidy \ -clang-apply-replacements-binary \ <PATH_TO_LLVM_BUILD>/bin/clang-apply-replacements \ -checks=<CHECKS> \ -header-filter=<FILTER> \ -fix \ chrome/browser
*It‘s not clear which, if any,
gn flags may cause issues for
clang-tidy. I’ve had no problems building a component release build, both with and without goma. if you run into issues, let us know!
If you see errors like
src/build/linux/debian_wheezy_amd64-sysroot/usr/include/wchar.h:40:11: error: 'stdarg.h' file not found [clang-diagnostic-error]
then you should also build the
clang-headers target in your llvm checkout. This is fixed by http://reviews.llvm.org/D22046, so if your llvm checkout is up-to-date, this shouldn't be needed.