[PATCH] D52089: [clangd] Get rid of AST matchers in SymbolCollector. NFC

[Marc-Andre Laperle via Phabricator via cfe-commits]

malaperle added a comment.

In https://reviews.llvm.org/D52089#1235851, @ioeric wrote:

> In https://reviews.llvm.org/D52089#1235777, @malaperle wrote:
>
> > why?
>
>
> I wanted to get some numbers and update the patch summary, but somehow forgot. Sorry about that and thanks for asking!
>
> The AST matcher pops up in performance profile. Although not the most expensive thing we could optimize, this still brings `indexAST` latency from ~5.3s to ~4.4s for a large TU with big preamble.


Thanks! That's what I was looking for :) Sounds good to me.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52089