[PATCH] D115848: tidy-llvm

2023-01-12 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri resigned from this revision. lebedev.ri added a comment. This revision now requires review to proceed. Herald added a subscriber: StephenFan. Herald added a reviewer: njames93. Herald added a project: All. This review may be stuck/dead, consider abandoning if no longer relevant. Removin

[PATCH] D115848: tidy-llvm

2022-01-19 Thread Chris Bieneman via Phabricator via cfe-commits
beanz added a comment. Sorry I’m a bit late here. If you want to use clang-tidy on a CMake built project (like LLVM) you can use CMake’s built-in support for clang-tidy: https://cmake.org/cmake/help/latest/variable/CMAKE_LANG_CLANG_TIDY.html This will run clang-tidy during the project build wit

[PATCH] D115848: tidy-llvm

2021-12-15 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri requested changes to this revision. lebedev.ri added a comment. This revision now requires changes to proceed. Just use compilation database? https://github.com/llvm-mirror/clang-tools-extra/blob/1c8cadde7ea4ca20a449edcffe10d23b612fe5d6/clang-tidy/tool/run-clang-tidy.py#L100 Repositor

[PATCH] D115848: tidy-llvm

2021-12-15 Thread Chris Sears via Phabricator via cfe-commits
CSears created this revision. CSears added a reviewer: jroelofs. Herald added a subscriber: carlosgalvezp. CSears requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Using clang-tidy on LLVM itself (the proverbial eating your own do