HomePhabricator

[clangd] Support clang-tidy configuration in clangd.

Description

[clangd] Support clang-tidy configuration in clangd.

Summary:
This patch adds some basic supports for clang-tidy configurations in clangd:

  • clangd will respect .clang-tidy configurations for each file
  • we don't aim to support all clang-tidy options in clangd, only a small subset of condfigurations (options related to which checks will be enabled) are supported.
  • add a clang-tidy-checks CLI option that can override options from .clang-tidy file

Reviewers: ilya-biryukov, sammccall

Reviewed By: sammccall

Subscribers: javed.absar, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits

Differential Revision: https://reviews.llvm.org/D55256

Details

Event Timeline

amyk added a subscriber: amyk.Jan 22 2019, 10:32 AM

This is causing buildbot failures for PowerPC: http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage

This buildbot seems to have had two separate errors: a build error and a ninja check error. It seems like this change caused the ninja check error. However, the original build error would have hidden the ninja check error and so you may not have received an email about it.