[PATCH] D54945: This commit adds a chapter about clang-tidy integrations

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 27 09:32:34 PST 2018


sammccall added a comment.

In D54945#1309467 <https://reviews.llvm.org/D54945#1309467>, @JonasToth wrote:

> I like the overview, maybe a link to `clangd` here might help, as there is currently a lot of effort of integrating `clang-tidy` into it. (@sammccall WDYT?)


This would be great! The integration is not yet complete, so I'm not sure if it's best to add now or later.

For the table:

- on-the-fly inspection: yes (I think - this means showing fixes overlaid on the code as you edit, right?)
- check list configuration (GUI): no
- options to checks (GUI): no
- configuration via .clang-tidy file: this will be the supported mechanism. Not done yet.
- custom clang-tidy binary: no

The behavior today is that three hard-coded checks are always enabled, as a proof-of-concept. Configuration via .clang-tidy is the main missing piece.
Clang-tidy diagnostics are shown along with others, as-you-type. Fixes can be applied.
Static analyzer checks will probably never be supported.


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54945/new/

https://reviews.llvm.org/D54945





More information about the cfe-commits mailing list