[PATCH] D83224: [clangd] Move clang-tidy check modifications into ClangdServer

Tue Jul 28 07:56:12 PDT 2020

kadircet added a comment.

In D83224#2133457 <https://reviews.llvm.org/D83224#2133457>, @njames93 wrote:

> Do you think there should be a hidden command line option to disable disabling blacklisted checks, purely to prevent hindering attempts to fix these problematic checks.

I would expect those to be fixed and verified with unittests that are using TestTU, so the changes in ClangdServer shouldn't really be relevant to them. And if developer is trying to test something locally, i think it is better for them to just update the blacklist file(they are going to end up doing that anyways and perform possibly multiple compiles), so a flag to change blacklists behaviour might not be that meaningful.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83224