[PATCH] D117529: [clangd][NFC] Cache ClangTidy check globs to speed up createChecks

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 18 12:01:16 PST 2022


njames93 added a comment.

In D117529#3251949 <https://reviews.llvm.org/D117529#3251949>, @carlosgalvezp wrote:

> Haven't looked much in detail so apologies if my comment is stupid - can't CachedGlobList be used for this purpose? Should be a one-liner change I think.

Not a stupid question, but `CachedGlobList` is doing something very different. That's just caching the result of strings when they are passed to a glob to simplify re-evaluation of the same string.
However this implementation is about avoiding the need to check any check-names when building the list of checks. It's also thread safe unlike `CachedGlobList`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117529



More information about the cfe-commits mailing list