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