sammccall added a comment. Can you motivate this change a bit, and add tests?
================ Comment at: clangd/index/SymbolCollector.h:61 + /// AST matchers require non-const ASTContext. + static bool shouldCollectSymbol(const NamedDecl &ND, ASTContext &ASTCtx); + ---------------- in principle, this should take const Options&? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D48418 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits