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

[Nathan James via Phabricator via cfe-commits]

njames93 added inline comments.


================
Comment at: clang-tools-extra/clangd/TidyProvider.h:25
+                                                /*Filename=*/llvm::StringRef,
+                                                /*CWD*/ PathRef) const>;
+
----------------
sammccall wrote:
> CWD is now effectively unused and can be dropped
So I tried removing CWD and it leads to assertions failing in the tests(the ones with the extension `.test`). 
I do know that the TFS has some issues in those tests anyway:
```
[build] E[16:04:40.948] VFS: failed to set CWD to /clangd-test: No such file or directory.```
I could remove the CWD but not have assertions, instead just bail out if we encounter those cases, maybe with a log. WDYT?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91029