[PATCH] D91029: [clangd] Implement clang-tidy options from config

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 12 04:33:54 PST 2020


njames93 updated this revision to Diff 304780.
njames93 marked 4 inline comments as done.
njames93 added a comment.

Reworked large chunks of this:

- Renamed ClangdTidyProvider->TidyProvider.
- Removed the obselete interface mirroring ClangTidyOptionsProvider, it wasn't needed.
- Incorporated the ThreadsafeFS inside the provider.
- Removed some unnecessary changes.
- Removed the internal caching of the ThreadsafeFileTidyProvider.
- Trimmed down inheritance hierarchy.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91029

Files:
  clang-tools-extra/clangd/CMakeLists.txt
  clang-tools-extra/clangd/ClangdServer.cpp
  clang-tools-extra/clangd/ClangdServer.h
  clang-tools-extra/clangd/Compiler.h
  clang-tools-extra/clangd/ParsedAST.cpp
  clang-tools-extra/clangd/TidyProvider.cpp
  clang-tools-extra/clangd/TidyProvider.h
  clang-tools-extra/clangd/tool/Check.cpp
  clang-tools-extra/clangd/tool/ClangdMain.cpp
  clang-tools-extra/clangd/unittests/CMakeLists.txt
  clang-tools-extra/clangd/unittests/ClangdTests.cpp
  clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
  clang-tools-extra/clangd/unittests/ParsedASTTests.cpp
  clang-tools-extra/clangd/unittests/TestTU.cpp
  clang-tools-extra/clangd/unittests/TestTU.h
  clang-tools-extra/clangd/unittests/TestTidyProvider.cpp
  clang-tools-extra/clangd/unittests/TestTidyProvider.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91029.304780.patch
Type: text/x-patch
Size: 33006 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201112/e0234a86/attachment-0001.bin>


More information about the cfe-commits mailing list