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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
