[PATCH] D88172: [clangd] Extract common file-caching logic from ConfigProvider.

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added inline comments.


================
Comment at: clang-tools-extra/clangd/unittests/support/FileCacheTests.cpp:15
+#include <atomic>
+#include <chrono>
+
----------------
njames93 wrote:
> kbobyrev wrote:
> > Please sort the includes here (clangd should have a Code Action here).
> The includes are sorted, this is a bug in the include order check that was fixed in D91602, but the pre merge bot isnt using the updated version.
Ahh, I didn't know that and I had some of the valid warnings in my patches. I see, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88172